Projet

Général

Profil

Development #19260

crash après le 'where are you from?' RENATER

Ajouté par Paul Marillonnet il y a plus de 6 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Benjamin Dauvergne
Version cible:
-
Début:
06 octobre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Certains IdP semblent pouvoir être gérés par mellon sans pour autant posséder de champ ENTITY_ID.
On dirait que la génération des logs mérite d'être adaptée en conséquence.

Fichiers

0001-LoginView-do-not-crash-if-an-IdP-doesn-t-have-an-ENT.patch (973 octets) 0001-LoginView-do-not-crash-if-an-IdP-doesn-t-have-an-ENT.patch Paul Marillonnet, 06 octobre 2017 18:22
0001-LoginView-do-not-crash-if-an-idp-doesn-t-provide-ade.patch (878 octets) 0001-LoginView-do-not-crash-if-an-idp-doesn-t-provide-ade.patch Paul Marillonnet, 12 octobre 2017 11:06
0001-LoginView-do-not-crash-if-an-idp-doesn-t-provide-ade.patch (861 octets) 0001-LoginView-do-not-crash-if-an-idp-doesn-t-provide-ade.patch Paul Marillonnet, 12 octobre 2017 11:13
0001-do-not-crash-if-no-idp-is-found-19260.patch (3,18 ko) 0001-do-not-crash-if-no-idp-is-found-19260.patch Benjamin Dauvergne, 07 juin 2019 10:24
0001-do-not-crash-if-no-idp-is-found-19260.patch (2,21 ko) 0001-do-not-crash-if-no-idp-is-found-19260.patch Benjamin Dauvergne, 04 juillet 2019 19:10
0001-do-not-crash-if-no-idp-is-found-19260.patch (2,21 ko) 0001-do-not-crash-if-no-idp-is-found-19260.patch Benjamin Dauvergne, 22 juillet 2019 10:54

Révisions associées

Révision b673b3a7 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 4 ans

do not crash if no idp is found (#19260)

Also improve logging of no idp situation in default backend.

Historique

#1

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

  • Patch proposed changé de Non à Oui

Fausse manip' de ma part, le patch est dans les fichiers de description.

#2

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

Petit ajustement après vérification.
mellon.utils.get_idp ne renvoie pas None mais un dictionnaire vide si l'IdP n'a pas été trouvé. Je pense qu'il faut en tenir compte dans la vue de login.

#4

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

Je ne comprends pas le souci ici en lien avec la description du ticket, ce serait possible de redonner un intitulé intelligible ?

Ensuite si ça devient fréquent ce 'no idp found' une HttpResponseBadRequest c'est un peu violent pour l'utilisateur, il faudrait utiliser sso_failure() (que je viens de modifier dans #14810, en tenir compte) pour afficher un message intelligible à l'utilisateur et fournir un lien continuer vers next_url.

#5

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

  • Patch proposed changé de Oui à Non

Patchs acceptables avec un test pour montrer les problèmes que cela résout.

#6

Mis à jour par Benjamin Dauvergne il y a presque 5 ans

  • Assigné à mis à Benjamin Dauvergne
#7

Mis à jour par Benjamin Dauvergne il y a presque 5 ans

J'ai repris le dernier patch en rajoutant quelques petites choses.

#10

Mis à jour par Serghei Mihai il y a plus de 4 ans

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

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit b673b3a7fe147472596eb81c7dafa01737547380
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Jun 7 10:22:10 2019 +0200

    do not crash if no idp is found (#19260)

    Also improve logging of no idp situation in default backend.
#12

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

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

Formats disponibles : Atom PDF