MerginMaps / MerginMaps/python-api-client

Cleanup of failed download may misbehave

Ouverte
#157 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

bug
Langage dominant
Python
Étoiles
23
Forks
12
Merge moyen
2 j 13 h
PR mergées (30 j)
7

Description

Sometimes if a download fails, the subsequent cleanup of the project directory may fail as well: https://github.com/MerginMaps/qgis-mergin-plugin/issues/442#issuecomment-1212909680
While it looks like some other app is using the downloaded files, it is possible that we are doing the cleanup too early - e.g. in download_project_is_running() we do the cleanup while some other futures may be still active: we should wait until all jobs are done before doing any cleanup.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par download_project_is_running() et suivez le lien entre le nettoyage du répertoire du projet et les autres futures actifs. Utilisez l’issue Mergin Maps liée pour le scénario d’échec signalé. Le travail est considéré comme terminé lorsque le nettoyage attend que toutes les tâches de téléchargement soient terminées et n’échoue plus lorsqu’un téléchargement échoue.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
api
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
48/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.