Bug #21731
AttachmentEvolutionPart: le filename n'est pas relatif au publisher.app_dir
Début:
08 février 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Dans AttachmentEvolutionPart il y a cette partie :
def __getstate__(self):
odict = self.__dict__.copy()
if not odict.has_key('fp'):
return odict
del odict['fp']
dirname = os.path.join(get_publisher().app_dir, 'attachments')
if not os.path.exists(dirname):
os.mkdir(dirname)
if not 'filename' in odict:
filename = file_digest(self.fp)
odict['filename'] = os.path.join(dirname, filename)
self.filename = odict['filename']
self.fp.seek(0)
atomic_write(self.filename, self.fp)
return odict
qui enregistre dans AttachmentEvolutionPart.filename le chemin absolu du fichier.
Et donc crash si on change l'emplacement du tenant, par exemple de /var/lib/wcs-au-quotidien vers /var/lib/wcs, ou plus simplement si on change le nom du site.
(note 1: pour l'instant ces situations ont été gérées à coup de liens symboliques)
(note 2: pas regardé si on fait pas de même par ailleurs, genre PicklableUpload)
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Assigné à mis à Frédéric Péters
- Planning mis à Non
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-misc-reference-attachments-using-a-relative-path-217.patch 0001-misc-reference-attachments-using-a-relative-path-217.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Voilà sans exécution de migration générale mais si on voulait il y aurait juste à provoquer un .store() sur tous les formdata (ce qu'on pourrait limiter à l'SQL et ça serait trivial).
Mis à jour par Serghei Mihai il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e224522eccc66b15cc303c68b9660a1967d6e868
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Oct 31 13:49:21 2021 +0100
misc: reference attachments using a relative path (#21731)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Emmanuel Cazenave il y a plus de 2 ans
- Lié à Development #60302: Comprendre pourquoi on stocke des chemins absolus de fichiers (et corriger si ça se produit encore) ajouté
misc: reference attachments using a relative path (#21731)