Projet

Général

Profil

Development #55667

sms ovh, KeyError: 'consumerKey'

Ajouté par Sentry Io il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
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

Révision f9be313c (diff)
Ajouté par Valentin Deniaud il y a plus de 2 ans

ovh: handle error during token request (#55667)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Projet changé de Suivi des traces à Passerelle
#2

Mis à jour par Valentin Deniaud il y a plus de 2 ans

  • Assigné à mis à Valentin Deniaud
#3

Mis à jour par Valentin Deniaud il y a plus de 2 ans

C'était Brice le coupable, je dénonce

#4

Mis à jour par Frédéric Péters il y a plus de 2 ans

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.

#6

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.

#7

Mis à jour par Valentin Deniaud il y a plus de 2 ans

else inutile enlevé et internationalisation ajoutée comme m'a fait remarquer Paul.

#8

Mis à jour par Paul Marillonnet il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée

D’ac.

#9

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)
#10

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

Formats disponibles : Atom PDF