Projet

Général

Profil

Bug #10049

Handle the access token request like the ressource request.

Ajouté par Mikaël Ates il y a environ 8 ans. Mis à jour il y a environ 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
22 février 2016
Echéance:
% réalisé:

0%

Temps estimé:
Hors marché:
Non
Patch proposed:
Non
Planning:

Description

As discussed in #10015.


Fichiers

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

When we cannot decode the JSON content I would log an extract of the received content:

            except ValueError:
                logger.warning('no JSON object can be decoded from '
                    'the data received from {}'.format(app_settings.token_url))

replaced by

            except ValueError:
                logger.warning('no JSON object can be decoded from '
                    'the data received from {}: %r'.format(app_settings.token_url, response.content[:100))
#2

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

In which case does if response: fail ? I think request should always return something or launch an exception.

#3

Mis à jour par Mikaël Ates il y a environ 8 ans

I got that case playing errors with the oidc provider and you're right that one should surely raise.

And content logged if no json can be decoded.

#4

Mis à jour par Mikaël Ates il y a environ 8 ans

  • Fichier 0001-Handle-errors-in-access-token-call-as-for-ressource-.patch supprimé
#5

Mis à jour par Mikaël Ates il y a environ 8 ans

  • Statut changé de Nouveau à Fermé

Formats disponibles : Atom PDF