Project

General

Profile

Development #28210

backoffice: indiquer que le compte usager est issu d'un IDP externe

Added by Serghei Mihai over 5 years ago. Updated about 3 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
21 November 2018
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Par exemple que le compte a été créé via France Connect (ou plus tard un autre IDP)


Files


Related issues

Related to Authentic 2 - Bug #27392: BO : Lorsque le compte est lié à FranceConnect, indiquer cela sur la fiche utilisateurFermé18 October 2018

Actions
Related to Authentic 2 - Development #41912: En backoffice inclure sur la page d'un utilisateur créé ses fédérations "parentes"Rejeté21 April 2020

Actions

Associated revisions

Revision fd74358a (diff)
Added by Valentin Deniaud about 3 years ago

manager: include oidc and saml federation info in user details (#28210)

History

#1

Updated by Frédéric Péters (de retour le 27 mai) almost 4 years ago

  • Related to Bug #27392: BO : Lorsque le compte est lié à FranceConnect, indiquer cela sur la fiche utilisateur added
#2

Updated by Frédéric Péters (de retour le 27 mai) almost 4 years ago

  • Related to Development #41912: En backoffice inclure sur la page d'un utilisateur créé ses fédérations "parentes" added
#3

Updated by Valentin Deniaud about 3 years ago

  • Assignee set to Valentin Deniaud
#4

Updated by Valentin Deniaud about 3 years ago

#5

Updated by Thomas Noël about 3 years ago

  • Planning set to No
Dans « <p title="{{ identifier.issuer }}">{% trans "Link with SAML identity provider created on" %} {{ identifier.created }}</p> » j'ai du mal à voir à quoi ça va ressemble, le title= il va être visible ou c'est de la popup ? Perso ce qui m'interesserait de voir c'est :
  • le nom de l'IdP qui a fourni l'identité
  • le NameID ou autre attribut de fédération utilisé
  • la date de la création de la fédération
  • la date de la dernière connexion via cette fédération

enfin bref, tout ce qui est possible de mettre ici, je prends :)

(et idem pour OIDC mais je sais un peu moins la terminologie.)

#6

Updated by Valentin Deniaud about 3 years ago

Thomas Noël a écrit :

Dans « <p title="{{ identifier.issuer }}">{% trans "Link with SAML identity provider created on" %} {{ identifier.created }}</p> » j'ai du mal à voir à quoi ça va ressemble, le title= il va être visible ou c'est de la popup ?

C'est visible au survol, dans l'idée que c'est une URL moche et qu'on a pas forcément envie d'alourdir la lecture de la page avec (alors que j'affiche le nom du fournisseur OIDC qui je suppose est plus lisible). Mais si tu me dis que ces considérations esthétiques n'ont pas lieu d'être, j'affiche volontiers la totale (alternativement, tout afficher seulement si request.user.is_superuser ?).

#7

Updated by Serghei Mihai about 3 years ago

Valentin Deniaud a écrit :

C'est visible au survol, dans l'idée que c'est une URL moche et qu'on a pas forcément envie d'alourdir la lecture de la page avec (alors que j'affiche le nom du fournisseur OIDC qui je suppose est plus lisible). Mais si tu me dis que ces considérations esthétiques n'ont pas lieu d'être, j'affiche volontiers la totale (alternativement, tout afficher seulement si request.user.is_superuser ?).

J'aurais vu des sections pliables (pliées par defaut) avec un intitulé du genre "Link with SAML/OIDC provider", avec les détails dont parle Thomas quand on déplie la section.

Il faudrait sûrement un peu de réflexion sur l'érgonomie de la page d'un compte.

#8

Updated by Thomas Noël about 3 years ago

(j'avoue avoir assez peu de considérations esthétiques sur cette page ;) )

#9

Updated by Benjamin Dauvergne about 3 years ago

C'est mieux de mettre des screenshots si vous voulez un avis esthétique.

#10

Updated by Valentin Deniaud about 3 years ago

  • Status changed from Solution proposée to En cours
#11

Updated by Valentin Deniaud about 3 years ago

Revoici, avec plus d'infos et des sections dépliables.

#13

Updated by Thomas Noël about 3 years ago

Ma seule remarque serait de remplacer "Creation date:" par "Created on:" parce que 1) c'est plutôt une date+heure et 2) ça sera plus court en français.

#14

Updated by Frédéric Péters (de retour le 27 mai) about 3 years ago

On n'a pas de libellé à afficher à la place de l'URL "issuer" ?

Ça pourrait être juste dans la continuité du reste, pas dans une section dédiée, comme l'est l'info FranceConnect ?

ex: Liaison à "nom de l'idp" créée le 25 mars 2021 14:47. (+ une parenthèse avec le NameID si c'est pertinent).

#15

Updated by Valentin Deniaud about 3 years ago

Frédéric Péters a écrit :

On n'a pas de libellé à afficher à la place de l'URL "issuer" ?

Si, dans le code il y a {% firstof identifier.idp.DISPLAY_NAME identifier.issuer %} qui s'en occupe (mais dans mon screenshot pas des display_name donc l'URL est affichée). Code repris de ce qu'on affiche déjà en frontoffice.

Ça pourrait être juste dans la continuité du reste, pas dans une section dédiée, comme l'est l'info FranceConnect ?

