lbryio/lbry-sdk

Ensure that default download directory is writable on daemon startup

Open

#3.270 aberto em 20 de abr. de 2021

Ver no GitHub
 (10 comments) (0 reactions) (0 assignees)Python (500 forks)batch import
area: downloadsgood first issuehelp wantedtype: improvement

Métricas do repositório

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

Description

Creating a new issue based on lbryio/lbry-sdk#877

  • Update ensure_directory_exists() to check that the directory is writable by the current process.
  • If it's not, return a useful error
  • Add tests to cover the case when
    • download directory does not exist
    • download dir exists but is not writable
    • dir exists and is writable
    • the path refers to a file not a directory

bonus: also check if disk is full when sdk starts. this check should be outside ensure_directory_exists(). it probably goes somewhere near here

Guia do colaborador

Direção de pesquisa
Atualize `ensure directory exists()` em `lbry/extras/cli.py` para verificar se o diretório de download é gravável e retornar um erro útil caso não seja. Adicione testes unitários que cubram os casos: diretório não existe, existe mas não é gravável, existe e é gravável, e o caminho é um arquivo. Opcionalmente, adicione uma verificação de espaço em disco perto da linha 296. Consulte o problema relacionado lbryio/lbry sdk#877 e os padrões de teste existentes no repositório.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Bug
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.
1-3 horas
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
PythonGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75