Improve sanitization options: making it pluggable
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
Línea de trabajo
Lea lib/html.js, especialmente potentiallyUnsafe, y rastree cómo HtmlInline y HtmlBlock gestionan actualmente el HTML sin procesar. Defina el hook del sanitizer y el comportamiento de reemplazo descritos en el issue, y luego verifique que se puedan conservar etiquetas seguras como table, tr y /table, mientras se rechaza el contenido de script.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- 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 11 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
-
area:plugins enhancement platform:vscode
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
anthropics/claude-code#103166 ·
Los mantenedores suelen responder en 1 día
-
Add: Kids Zone LogoAbiertocheck:failed logos:add
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
-
[workshop-sim] Improve: Add verifiable checkpoint and bridge to Step 5 (What Are Agentic Workflows?)Abiertofeedback simulation workshop
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
githubnext/gh-aw-workshop#4715 ·
Los mantenedores suelen responder en 1 día
-
getComputedStyle ignores a rule whose selector list contains a pseudo-element (regression in 30.x)Posiblemente ocupada @chiragjangiir la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
Los mantenedores suelen responder en 1 día
-
external-plugin external-plugin-canvas needs-review:MEDIUM ready-for-review
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
github/awesome-copilot#5153 · 2 comentarios ·
Los mantenedores suelen responder en 1 día