Project

General

Profile

Development #61029

Agent authentic : enregistrer le 'base_url' d'un service sur l'OU correspondante

Added by Emmanuel Cazenave over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
24 January 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Suite à #60345.


Files

Associated revisions

Revision 3c8927d8 (diff)
Added by Emmanuel Cazenave over 1 year ago

agent/authentic: store portal-user url on correponding OU (#61029)

Revision c1820e1f (diff)
Added by Benjamin Dauvergne over 1 year ago

agent/authentic: fix ou.base_url -> ou.home_url (#61029)

History

#1

Updated by Benjamin Dauvergne over 1 year ago

Tu veux sauver le base_url dans un champ du modèle concerné ? Si besoin je peux ajouter ça au modèle Service quelque part (modèle parent de LibertyProvider).

#2

Updated by Emmanuel Cazenave over 1 year ago

Oui c'est déjà en cours #60345.

#3

Updated by Benjamin Dauvergne over 1 year ago

Je suis plutôt d'accord avec Fred sur le salon :

(19:09:35) bdauvergne: ecazenave: tu peux avoir l'URL automatique via l'OU c'est pas un souci
(19:10:11) bdauvergne: je dirais qu'il faut https://dev.entrouvert.org/issues/60349 (c'est ce qu'on veut coté GLC) et ensuite coté hobo remplir ça avec portal_url
(19:10:19) bdauvergne: coté hobo-agent je veux dire

voilà ça n'empêche pas d'avoir en même temps un service.url pour l'URL du service, mais ça ne marchera pas pour un service externe en OIDC qui va sur /accounts/ (le cas d'usage GLC); en fait ton idée ne marche pas non plus quand on arrive de w.c.s. plutôt que combo non plus, ça renverra vers la racine w.c.s au lieu de combo (ce qui ne change pas grand chose au final, mais ce n'est pas la bonne URL).

#4

Updated by Emmanuel Cazenave over 1 year ago

  • Subject changed from Agent authentic : enregistrer le 'base_url' d'un service to Agent authentic : enregistrer le 'base_url' d'un service sur l'OU correspondante

Ok.

#5

Updated by Emmanuel Cazenave over 1 year ago

Il faut #60349 pour que les tests passent.

#6

Updated by Benjamin Dauvergne over 1 year ago

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

Ok, en attendant #60349 (et que le build soit vert).

#7

Updated by Benjamin Dauvergne over 1 year ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 3c8927d8ee168b9e51af2b836012bbf03fcf7e3c
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Tue Jan 25 17:47:28 2022 +0100

    agent/authentic: store portal-user url on correponding OU (#61029)
#8

Updated by Frédéric Péters over 1 year ago

Ça a pété le build.

#9

Updated by Benjamin Dauvergne over 1 year ago

Je me suis basé sur le patch attaché et pas sur la branche, j'ai poussé la correction.

#10

Updated by Frédéric Péters over 1 year ago

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

Updated by Transition automatique about 1 year ago

Automatic expiration

Also available in: Atom PDF