Improve sanitization options: making it pluggable
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- 機能追加
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- javascript
調査の方向性
lib/html.js、特に potentiallyUnsafe を読み、HtmlInline と HtmlBlock が現在 raw HTML をどのように処理しているかを追跡してください。Issue に記載された sanitizer hook と置換動作を定義し、そのうえで table、tr、/table などの安全なタグを保持できる一方、script の内容は拒否されることを確認してください。
索引モデルが issue の本文から書いたものです。
説明
I would like to have more sanitization options for raw html in markdown. For example: i would like to allow raw html bits like <table>, <tr> and </table> but disallow evil things like <script>takeOverTheWorld();</script>. Right now: we can either allow or disallow all raw html. If we make the sanitation pluggable, the user can choose which sanitation library he/she wants to plug in.
The most straightforward way to make sensitization pluggable would be to provide a function that does what 'potentiallyUnsafe' in lib/html.js does, plus a sanitizeRawHtml function that could be used for HtmlInline and HtmlBlock instead of just always omitting a comment '<!-- raw HTML omitted -->'.
This would be a minimum modification from what we have. I will create a pull request for this.
- 主要言語
- JavaScript
- スター
- 1.6k
- フォーク
- 234
- 平均マージ
- 1日 27分
- マージ済み PR（30日）
- 1
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
commonmark/commonmark.js のほかの issue
-
White space handling looks inconsistent between HTML block element vs HTML inline element対応中かも @mamicicekel が 13 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
commonmark/commonmark.js#303 · コメント 2 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
commonmark/commonmark.js#287 · リアクション 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 20/100
commonmark/commonmark.js#275 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
commonmark/commonmark.js#267 · コメント 3 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
commonmark/commonmark.js#264 · リアクション 1 件 ·
commonmark/commonmark.js の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
murdos/musicbrainz-userscripts#1150 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
sindresorhus/eslint-plugin-unicorn#3921 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
web-scrobbler/web-scrobbler#6266 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
exa-studio/ApiVault#329 ·