Ensure that default download directory is writable on daemon startup

(10 commentaires) (0 réaction) (0 personne assignée)Python (500 forks)batch import

area: downloadsgood first issuehelp wantedtype: improvement

Métriques du dépôt

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

Direction de recherche: Mettez à jour `ensure directory exists()` dans `lbry/extras/cli.py` pour vérifier que le répertoire de téléchargement est accessible en écriture et renvoyer une erreur utile dans le cas contraire. Ajoutez des tests unitaires couvrant les cas suivants : le répertoire n'existe pas, existe mais n'est pas accessible en écriture, existe et est accessible en écriture, et le chemin est un fichier. Optionnellement, ajoutez une vérification de l'espace disque près de la ligne 296. Consultez l'issue associée lbryio/lbry sdk#877 et les modèles de tests existants dans le dépôt.
Stack technique: python
Domaine: backend
Type d'issue: Bug
Difficulté: 2
Temps estimé: 1-3 heures
Statut d'activité: Active
Clarté: Claire
Prérequis: PythonGit
Accessibilité débutant: 75