lm-sys/FastChat

UI contains cross-site scripting (XSS) vulnerabilities

Open

#88 ouverte le 31 mars 2023

Voir sur GitHub
 (11 commentaires) (0 réactions) (0 assignés)Python (4 736 forks)batch import
good first issuehelp wanted

Métriques du dépôt

Stars
 (38 959 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

the UI is not filtering input/output appropriately

Guide contributeur

Direction de recherche
Examinez les composants d'interface utilisateur qui affichent les entrées utilisateur (par exemple, les messages de chat, les noms d'utilisateur) et vérifiez l'absence de désinfection de sortie. Recherchez l'utilisation directe d'innerHTML ou de dangerouslySetInnerHTML non sécurisé de React. Identifiez quelles entrées sont vulnérables en utilisant la capture d'écran comme référence. Après avoir localisé le code vulnérable, appliquez une bibliothèque de désinfection comme DOMPurify ou assurez un échappement HTML approprié. Testez avec des charges utiles XSS courantes pour confirmer que la correction fonctionne.
Stack technique
pythonjavascript
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
PythonJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60