OWASP/OpenCRE

Input validation missing on import csv functionality

Offen

#554 geöffnet am 18.09.2024

Auf GitHub ansehenContributor Guide
 (8 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Python (116 Forks)auto 404
GSOCenhancementgood first issue

Repository-Metriken

Stars
 (167 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die CSV Importfunktionalität im Code, insbesondere das Parsen und Validieren der Spalte 'CRE 0'. Verfolgen Sie, wie Fehler behandelt werden, wenn das Format falsch ist, und untersuchen Sie, wie dreifache Anführungszeichen im Text behandelt werden. Suchen Sie nach vorhandenen Tests für den Import und nach Validierungshilfen.
Tech Stack
python
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitPython
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75