OWASP/OpenCRE

Input validation missing on import csv functionality

Ouverte

#554 ouverte le 18 sept. 2024

Voir sur GitHubGuide contributeur
 (8 commentaires) (0 réaction) (1 personne assignée)Python (116 forks)auto 404
GSOCenhancementgood first issue

Métriques du dépôt

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

Description

Issue

When importing a new standard, no validation is performed on the imported csv file, a generic non-descriptive "500 - Internal Server Error" is returned or new CREs are wrongfully injected.

More specifically, in the outlined case, if the format of "CRE 0" column is XX-XXX| instead of XXX-XXX|, a non-descriptive error is returned. Also, I noticed that if in the "<standard_name>|name" column the requirement's text is enclosed between three double quotes '"""', the csv is treated as valid and the whole row is entered as a new root CRE.

image

Guide contributeur

Direction de recherche
Inspectez la fonctionnalité d'importation CSV dans le code, en particulier l'analyse et la validation de la colonne « CRE 0 ». Tracez la gestion des erreurs lorsque le format est incorrect et étudiez comment les triples guillemets dans le texte sont traités. Recherchez les tests existants pour l'importateur et les éventuels utilitaires de validation.
Stack technique
python
Domaine
backend
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.
1-3 heures
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
GitPython
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75