Projet

Général

Profil

Development #26813

ajouter un slug au modèle OIDCProvider

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

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Le champ "name" était traduit en "identifiant" (#26812) et je l'ai utilisé ainsi (côté GNM, notamment dans le script de synchro des comptes, pour faire du provisionning uniquement sur les agents).


Fichiers

Révisions associées

Révision f70ebc07 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

auth oidc: add slug to provider model (#26813)

Historique

#1

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

Je ne sais pas si je contreviens à quelque chose en autorisant le slug à être vide. (mais ça fonctionne)

#3

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

Avec en plus une modif de template parce qu'il n'est plus nécessaire de faire provider.name|slugify pour faire croire à un slug.

#4

Mis à jour par Paul Marillonnet il y a plus de 5 ans

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

Testé en local, c'est bon pour moi.

#5

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit f70ebc07d1c4afd2fc6d0e64ec9b14b4251da7d5
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Sep 28 11:07:01 2018 +0200

    auth oidc: add slug to provider model (#26813)
#6

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

Euh on a déjà un nom canonique pour les fournisseurs OIDC, c'est provider.issuer (équivalent à saml.entity_id; le slug c'est bien quand on a rien d'intélligible et canonique naturellement mais dans le cas SAML et OIDC on a ça (au moins pour les IdPs dans les deux cas..).

PS: je vois la suite, si c'est pour construire une classe HTML, on aurait plus slugifier une partie d'issuer, à voir...

#7

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

c'est provider.issuer

Mais ça n'est pas stable entre recette et prod, qui est le truc cherché ici.

#8

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

D'ac. En unique/null ça me va de toute façon comme identifiant optionnel.

#9

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF