Development #57018
multitenant: définir un SLUG sur l'IdP
0%
Description
Je met 'idp' en dur vu qu'il n'y en a qu'un.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Fichier 0001-mellon-define-SLUG-idp-57018.patch 0001-mellon-define-SLUG-idp-57018.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai il y a plus de 2 ans
Je pense qu'il serait mieux de prendre la valeur de l'attribut service_default_slug
, genre:
return [{'METADATA_URL': self_idp.get_saml_idp_metadata_url(), 'SLUG': self_idp.Extra.service_default_slug}]
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
Serghei Mihai a écrit :
Je pense qu'il serait mieux de prendre la valeur de l'attribut
service_default_slug
, genre:
[...]
Pourrais-tu justifier ? Il n'y a qu'une ligne dans cette liste et il n'y en aura qu'une jusqu'à ce que le code change (et le code coté mellon gère la mise à jour du slug, quand l'entity id est stable, faut pas changer les deux en même temps).
Mis à jour par Serghei Mihai il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Benjamin Dauvergne a écrit :
Pourrais-tu justifier ? Il n'y a qu'une ligne dans cette liste et il n'y en aura qu'une jusqu'à ce que le code change (et le code coté mellon gère la mise à jour du slug, quand l'entity id est stable, faut pas changer les deux en même temps).
Justement c'est pour si un jour on décide de changer le slug de l'IDP dans le code Hobo (ça n'arrivera probablement jamais et l'entity_id ne bougera pas), ça sera une modif à un seul endroit à faire.
Mais on avisera le jour quand cela arrivera. Go.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
Serghei Mihai a écrit :
Justement c'est pour si un jour on décide de changer le slug de l'IDP dans le code Hobo (ça n'arrivera probablement jamais et l'entity_id ne bougera pas), ça sera une modif à un seul endroit à faire.
Mais on avisera le jour quand cela arrivera. Go.
Bah ce sera pareil il n'y aura qu'un endroit où changer, ici.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e599e1c5f8221386237f93e2de62feb1453b3ee4 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Thu Sep 16 17:44:47 2021 +0200 mellon: define SLUG=idp (#57018) It should help when changing domain name of a service.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
mellon: define SLUG=idp (#57018)
It should help when changing domain name of a service.