Project

General

Profile

Development #55667

sms ovh, KeyError: 'consumerKey'

Added by sentry io 12 days ago. Updated 4 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
19 Jul 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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 

Files

Associated revisions

Revision f9be313c (diff)
Added by Valentin Deniaud 5 days ago

ovh: handle error during token request (#55667)

History

#1

Updated by Frédéric Péters 12 days ago

  • Project changed from Suivi des traces to Passerelle
#2

Updated by Valentin Deniaud 12 days ago

  • Assignee set to Valentin Deniaud
#3

Updated by Valentin Deniaud 12 days ago

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

#4

Updated by Frédéric Péters 12 days ago

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

Updated by Thomas Noël 12 days ago

Bah allez quoi, le else: il ne sert à rien puisque tu fais un return dans le except juste au dessus.

#7

Updated by Valentin Deniaud 12 days ago

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

#8

Updated by Paul Marillonnet 9 days ago

  • Status changed from Solution proposée to Solution validée

D’ac.

#9

Updated by Valentin Deniaud 5 days ago

  • Status changed from Solution validée to 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

Updated by Frédéric Péters 4 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF