console icon indicating copy to clipboard operation
console copied to clipboard

Masthead username toggle does not uses fullName

Open ShaharIlany opened this issue 1 year ago • 11 comments

Just updated to 4.15.0-0.okd-2024-03-10-010116

The text at the top left corner of the console used to be my full name, now it just displays my username (my email) instead.

Is this intended behavior?

image

ShaharIlany avatar Apr 09 '24 20:04 ShaharIlany

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close. Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

openshift-bot avatar Jul 09 '24 01:07 openshift-bot

Hi, is this is an intentional change. We can't always rely on having access to the user's full name with external OIDC providers, so we've changed the masthead to show the user's email address. Please let us know if this causes an issue.

/close

spadgett avatar Jul 12 '24 14:07 spadgett

@spadgett: You can't close an active issue/PR unless you authored it or you are a collaborator.

In response to this:

Hi, is this is an intentional change. We can't always rely on having access to the user's full name with external OIDC providers, so we've changed the masthead to show the user's email address. Please let us know if this causes an issue.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

openshift-ci[bot] avatar Jul 12 '24 14:07 openshift-ci[bot]

It can be nice to have the ability to choose what field will be shown there. As an alternative, email can be shown as a fallback only if the full name is undefined/null

I work at a company where most of the user's emails are just meanless numbers and letters before the domain part, which says nothing to the user.

@spadgett

ShaharIlany avatar Jul 12 '24 20:07 ShaharIlany

Let me reopen the issue, and we'll consider adding it back. Thanks for the feedback!

@jhadvig @alimobrem FYI

spadgett avatar Jul 15 '24 17:07 spadgett

Stale issues rot after 30d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle rotten. Rotten issues close after an additional 30d of inactivity. Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle rotten /remove-lifecycle stale

openshift-bot avatar Aug 15 '24 00:08 openshift-bot

Rotten issues close after 30d of inactivity.

Reopen the issue by commenting /reopen. Mark the issue as fresh by commenting /remove-lifecycle rotten. Exclude this issue from closing again by commenting /lifecycle frozen.

/close

openshift-bot avatar Sep 14 '24 08:09 openshift-bot

@openshift-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.

Reopen the issue by commenting /reopen. Mark the issue as fresh by commenting /remove-lifecycle rotten. Exclude this issue from closing again by commenting /lifecycle frozen.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

openshift-ci[bot] avatar Sep 14 '24 08:09 openshift-ci[bot]

/remove-lifecycle rotten /reopen

Still an issue

logonoff avatar Sep 23 '24 19:09 logonoff

@logonoff: Reopened this issue.

In response to this:

/remove-lifecycle rotten /reopen

Still an issue

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

openshift-ci[bot] avatar Sep 23 '24 19:09 openshift-ci[bot]

/lifecycle frozen

logonoff avatar Sep 23 '24 19:09 logonoff

I think this should be fixed as @ShaharIlany suggests (or at least partially).

I don't think it's a good idea to just hardcode the value to the name of the mapped value of the claim preferred_username (if the user comming from an OpenID for example). Different companies have different reasons on why they are mapping users various names. Internally you might just have an ID for exmple, which doesn't say anything about the user (similar to what @ShaharIlany describes).

Previously you actually displayed the value that was mapped to the claim name, which according to the documentaion here is suppose to be some sort of "Display Name" (which is what is being mapped to the fullName attribute on the User resource (user.openshift.io/v1)), which somewhat makes sense.

So to sum it up, I would suggest a couple of ways forward,

  1. If the User resource (user.openshift.io/v1) has populated the key fullName (with whatever the user wants (today that value is being populated from thee name claim if using OpenID)), use that value in the menu.
  2. Make an option to allow the user to define which key from the User resource (user.openshift.io/v1) that should be displayed in the menu.
  3. Document the behaviour accordingly.

That's my two cents 💯🤷

patchon avatar May 13 '25 11:05 patchon