Produits Entr'ouvert » w.c.s.

Ça vient vraisemblablement de #98182; dans les traces liées il y a :

subprocess, gm convert +profile "*" HEIC:- JPEG:-

je ne sais pas si c'est une information fiable.

Ok j'ai compris le truc, PicklableUpload descend de Upload dans quixote.http_request, celui-ci définit Upload.receive() (la fonction principale pour ajouter du contenu à un objet Upload) ainsi :

    def receive(self, lines):
        self.fp = tempfile.TemporaryFile("w+b")
        for line in lines:                                                                                                                                                                                   
            self.fp.write(line)
        self.fp.seek(0)

Un fichier créé via tempfile.TemporaryFile n'a pas de nom fd.name est remplacé par l'entier du descripteur de fichier associé:

>>> fd = tempfile.TemporaryFile('w+b')
>>> fd.name
3
>>> type(fd.name)
<class 'int'>
>>> fd.close()
>>> open(fd.name)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
OSError: [Errno 9] Bad file descriptor

et quand on passe un entier à open je suppose qu'il essaie bête de recréer un objet fichier python autour sans même le dupliquer:

>>> fd = tempfile.TemporaryFile('w+b')
>>> fd2 = open(fd.name)
>>> fd.fileno()
3
>>> fd.name
3

et donc s'il est déjà fermé ou fermé plus tard (ce qui arrive je suppose dans le cas atomic_write()/read()) boom Bad file descriptor.

On ne peut pas faire upload.fd.close() et supposer que quoi que ce soit de upload.fd (comme upload.fd.name) soit encore utilisable.

Benjamin Dauvergne (bdauvergne) a demandé une relecture de Frédéric Péters (fpeters) sur une pull request sur Gitea concernant cette demande :

• URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2028