Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI: Allow org admin to hide change email and change password #4660

Open
gagantrivedi opened this issue Sep 26, 2024 · 5 comments · Fixed by #4712 or #4810
Open

UI: Allow org admin to hide change email and change password #4660

gagantrivedi opened this issue Sep 26, 2024 · 5 comments · Fixed by #4712 or #4810
Assignees
Labels
feature New feature or request front-end Issue related to the React Front End Dashboard

Comments

@gagantrivedi
Copy link
Member

Organizations using LDAP don't need the "Change Email" and "Change Password" UI elements.

@gagantrivedi gagantrivedi added feature New feature or request front-end Issue related to the React Front End Dashboard labels Sep 26, 2024
@matthewelwell
Copy link
Contributor

matthewelwell commented Sep 27, 2024

We need to determine if the fact that LDAP auth is used is surfaced anywhere for the FE.

Edit: yes - the UI can use the auth_type attribute returned on the /users/me endpoint.

@matthewelwell
Copy link
Contributor

Also, this would likely be helpful in scenarios where a user authentication is OAuth.

@kyle-ssg kyle-ssg linked a pull request Oct 9, 2024 that will close this issue
4 tasks
@matthewelwell matthewelwell reopened this Nov 1, 2024
@matthewelwell
Copy link
Contributor

This might need some further thought. I'm not entirely certain that preventing OAuth users from changing their email address is correct.

Let's take a scenario here:

  1. A user ([email protected]) authenticates with OAuth - a user account is created, and the ID from the OAuth provider is stored against the user
  2. The user changes their email address (to [email protected])

In this case:

  1. The next time they try to authenticate with the original OAuth method, a new account will be created
  2. If they re-authenticate with the same OAuth method, but with the new email address, the ID from the OAuth provider will be updated

So maybe, what we should do is to re-add the ability to change your email as an OAuth user, but we should clear the ID from the OAuth provider when an email changes?

@khvn26
Copy link
Member

khvn26 commented Nov 1, 2024

we should clear the ID from the OAuth provider when an email changes

As long as we're not doing it silently, I'm in favour of this option.

@matthewelwell
Copy link
Contributor

The change here did not fully resolve this issue since changing one's email requires the user's password. OAuth users will likely not have a password set so we need to find another workflow for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request front-end Issue related to the React Front End Dashboard
Projects
None yet
4 participants