Development #69942
Ne pas accéder à connection dans Thread.__init__
0%
Description
Ça clashe avec asgiref utilisé par Django 3.2 qui fait un appel à currentthread lors de l'initialisation de son propre objet thread/async local.
Pour cela je pense qu'il suffira de déplacer l'initialisation de Thread.tenant dans Thread.start (à valider).
Cf. #67760 et #69941
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Lié à Development #67760: fonctionner avec django 3.2 ajouté
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-misc-move-tenant-conservation-in-Thread.start-69942.patch 0001-misc-move-tenant-conservation-in-Thread.start-69942.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par A. Berriot il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Cette petite indentation subtile de t1.start()
et de t2.start()
mais qui a l'air de corriger le problème, magnifique !
Ack pour moi, je testerai sur mon patch hobo django 3 à l'occasion.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Agate Berriot a écrit :
Cette petite indentation subtile de
t1.start()
et det2.start()
mais qui a l'air de corriger le problème, magnifique !
Les tests sont inutilement génériques, le cas d'un thread créé dans le contexte d'un tenant mais lancé hors du contexte n'arrive pas, et si ça arrive c'est un bug.
Ack pour moi, je testerai sur mon patch hobo django 3 à l'occasion.
Ok.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 11ad397384591a968f3a85528eb9c87fd50f2f60 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Thu Oct 6 11:11:37 2022 +0200 misc: move tenant conservation in Thread.start (#69942)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
misc: move tenant conservation in Thread.start (#69942)