Project

General

Profile

Development #19260

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

Added by Paul Marillonnet about 2 years ago. Updated 4 months ago.

Status:
Solution proposée
Priority:
Normal
Target version:
-
Start date:
06 Oct 2017
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-LoginView-do-not-crash-if-an-IdP-doesn-t-have-an-ENT.patch View (973 Bytes) Paul Marillonnet, 06 Oct 2017 06:22 PM

0001-LoginView-do-not-crash-if-an-idp-doesn-t-provide-ade.patch View (878 Bytes) Paul Marillonnet, 12 Oct 2017 11:06 AM

0001-LoginView-do-not-crash-if-an-idp-doesn-t-provide-ade.patch View (861 Bytes) Paul Marillonnet, 12 Oct 2017 11:13 AM

0001-do-not-crash-if-no-idp-is-found-19260.patch View (3.18 KB) Benjamin Dauvergne, 07 Jun 2019 10:24 AM

0001-do-not-crash-if-no-idp-is-found-19260.patch View (2.21 KB) Benjamin Dauvergne, 04 Jul 2019 07:10 PM

0001-do-not-crash-if-no-idp-is-found-19260.patch View (2.21 KB) Benjamin Dauvergne, 22 Jul 2019 10:54 AM

History

#1 Updated by Paul Marillonnet about 2 years ago

  • Patch proposed changed from No to Yes

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

#2 Updated by Paul Marillonnet about 2 years ago

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.

#3 Updated by Paul Marillonnet about 2 years ago

Simplification suggérée par Mik, nouveau patch donc.

#4 Updated by Benjamin Dauvergne about 2 years ago

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 Updated by Benjamin Dauvergne over 1 year ago

  • Patch proposed changed from Yes to No

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

#6 Updated by Benjamin Dauvergne 5 months ago

  • Assignee set to Benjamin Dauvergne

#7 Updated by Benjamin Dauvergne 5 months ago

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

Also available in: Atom PDF