Projet

Général

Profil

Bug #23003

Utiliser l'API correcte pour poser eo:next_url dans le noeud Extensions

Ajouté par Benjamin Dauvergne il y a environ 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
05 avril 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Suite à #23001 et #23002.


Fichiers

Révisions associées

Révision d4d0b859 (diff)
Ajouté par Benjamin Dauvergne il y a presque 6 ans

use good API from lasso to set Extensions node content (#23003)

- use extensions.any tuple to set the content of the Extensions node
- add tests for the presence of the eo:next_url node when
ADD_AUTHNREQUEST_NEXT_URL_EXTENSION is used
- add tests for next_url propagation through the RelayState value

Historique

#1

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

Voilà avec plein de tests en prime.

#2

Mis à jour par Christophe Siraut il y a environ 6 ans

demi-ack

(ça ne me semble pas déconnant mais y'a un ou deux chapitres sur les sessions utilisateurs que je n'ai pas encore lu)

#3

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

Tu dois parler du session.flush() -> del session['_auth_user_id'], en fait il y a du passage de valeur via la session entre les différents appels à mellon_login et j'ai besoin d'être déloggé pour tester mais pas de perdre toute la session, avant je ne testais pas le bon maintien de l'URL de retour en session donc ça ne posait pas de problème. Là je vérifie que si l'artifact qu'on a est périmé et qu'on retente un SSO, je maintiens bien la même URL de retour.

#4

Mis à jour par Christophe Siraut il y a environ 6 ans

ack.

#5

Mis à jour par Thomas Noël il y a environ 6 ans

Ça me semble différent de ce qui a été fait dans w.c.s. sur #23002. C'est qui qui gagne ?

#6

Mis à jour par Frédéric Péters il y a presque 6 ans

Vraiment, confirmation du ack de l'époque, parce qu'avec lasso maintenant taggué, ça bugge à gnm. (#24305)

#8

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de Nouveau à Résolu (à déployer)

J'ai rebasé et mis à jour les tests pour marcher avec Python 3.

commit d4d0b859444b39743054e6dca4f87e2ea2524200
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Apr 5 19:34:59 2018 +0200

    use good API from lasso to set Extensions node content (#23003)

    - use extensions.any tuple to set the content of the Extensions node
    - add tests for the presence of the eo:next_url node when
      ADD_AUTHNREQUEST_NEXT_URL_EXTENSION is used
    - add tests for next_url propagation through the RelayState value
#9

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

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF