microsoft/vasim

Reading in CSVs with glob

Ouverte

#18 ouverte le 30 août 2024

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)Python (7 forks)auto 404
buggood first issuerefactor

Métriques du dépôt

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

Description

We are currently reading in all of the CSVs of the performance trace data using python's glob (list(self.data_dir.glob("**/*.csv"))) with no security checks. It seems like we need to do a bit more here in terms of sanitizing inputs.

Guide contributeur

Direction de recherche
Ajoutez une validation d'entrée pour restreindre les chemins de fichiers aux répertoires et motifs attendus, par exemple en utilisant une liste blanche des chemins autorisés ou en nettoyant le motif glob.
Stack technique
python
Domaine
backendsecurity
Type d'issue
Sécurité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
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.
70