RipMeApp/ripme

Implement subdirectory sanitisation

Open

#16 ouverte le 9 juil. 2017

Voir sur GitHub
 (10 commentaires) (0 réactions) (0 assignés)Java (640 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (3 524 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

The addURLToDownload method in AbstractRipper should have sanitisation implemented for sub directories. It's currently possible to pass invalid characters and have the directory creation fail. This is particularly an issue on Windows platforms.

Another side effect of no sanitisation is that slashes in the directory name create further subdirectories.

Guide contributeur

Direction de recherche
Examinez la méthode addURLToDownload dans AbstractRipper.java. Implémentez une fonction de nettoyage qui supprime ou remplace les caractères invalides pour le système de fichiers (par exemple <, >, :, ", /, \, |, ?, *) des noms de sous répertoires. Envisagez d'utiliser des vérifications spécifiques à la plateforme pour les noms de fichiers, comme File.separatorChar ou un motif regex. Consultez les fichiers de test existants pour les méthodes ripper et envisagez d'ajouter un test unitaire pour la logique de nettoyage. Vérifiez les commentaires de l'issue pour toute solution proposée ou discussion.
Stack technique
java
Domaine
tooling
Type d'issue
Fonctionnalité
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.
Une demi journée
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
JavaGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75