Bug #26010
mdel: crash sur un int()
Début:
01 septembre 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Il y a sans doute une vérification préalable ou un try/except à faire
Subject: [passerelle.test] ERROR (EXTERNAL IP): Error occurred while processing request
Error occurred while processing request
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/passerelle/utils/jsonresponse.py", line 120, in api
resp = f(*args, **kwargs)
File "/usr/lib/python2.7/dist-packages/passerelle/views.py", line 345, in perform
result = self.endpoint(request, **params)
File "/usr/lib/python2.7/dist-packages/passerelle/contrib/mdel/models.py", line 134, in status
status = demand.get_status()
File "/usr/lib/python2.7/dist-packages/passerelle/contrib/mdel/models.py", line 338, in get_status
path = max(zfiles, key=fnum) # get the file with the highest number
File "/usr/lib/python2.7/dist-packages/passerelle/contrib/mdel/models.py", line 335, in fnum
return int(fname.split('--')[-1])
ValueError: invalid literal for int() with base 10: '1 - copie2'
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 5 ans
C'est lié à la présence de fichier du genre "outputs/136-3-aec-la--1 - copie2.zip" ... il faut les exclure lors du scan :
for zfile in os.listdir(output_dir):
if zfile.lower().startswith(self.demand_id.lower()) and zfile.lower().endswith('.zip'): <-- ce if laisse passer de la saleté
zfiles.append(zfile)
ie être plus restrictif sur le format, qui doit être <demand_id>-<flow_type>--<num>.zip comme indiqué 3 lignes en dessous
Mis à jour par Thomas Noël il y a plus de 5 ans
- Fichier 0001-mdel-ignore-zip-file-with-invalid-step-number-26010.patch 0001-mdel-ignore-zip-file-with-invalid-step-number-26010.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 5 ans
(on voit pas bien via redmine qu'il y a un test au travers de l'ajout de mdel/test/outputs/1-14-ILE-LA--4bog.zip)
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Si, on le voit.
Mis à jour par Thomas Noël il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 346929d2ffa42123ee49b879e8ce69f62c67405c
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Mon Sep 3 10:05:37 2018 +0200
mdel: ignore zip file with invalid step number (#26010)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
mdel: ignore zip file with invalid step number (#26010)