OWASP/OpenCRE

Input validation missing on import csv functionality

Aperta

#554 aperta il 18 set 2024

Vedi su GitHubGuida contributor
 (8 commenti) (0 reazioni) (1 assegnatario)Python (116 fork)auto 404
GSOCenhancementgood first issue

Metriche repository

Star
 (167 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina la funzionalità di importazione CSV nel codebase, in particolare l'analisi e la validazione della colonna 'CRE 0'. Traccia come vengono gestiti gli errori quando il formato non è corretto e indaga come vengono trattate le triple virgolette nel testo. Cerca test esistenti per l'importatore e eventuali helper di validazione.
Tech stack
python
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75