Link destination parsing accepts control characters (but should not)
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 48/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript
- 領域
- tooling
調査の方向性
Issue 内の JavaScript API の再現から始め、それが実行する手書きのリンク先パーサーを追跡してください。期待される動作として CommonMark 仕様の制御文字に関する規則を使用します。制御文字がリンク先として受け入れられず、表示された href が生成されなくなれば完了です。
索引モデルが issue の本文から書いたものです。
説明
From the spec:
A link destination consists of either
[...]
- a nonempty sequence of characters that does not include ASCII space or control characters [...]
In https://github.com/commonmark/commonmark.js/commit/531fd57c0777896471d1f9773d40c506b28e9d38, it looks like the handwritten parser doesn't stop on control characters. Saw this while looking at the changes for the new spec.
This can be verified using the JS API on dingus:
new commonmark.HtmlRenderer().render(new commonmark.Parser().parse("[a](\x01)"))
Results in:
"<p><a href=\"%01\">a</a></p>
"
- 主要言語
- JavaScript
- スター
- 1.6k
- フォーク
- 231
- 平均マージ
- 2日 17時間
- マージ済み PR（30日）
- 2
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
commonmark/commonmark.js のほかの issue
-
White space handling looks inconsistent between HTML block element vs HTML inline element対応中かも @mamicicekel が 3 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
commonmark/commonmark.js#303 · コメント 2 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
commonmark/commonmark.js#298 ·
-
難易度 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 件 ·
commonmark/commonmark.js の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
no-human-ai/no_human#659 ·
メンテナーはふだん 1 日以内に返信
-
[BUG] Multi-day events show "Ended" while still in progress対応中かも @tarunagnihotri534 が今日担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
data-umbrella/du-event-board#231 · コメント 2 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
-
IO.get_env on Node truncates names at embedded NUL対応中かも @Yi-111-a が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
HigherOrderCO/Bend#1449 · コメント 1 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
cryptpad/documentation#162 ·