Projet

Général

Profil

Bug #13509

courriers : vérifier un rien les fichiers passés à la commande de feed

Ajouté par Thomas Noël il y a plus de 7 ans. Mis à jour il y a environ 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
07 octobre 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

From: Cron Daemon <root@welco.test.entrouvert.org>
To: root@welco.test.entrouvert.org
Subject: Cron <welco@welco> test -x /usr/local/bin/welco-ftp-download-alfortville && /usr/local/bin/welco-ftp-download-alfortville

gm convert: Postscript delegate failed (/var/lib/welco/tenants/accueil2016.alfortville.fr/media/3D2C.tmp.pdf).

Le PDF concerne est mauvais (vide).

Je fais un ticket pour me demander que faire dans ce cas... pour l'instant ça arrête le process de feed complétement, parce qu'il y a un "set -e" dans le script welco-ftp-download-alfortville

Peut-être ajouter un || echo "erreur d'import de $pdf" && true à la fin de :

$FEEDMAILCMD --category "$category" "$pdf" && rm "$pdf"

Fichiers

0001-mail-check-files-are-PDF-before-importing-them-13509.patch (1005 octets) 0001-mail-check-files-are-PDF-before-importing-them-13509.patch Frédéric Péters, 13 novembre 2016 21:29

Révisions associées

Révision 24821438 (diff)
Ajouté par Frédéric Péters il y a plus de 7 ans

mail: check files are PDF before importing them (#13509)

Historique

#1

Mis à jour par Thomas Noël il y a plus de 7 ans

En fait je pense aussi que le problème est dans welco : le document a été importé mais il n'aura pas de miniature, sans doute faudrait-il ne pas l'importer (ou le retirer) ?

#2

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Sujet changé de welco: crash lors du feed d'un document à courriers : vérifier un rien les fichiers passés à la commande de feed

Ok, je renomme le ticket.

Aujourd'hui il y a zéro vérif, il suffit que le fichier existe :

        for filepath in args:
            if not os.path.exists(filepath):
                continue
            mail = Mail(content=File(open(filepath)))
#3

Mis à jour par Frédéric Péters il y a plus de 7 ans

Vérification légère que c'est bien un fichier PDF.

Reste que ça retournera un code != 0, à gérer dans le script d'import.

#4

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 24821438509ffc605d470a4a20e8736a9a02b305
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Nov 13 21:27:30 2016 +0100

    mail: check files are PDF before importing them (#13509)
#5

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF