PyCQA/bandit

yaml_load should not be B5xx cryptography group

Ouverte

#306 ouverte le 14 mai 2018

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)Python (559 forks)batch import
buggood first issue

Métriques du dépôt

Stars
 (5 660 étoiles)
Métriques de merge PR
 (Merge moyen 11h 55m) (1 PR mergée en 30 j)

Description

Describe the bug The yaml_load plugin has bandit ID B506. The 5xx group according to [1] is defined as the group for cryptography. This plugin would be more appropriate as a type of injection B6xx

To Reproduce n/a

Expected behavior n/a

Bandit version

bandit 1.4.0

Additional context Add any other context about the problem here.

Guide contributeur

Direction de recherche
Le plugin yaml load utilise actuellement l'ID bandit B506, qui appartient au groupe cryptographie (B5xx). Selon l'issue, il devrait être sous injection (B6xx). Pour corriger, localisez la définition du plugin dans bandit/plugins/yaml load.py et changez le champ 'id' de 'B506' en un ID B6xx approprié (par exemple B601), en mettant à jour tous les fichiers de test associés. Vérifiez la documentation de bandit pour les ID B6xx existants afin d'éviter les doublons. Il n'y a pas de discussions ou PR supplémentaires ; le changement est simple et isolé.
Stack technique
python
Domaine
security
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Python
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80