Développement #46729
Mettre le slug du portail dans idp_registration_url
0%
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.
Files
Associated revisions
History
Updated by Frédéric Péters about 4 years ago
Mais idp_registration_url peut être utilisé depuis d'autres services, on ne veut pas (?) que systématiquement ça renvoie sur le portail usager.
Updated by Valentin Deniaud about 4 years ago
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.
Updated by Frédéric Péters about 4 years ago
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.
Updated by Valentin Deniaud about 4 years ago
- File 0001-multitenant-add-portal-slug-in-idp_registration_url-.patch 0001-multitenant-add-portal-slug-in-idp_registration_url-.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Frédéric Péters about 4 years ago
- Status changed from Solution proposée to Solution validée
(prochain cycle)
Updated by Valentin Deniaud about 4 years ago
- Status changed from Solution validée to 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)Updated by Frédéric Péters about 4 years ago
- Status changed from Résolu (à déployer) to Solution déployée
multitenant: add portal slug in idp_registration_url (#46729)