Projet

Général

Profil

Development #74969

auth_oidc : masquer les options hors-spécs de l’authentificateur OIDC à quiconque n’est pas superuser

Ajouté par Paul Marillonnet il y a environ un an. Mis à jour il y a 12 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
01 mars 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

On avait déjà fait ce travail côté configuration des clients, dans #71905. Il faudrait en faire de même pour cet écran, où est exposée l’option de support de la synchro a2<->a2, rien à voir avec la choucroute OIDC.


Demandes liées

Lié à Gadjo - Development #77184: à la validation d’un formulaire, déplacer le focus sur l'onglet avec des erreursNouveau02 mai 2023

Actions

Révisions associées

Révision 9d78da38 (diff)
Ajouté par Valentin Deniaud il y a 12 mois

manager: render authenticator form correctly when no tabs (#74969)

Révision dc60433a (diff)
Ajouté par Valentin Deniaud il y a 12 mois

authenticators: record edited fields when using tabs (#74969)

Révision b782ec7b (diff)
Ajouté par Valentin Deniaud il y a 12 mois

auth_oidc: split configuration form into general and advanced tabs (#74969)

Historique

#2

Mis à jour par Valentin Deniaud il y a environ un an

Ma préférence irait à ranger ça dans un onglet « Avancé », comme c'est le cas pour SAML.

#3

Mis à jour par Paul Marillonnet il y a environ un an

Valentin Deniaud a écrit :

Ma préférence irait à ranger ça dans un onglet « Avancé », comme c'est le cas pour SAML.

Oui ok pourquoi pas, mais l’objet initial du ticket c’était prévenir la situation des clients qui se demandent ce que vient faire cette option de synchronisation dans la config OIDC. Comme on n’a pas la réponse, je serais pour masquer cette option aux non superusers :)
(ce qui n’empêche pas d’en profiter pour réorganiser le truc en plusieurs onglets, oui.)

#4

Mis à jour par Valentin Deniaud il y a environ un an

Paul Marillonnet a écrit :

l’objet initial du ticket c’était prévenir la situation des clients qui se demandent ce que vient faire cette option

So was l'objectif de « cacher » des options dans le menu avancé côté SAML, tentative couronnée de succès je ne pense pas qu'on ait eu de ticket depuis.

D'où l'idée de faire pareil ici (on pourra également ajouter un help_text, rien que pour nous ça peut être utile).

#5

Mis à jour par Paul Marillonnet il y a environ un an

Valentin Deniaud a écrit :

So was l'objectif de « cacher » des options dans le menu avancé côté SAML, tentative couronnée de succès je ne pense pas qu'on ait eu de ticket depuis.

Ok, je reste persuadé que pas mal de gens lisent cet intitulé “Avancé” comme une invitation à cliquer partout dans l’onglet, mais on peut essayer oui, et on ajoutera le masquage aux non-superusers si ça ne suffit pas :)

#6

Mis à jour par Robot Gitea il y a 12 mois

  • Statut changé de Nouveau à Solution proposée
  • Assigné à mis à Valentin Deniaud

Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Robot Gitea il y a 12 mois

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

Paul Marillonnet (pmarillonnet) a approuvé une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Paul Marillonnet il y a 12 mois

  • Lié à Development #77184: à la validation d’un formulaire, déplacer le focus sur l'onglet avec des erreurs ajouté
#9

Mis à jour par Robot Gitea il y a 12 mois

  • Statut changé de Solution validée à Résolu (à déployer)

Paul Marillonnet (pmarillonnet) a mergé une pull request sur Gitea concernant cette demande :

#10

Mis à jour par Transition automatique il y a 12 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#11

Mis à jour par Transition automatique il y a 10 mois

Automatic expiration

Formats disponibles : Atom PDF