biocore/empress

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

Ouverte

#216 ouverte le 26 juin 2020

Voir sur GitHubGuide contributeur
 (0 commentaire) (1 réaction) (0 personne assignée)JavaScript (32 forks)auto 404
good first issuerefactoring

Métriques du dépôt

Stars
 (56 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.)

Guide contributeur

Direction de recherche
Recherchez toutes les utilisations de innerHTML dans le code. Pour chaque occurrence où vous définissez du texte, remplacez innerHTML par textContent pour éviter les risques de sécurité et garantir un échappement correct.
Stack technique
javascript
Domaine
frontend
Type d'issue
Sécurité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptDOM
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90