Link destination parsing accepts control characters (but should not)
Nessuno ha ancora preso questa issue.
- #309 di @cpruijsen — chiusa senza merge
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 48/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- tooling
Direzione di ricerca
Inizia con la riproduzione della JavaScript API nell’issue e segui il parser scritto a mano per le destinazioni dei link che essa esercita. Usa la regola della specifica CommonMark sui caratteri di controllo come comportamento previsto; il lavoro è terminato quando il carattere di controllo non viene accettato come destinazione e l’href mostrato non viene più prodotto.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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>
"
- Lingua principale
- JavaScript
- Stelle
- 1.6k
- Fork
- 233
- Merge medio
- 1g 27m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di commonmark/commonmark.js
-
White space handling looks inconsistent between HTML block element vs HTML inline elementForse già presa @mamicicekel l’ha presa 11 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
commonmark/commonmark.js#303 · 2 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
commonmark/commonmark.js#287 · 1 reazione ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 20/100
commonmark/commonmark.js#275 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
commonmark/commonmark.js#267 · 3 commenti ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
commonmark/commonmark.js#264 · 1 reazione ·
Tutte le issue di commonmark/commonmark.js
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
keyxmakerx/Chronicle#1532 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement feature good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
bug lsp sev-3
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 90/100
Miaotofu01/Study-Mate#206 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 95/100
Uuriko/project-room#3998 ·
I maintainer di solito rispondono entro 1 giorno