Project

General

Profile

Development #19260

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

Added by Paul Marillonnet over 2 years ago. Updated 3 months ago.

Status:
Solution déployé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

Associated revisions

Revision b673b3a7 (diff)
Added by Benjamin Dauvergne 3 months ago

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

Also improve logging of no idp situation in default backend.

History

#1 Updated by Paul Marillonnet over 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 over 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 over 2 years ago

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

#4 Updated by Benjamin Dauvergne over 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 almost 2 years 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 9 months ago

  • Assignee set to Benjamin Dauvergne

#7 Updated by Benjamin Dauvergne 9 months ago

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

#10 Updated by Serghei Mihai 3 months ago

  • Status changed from Solution proposée to Solution validée

#11 Updated by Benjamin Dauvergne 3 months ago

  • Status changed from Solution validée to 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 Updated by Frédéric Péters 3 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF