francois-dorin / previnpact-extension

MIT License
2 stars 2 forks source link

Scan secu #11

Open Wivik opened 7 months ago

Wivik commented 7 months ago

Hello,

Je me suis permis d'activer le scan CodeQL sur mon fork (vilaine déformation professionnelle) et il a relevé deux alertes medium. Les analyser va au delà de mes compétences par contre. Je te suggère d'activer Dependabot/CodeQL/Secret scanning dans les params secu du repo. Ces features sont gratuites pour les projets publics.

Egalement, tu peux aussi activer Dependabot version update pour qu'il te propose des mises à jour des dépendances node (non liées à la sécu).

francois-dorin commented 7 months ago

Je viens de les activer, au cas où. Il y a 2 erreurs, mais qui sont en fait la même :

L'alerte qui reste est que le code interprète directement des éléments du DOM en HTML. C'est normal, c'est au niveau de la réorganisation des commentaires.

D'un point de vue pratique, les risques sont très faibles :

Et le jour où Next proposera directement le choix de l'ordre des commentaires, je pourrais supprimer le code, et donc la vulnérabilité ;)

Wivik commented 7 months ago

Oui je me disais bien qu'une medium n'allait pas être bien méchante. :)