RipMeApp/ripme

Implement subdirectory sanitisation

Open

#16 aberto em 9 de jul. de 2017

Ver no GitHub
 (10 comments) (0 reactions) (0 assignees)Java (640 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (3.524 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
Examine o método addURLToDownload em AbstractRipper.java. Implemente uma função de sanitização que remova ou substitua caracteres inválidos do sistema de arquivos (por exemplo, <, >, :, ", /, \, |, ?, *) dos nomes dos subdiretórios. Considere usar verificações específicas da plataforma para nomes de arquivo, como File.separatorChar ou um padrão regex. Veja quaisquer arquivos de teste existentes para métodos ripper e considere adicionar um teste de unidade para a lógica de sanitização. Verifique os comentários da issue para quaisquer soluções propostas ou discussões.
Pilha de tecnologia
java
Domain
tooling
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75