Projet

Général

Profil

Development #9963

Gérer les erreurs de connexion lors de la token 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:
12 février 2016
Echéance:
% réalisé:

0%

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

Description

On peut rencontrer une erreur SSL, par exemple à la vérification des certificats ([SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:581)).
Les erreurs "réseaux" sont un peu superflues puisqu'on a obtenu l'authorization grant.

Le patch propose de capturer ces exceptions, de loguer et d'afficher un message générique à l'utilisateur.


Fichiers

Historique

#1

Mis à jour par Frédéric Péters il y a environ 8 ans

"unable to connect to France Connect server", plutôt ? (l'idée étant de montrer clairement dans le message que ce n'est pas chez nous le problème)

#2

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

factoriser logger = logger.getLogger(__name__) et plutôt que d'essayer de suivre toutes les exceptions de requests je ferai un simple

except requests.exceptions.RequestException, e:
   logger.error(u'unable to retrieve access token %s', e)

#3

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

  • Sujet changé de Gérer les erreurs de connexion lors de l'access request à Gérer les erreurs de connexion lors de la token request

@Fred, il ne devrait bientôt plus y avoir de spécifique à FranceConnect qui ne soit configurable dans les settings. Je souhaiterais aller vers un plugin oidc RP plutôt que FS FranceConnect. Je vais créer un ticket en ce sens qui impliquera notamment d'avoir un setting avec le nom de l'OP pour l'affichage en front.

@Benj, ok.

#4

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

J'ai rien contre prendre ce plugin comme base pour un futur plugin oidc mais il faudra en créer un nouveau, pour ne plus avoir A2_FC_ dans les clés de settings et puis qu'on puisse gérer plusieurs sources (une fédération d'IdP OIDC) et on pourra le mettre directement dans authentic.

#6

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

  • Statut changé de Nouveau à Fermé

commit 89f3efd15f0b60e25e20e1e7c129076e579f3dba

Formats disponibles : Atom PDF