Projet

Général

Profil

Development #46729

Mettre le slug du portail dans idp_registration_url

Ajouté par Valentin Deniaud il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
17 septembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

#20699 introduit la gestion globale du paramètre ?service pour qu'authentic sache de quel service provient l'usager.

Actuellement pour l'enregistrement hobo fait

['idp_registration_url'] += '?%s' % urlencode({'next': variables['portal_user_url']})

Ça me semble une bonne idée de rajouter 'service': portal_slug.


Fichiers

Révisions associées

Révision 96f78fc0 (diff)
Ajouté par Valentin Deniaud il y a plus de 3 ans

multitenant: add portal slug in idp_registration_url (#46729)

Historique

#1

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

Mais idp_registration_url peut être utilisé depuis d'autres services, on ne veut pas (?) que systématiquement ça renvoie sur le portail usager.

#2

Mis à jour par Valentin Deniaud il y a plus de 3 ans

Frédéric Péters a écrit :

Mais idp_registration_url peut être utilisé depuis d'autres services

OK, je n'en ai trouvé trace que dans combo via publik-base-theme. Mais je vois l'idée, plutôt gérer ça depuis les briques qui exposent un lien d'enregistrement ?

on ne veut pas (?) que systématiquement ça renvoie sur le portail usager.

Ben, pourquoi il y a un ?next=portail_usager qui est posé systématiquement alors ? Naïvement je voyais ce ticket comme une mise à jour d'un comportement déjà existant, pour correspondre aux dernières évolutions.

#3

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

Ah ok j'avais pas capté que le code du ticket était l'existant; et j'imaginais aussi qu'authentic arrivait à détecter le service d'origine avec juste le ?next= actuel.

#4

Mis à jour par Valentin Deniaud il y a plus de 3 ans

#5

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

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

(prochain cycle)

#6

Mis à jour par Valentin Deniaud il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 96f78fc0c7e5d31528f489bdec3a16e60f098162
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Thu Oct 15 16:34:47 2020 +0200

    multitenant: add portal slug in idp_registration_url (#46729)
#7

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

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

Formats disponibles : Atom PDF