Development #55667
sms ovh, KeyError: 'consumerKey'
Début:
19 juillet 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/50886/
KeyError: 'consumerKey' (4 additional frame(s) were not displayed) ... File "passerelle/urls_utils.py", line 67, in f return func(request, *args, **kwargs) File "django/views/generic/base.py", line 71, in view return self.dispatch(request, *args, **kwargs) File "django/views/generic/base.py", line 97, in dispatch return handler(request, *args, **kwargs) File "django/views/generic/base.py", line 188, in get url = self.get_redirect_url(*args, **kwargs) File "passerelle/apps/ovh/views.py", line 33, in get_redirect_url self.request.session['ovh-token-%s' % request_id] = result['consumerKey'] sms
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Projet changé de Suivi des traces à Passerelle
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-ovh-handle-error-during-token-request-55667.patch 0001-ovh-handle-error-during-token-request-55667.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
C'était Brice le coupable, je dénonce
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier Screenshot 2021-07-19 at 14-31-27 KeyError _consumerKey_.png Screenshot 2021-07-19 at 14-31-27 KeyError _consumerKey_.png ajouté
Yep ici le coupable est même affiché dans sentry.
C'est un peu chaud quand même le resp.json() qui a lieu avant, je me dis qu'il peut foirer aussi.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-ovh-handle-error-during-token-request-55667.patch 0001-ovh-handle-error-during-token-request-55667.patch ajouté
OK je mets la totale
Mis à jour par Thomas Noël il y a plus de 2 ans
Bah allez quoi, le else:
il ne sert à rien puisque tu fais un return
dans le except
juste au dessus.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-ovh-handle-error-during-token-request-55667.patch 0001-ovh-handle-error-during-token-request-55667.patch ajouté
else inutile enlevé et internationalisation ajoutée comme m'a fait remarquer Paul.
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
D’ac.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 64793d1e12d6da30db57c07832b4391d2157851b Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Mon Jul 19 14:26:30 2021 +0200 ovh: handle error during token request (#55667)
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
ovh: handle error during token request (#55667)