biocore/empress

When setting text in the DOM, use textContent instead of innerHTML

Aperta

#216 aperta il 26 giu 2020

Vedi su GitHubGuida contributor
 (0 commenti) (1 reazione) (0 assegnatari)JavaScript (32 fork)auto 404
good first issuerefactoring

Metriche repository

Star
 (56 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

See the MDN docs on some of the downsides of setting things with innerHTML here.

This shouldn't be a huge problem since Empress visualizations are (as of writing) inherently client-side applications, but there's the potential for users to break things if their data includes bizarre names -- for example, a metadata column is named <b>i'm a problematic metadata column</b>, or something silly like that. (Also, more realistically, backslashes or ampersands might also cause problems with innerHTML.)

Guida contributor

Direzione di ricerca
Cerca tutti gli usi di innerHTML nel codice. Per ogni occorrenza in cui si imposta del testo, sostituisci innerHTML con textContent per evitare rischi di sicurezza e garantire una corretta escape.
Tech stack
javascript
Dominio
frontend
Tipo issue
Sicurezza
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptDOM
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90