Project

General

Profile

Development #61029

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

Added by Emmanuel Cazenave 4 months ago. Updated 4 months ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
24 Jan 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 4 months ago

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

Revision c1820e1f (diff)
Added by Benjamin Dauvergne 4 months ago

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

History

#1

Updated by Benjamin Dauvergne 4 months 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 4 months ago

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

#3

Updated by Benjamin Dauvergne 4 months 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 4 months 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 4 months ago

Il faut #60349 pour que les tests passent.

#6

Updated by Benjamin Dauvergne 4 months 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 4 months 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 4 months ago

Ça a pété le build.

#9

Updated by Benjamin Dauvergne 4 months 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 4 months ago

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

Updated by Transition automatique about 2 months ago

Automatic expiration

Also available in: Atom PDF