J'avais fait comme ça avant d'appliquer la remarque de Serghei (#28210#note-7), il faudrait trancher.

#16

Updated by Frédéric Péters (de retour le 27 mai) about 3 years ago

il faudrait trancher

OK désolé juste réagi sur la capture; je veux bien trancher dans ma direction; voilà.

Alternativement, ce serait d'également passer les infos FranceConnect sous ce sous-titre; mais pas le mix actuel.

#17

Updated by Serghei Mihai about 3 years ago

Je tranche pour l'option alternative de Frédéric: passer les infos de liens avec des IDPs externes sous le sous-titre. Et dans ce cas le renommer en "Liens externes".
Un jour on y ajoutera les infos d'autres IDPs OIDC.

#18

Updated by Valentin Deniaud about 3 years ago

Je pense que l'exemple de mon screen a mal orienté le débat, il y a vraiment des endroits où un compte FC a aussi une liaision SAML ?

M'enfin, voici ce qui correspond au consensus établi par Serghei.

#19

Updated by Frédéric Péters (de retour le 27 mai) about 3 years ago

Déplié par défaut alors, sur l'idée donc qu'il n'y ait pas beaucoup de lignes, et qu'aujourd'hui l'info FranceConnect était directement visible. (non ?)

#20

Updated by Benjamin Dauvergne about 3 years ago

Frédéric Péters a écrit :

Déplié par défaut alors, sur l'idée donc qu'il n'y ait pas beaucoup de lignes, et qu'aujourd'hui l'info FranceConnect était directement visible. (non ?)

J'aurai dit de faire simple et de tout mettre au même niveau, comme le dit Valentin jamais personne n'a plusieurs liaison, donc juste afficher "Lié à FranceConnect/à IdP de Montargis/en SAML à https://trucmuche (ici le NameID/le sub OIDC sauf pour FC)" ça me parait suffisant, sans fioritures, je ne suis pas sûr que la date de création ce soit utile, ou au pire en <span title="créé le ....">..</span>.

#21

Updated by Frédéric Péters (de retour le 27 mai) about 3 years ago

Valentin a fait ça, Serghei lui a demandé de changer, j'ai réagi sans lire l'historique pour suggérer de faire comme la ligne FC, Valentin a demandé que ça soit tranché, je suis resté sur mon idée mais j'ai noté que ce qui comptait d'abord c'était la cohérence, là-dessus Serghei a suggéré l'option actuelle, et Valentin il fait et refait son patch.

#22

Updated by Benjamin Dauvergne about 3 years ago

Frédéric Péters a écrit :

Valentin a fait ça, Serghei lui a demandé de changer, j'ai réagi sans lire l'historique pour suggérer de faire comme la ligne FC, Valentin a demandé que ça soit tranché, je suis resté sur mon idée mais j'ai noté que ce qui comptait d'abord c'était la cohérence, là-dessus Serghei a suggéré l'option actuelle, et Valentin il fait et refait son patch.

Ben c'est Serghei qui a tort pas moi.

#23

Updated by Serghei Mihai about 3 years ago

Ok, j'ai tort. Avançons.
Je suis ok pour le patch proposé dans #28210#note-11.

#24

Updated by Benjamin Dauvergne about 3 years ago

Serghei Mihai a écrit :

Ok, j'ai tort. Avançons.
Je suis ok pour le patch proposé dans #28210#note-11.

Le patch que tu pointes contient les sections dépliables (enfin c'est écrit dans sa description) avec leur titre un peu lourd, je dis justement que c'est une présentation un peu too much donc plutôt le patch #28210#note-4 avec les ajouts demandés par Thomas en title=".." ou alors un truc foldable mais avec les informations directement dans le noeud legend parce "Link with OIDC provider" ça prend trop d'espace pour trop peu d'information, donc soit ça :

<p title="{% blocktrans date=user.oidc_account.created %}created on {{ date }}{% enblocktrans %}">Link with {{ user.oidc_account.provider.name }} (sub {{ user.oidc_account.sub }})</p>

soit ça
<legend>Link with {{ user.oidc_account.provider.name }} (sub {{ user.oidc_account.sub }})</legend>
....

Avec toutes mes excuses si je suis un peu trop chiant.

#25

Updated by Valentin Deniaud about 3 years ago

Bon en fait seul Serghei s'est prononcé en faveur des sections dépliables, donc par la règle de la majorité on les oublie, et retour aux phrases simples.

Benjamin Dauvergne a écrit :

avec les ajouts demandés par Thomas en title=".."

Thomas me dit non je peux pas copier coller facilement dans un ticket. Ça s'entend, donc non vraiment tout dans la phrase. Là aussi on peut se débrouiller pour dire que ce choix remporte l'avis de la majorité, il y aurait Thomas Fred et moi.

Et hop patch tout à fait démocratique (mais je me désassigne le ticket si ça ne va toujours pas et quelqu'un d'autre fera mieux).

#26

Updated by Benjamin Dauvergne about 3 years ago

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

Zou.

#27

Updated by Valentin Deniaud about 3 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 916d402f63abec58b41d824013a3c5ebeda9c6df
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Thu Mar 25 15:16:24 2021 +0100

    manager: include oidc and saml federation info in user details (#28210)
#28

Updated by Frédéric Péters (de retour le 27 mai) about 3 years ago

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

Also available in: Atom PDF