Link destination parsing accepts control characters (but should not)
Nadie ha tomado este issue todavía.
- #309 de @cpruijsen — cerrado sin fusionar
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- tooling
Línea de trabajo
Comienza con la reproducción de la JavaScript API en el issue y sigue el parser escrito a mano para destinos de enlaces que esta ejercita. Usa la regla de la especificación CommonMark sobre caracteres de control como comportamiento esperado; se considera terminado cuando el carácter de control no se acepta como destino y ya no se produce el href mostrado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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>
"
- Lenguaje dominante
- JavaScript
- Estrellas
- 1.6k
- Forks
- 234
- Merge medio
- 1 d 27 min
- PR fusionados (30 d)
- 1
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de commonmark/commonmark.js
-
White space handling looks inconsistent between HTML block element vs HTML inline elementPosiblemente ocupada @mamicicekel la tomó hace 14 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
commonmark/commonmark.js#303 · 2 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
commonmark/commonmark.js#287 · 1 reacción ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 20/100
commonmark/commonmark.js#275 ·
-
Delegate node walking to othersAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
commonmark/commonmark.js#267 · 3 comentarios ·
-
Inconsistent escape of image altAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
commonmark/commonmark.js#264 · 1 reacción ·
Todos los issues de commonmark/commonmark.js
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 2 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
swc-project/swc-next#1022 ·
Los mantenedores suelen responder en 1 día
-
area-clientside-dartpad
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
alphagov/public-asset-checker#378 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
headers-parser: `forRegExp` does not escape regexp characters in path segments that contain `*`Posiblemente ocupada @RaphaelFakhri la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día