Bug #61944
provisionning vers hobo qui échoue
0%
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/54582/
(a priori suite à #60572)
File "/usr/lib/python3/dist-packages/hobo/provisionning/middleware.py" in process_request 39. self.hobo_specific_setup() File "/usr/lib/python3/dist-packages/hobo/provisionning/middleware.py" in hobo_specific_setup 111. authentic.secret_key, local_hobo_dict['secret_key'] Exception Type: KeyError at /__provision__/ Exception Value: 'secret_key'
Fichiers
Révisions associées
tests: add some assertions about hobo json (#61944)
environment: store local hobo info in Hobo model (#61944)
Historique
Mis à jour par Frédéric Péters il y a environ 2 ans
J'ai poussé wip/61944-revert-60572 qui est une branche qui revert sauf migration et modification au modèle.
Mis à jour par Thomas Noël il y a environ 2 ans
- Assigné à mis à Emmanuel Cazenave
Je n'ai pas réussi à comprendre d'où ça vient, je remets lâchement l'affaire dans les mains d'Emmanuel...
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier 0001-tests-add-some-assertions-about-hobo-json-61944.patch 0001-tests-add-some-assertions-about-hobo-json-61944.patch ajouté
- Fichier 0002-environment-store-local-hobo-info-in-Hobo-model-6194.patch 0002-environment-store-local-hobo-info-in-Hobo-model-6194.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Je blinde d'abord de tests sur le hobo_json et ensuite rebelote sur le hobo local mais en essayant d'être aussi minimaliste que possible.
Dans l'état d'esprit minimaliste je zappe la migration au profit du get_or_create, si ça foire encore juste avec ce petit patch je démissionne.
Mis à jour par Thomas Noël il y a environ 2 ans
Au niveau de :
if 'local' in as_dict:
del as_dict['local']
tu penses que ça gêne si on le laisse ? C'est une indication presque utile dans le hobo.json, non ?
Dans get_local_hobo_dict tu as gardé le test « if not build_absolute_uri: » mais selon moi il n'a plus vraiment de sens ici.
Par rapport à #60572 j'aimais bien le Hobo::as_dict() surchargé, pourquoi n'as-tu pas repris ça ici ? Je trouve qu'on est un peu dans le micmac, à créer un dico à la main dans get_local_hobo_dict... non ?
En tout cas je vois que ton patch corrige « Exception Value: 'secret_key' » car tu ne le supprimes plus par rapport au patch précédent de #60572, c'est bien. En fait je pensais que tu repartirais de #60572 en supprimant juste ce del['secret_key'] :)
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier 0001-tests-add-some-assertions-about-hobo-json-61944.patch 0001-tests-add-some-assertions-about-hobo-json-61944.patch ajouté
- Fichier 0002-environment-store-local-hobo-info-in-Hobo-model-6194.patch 0002-environment-store-local-hobo-info-in-Hobo-model-6194.patch ajouté
En fait j'ai fait ce match dans le trauma d'avoir tout cassé et donc je ne voulais toucher à rien d'existant (genre as_dict) pour être sûr que ça passe cette fois.
Mais voilà je suis plus calme maintenant, j'ai tenu compte.
Mis à jour par Thomas Noël il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
C'est ok pour moi. A un moment j'allais proposer d'ajouter un commentaire qui explique le "as_dict" mais en fait la série de "del" permet vraimetn de comprendre que c'est un nettoyage et voilà.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 577438f7fafd579397827ec4f36b7ca5bbddf2d6
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Mon Feb 28 16:43:14 2022 +0100
environment: store local hobo info in Hobo model (#61944)
commit 003b15204f757aa752d92ac584df115ce5ddbbf0
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Tue Mar 1 13:03:01 2022 +0100
tests: add some assertions about hobo json (#61944)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: restore migrations and model change (#61944)