Projet

Général

Profil

Bug #73990

erreur sur gestion d'erreur oidc, AttributeError: 'str' object has no attribute 'client_id'

Ajouté par Sentry Io il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
30 janvier 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/104987/

InvalidRequest() attend un véritable objet en paramètre client mais reçoit ici une chaine.

InvalidRequest: missing client_secret
  File "authentic2_idp_oidc/views.py", line 828, in token
    response = tokens_from_authz_code(request)
  File "authentic2_idp_oidc/views.py", line 747, in tokens_from_authz_code
    client = authenticate_client(request)
  File "authentic2_idp_oidc/views.py", line 602, in authenticate_client
    raise InvalidRequest('missing client_secret', client=client_id)

AttributeError: 'str' object has no attribute 'client_id'
(2 additional frame(s) were not displayed)
...
  File "django/core/handlers/base.py", line 113, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "authentic2/decorators.py", line 40, in f
    return func(request, *args, **kwargs)
  File "django/views/decorators/csrf.py", line 54, in wrapped_view
    return view_func(*args, **kwargs)
  File "authentic2_idp_oidc/views.py", line 835, in token
    response = e.json_response(request, endpoint='token')
  File "authentic2_idp_oidc/views.py", line 77, in json_response
    content['client_id'] = self.client.client_id

Fichiers

Révisions associées

Révision e59226cb (diff)
Ajouté par Paul Marillonnet il y a environ un an

idp_oidc: fix erroneous exception-handling at client authn time (#73990)

Historique

#1

Mis à jour par Frédéric Péters il y a environ un an

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

Mis à jour par Paul Marillonnet il y a environ un an

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Paul Marillonnet
#3

Mis à jour par Paul Marillonnet il y a environ un an

#4

Mis à jour par Valentin Deniaud il y a environ un an

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

Mis à jour par Paul Marillonnet il y a environ un an

commit e59226cb5b24dbe060ee845f0836a8a7018c3d1c
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Thu Feb 9 14:01:35 2023 +0100

    idp_oidc: fix erroneous exception-handling at client authn time (#73990)
#6

Mis à jour par Paul Marillonnet il y a environ un an

  • Statut changé de Solution validée à Résolu (à déployer)
#7

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#8

Mis à jour par Transition automatique il y a 12 mois

Automatic expiration

Formats disponibles : Atom PDF