Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Patch d'une vulnérabilité reliée aux dépendances du projet #9

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

aexko
Copy link
Owner

@aexko aexko commented Oct 4, 2022

La description

Si un attaquant distant était capable de contrôler l'option pretty du compilateur pug, ex: si vous diffusez un objet fourni par l'utilisateur, tel que les paramètres de requête d'une requête dans les entrées du modèle pug, il leur est possible d'exécuter du code à distance sur le backend Node.js.

Corrections

Mettre à niveau vers [email protected] ou [email protected], qui nettoient correctement le paramètre.

Type de changement

  • Correction de bogue (changement ininterrompu qui résout un problème)
  • Nouvelle fonctionnalité (changement ininterrompu qui ajoute des fonctionnalités)
  • Modification avec rupture (correction ou fonctionnalité qui empêcherait la fonctionnalité existante de fonctionner comme prévu)
  • Cette modification nécessite une mise à jour de la documentation

Comment cela a-t-il été testé ?

J'ai configuré un bot qui s'appelle Dependabot (GitHub) qui vérifie les vulnérabilités liées aux dépendances du projet. Il n'y a pas de tests nécessaires puisque le bot vérifie par lui-même de façon automatique la détection d'erreurs

  • Matériel: Dependabot

Liste de contrôle :

  • Mon code suit les directives de style de ce projet
  • Mes modifications ne génèrent aucun nouvel avertissement
  • J'ai ajouté des tests qui prouvent que mon correctif est efficace ou que ma fonctionnalité fonctionne

Références

Vulnérabilité d'injection de code dans visitMixin et visitMixinBlock via l'option pretty : pugjs/pug#3312

@aexko aexko requested review from NarguSabz and Lyna-Abd October 4, 2022 19:30
@aexko aexko merged commit af69f7a into main Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant