RipMeApp/ripme

Implement subdirectory sanitisation

Open

#16 aperta il 9 lug 2017

Vedi su GitHub
 (10 commenti) (0 reazioni) (0 assegnatari)Java (640 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (3524 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina il metodo addURLToDownload in AbstractRipper.java. Implementa una funzione di sanificazione che rimuova o sostituisca i caratteri non validi per il filesystem (ad es. <, >, :, ", /, \, |, ?, *) dai nomi delle sottodirectory. Considera l'uso di controlli specifici per la piattaforma per i nomi dei file, come File.separatorChar o un pattern regex. Controlla eventuali file di test esistenti per i metodi ripper e considera l'aggiunta di un test unitario per la logica di sanificazione. Controlla i commenti dell'issue per eventuali soluzioni proposte o discussioni.
Tech stack
java
Dominio
tooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
JavaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75