Possible injection because of sanitize cache

Offen

#34 erstellt am 21.1.2022 4 Kommentare 8 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 38/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: javascript
Bereich: frontend, security

Rechercherichtung

Beginne mit der JavaScript-Reproduktion im Issue und vergleiche ihre drei Renderings mit dem verlinkten CodePen. Verfolge das Verhalten des Sanitizer-Caches, das für die Wiederverwendung des zuvor gerenderten -Inhalts verantwortlich ist; als erledigt gilt die Aufgabe, wenn das dritte Rendering den String wie das erste escaped und gleichzeitig das direkte HTML-Rendering beibehält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Basically it's possible to inject dirty html:

const striked = '<strike>test</strike>';

console.log(<div>{striked}</div>);
console.log(<div><strike>test</strike></div>);
console.log(<div>{striked}</div>);

This is the output:

<div>&lt;strike&gt;test&lt;/strike&gt;</div>
<div><strike>test</strike></div>
<div><strike>test</strike></div>

Expected output:

<div>&lt;strike&gt;test&lt;/strike&gt;</div>
<div><strike>test</strike></div>
<div>&lt;strike&gt;test&lt;/strike&gt;</div>

After rendering <div><strike>test</strike></div>, it caches <strike>test</strike> and doesn't sanitize it anymore. It can be seen live here as well. Just because something was rendered before, it shouldn't mean that it's sanitized.

Vorherrschende Sprache: JavaScript
Sterne: 814
Forks: 37
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus developit/vhtml

Maintenance Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 10/100

developit/vhtml#45 · 2 Kommentare · 3 Reaktionen · Erstellt am 12.3.2024
unable to use vhtml in vite Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 25/100

developit/vhtml#42 · Erstellt am 01.7.2023
Interpolated zero values are taken as false (zeros are not diplayed) Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 20/100

developit/vhtml#40 · 1 Kommentar · 1 Reaktion · Erstellt am 25.3.2023
Why does this library exist? Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

developit/vhtml#38 · 7 Kommentare · 1 Reaktion · Erstellt am 31.12.2022
render nothing if the param is the number zero Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 38/100

developit/vhtml#37 · Erstellt am 10.10.2022

Alle Issues in developit/vhtml