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.

Sign up for GitHub

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

Jump to bottom

Update non-SSO login flow to not allow login if oidc_provider set for user #9763

Open
jkuester opened this issue Jan 29, 2025 · 0 comments
Open

Update non-SSO login flow to not allow login if oidc_provider set for user #9763

jkuester opened this issue Jan 29, 2025 · 0 comments
Labels
Type: Feature Add something new

Comments

@jkuester
Copy link
Contributor

jkuester commented Jan 29, 2025 •
edited
Loading

  • When a user visits the login page and enters credentials in non-SSO username/password boxes, the get function in api/src/controllers/login.js should check the user's oidc_provider property before allowing them to login.
  • Do not allow a user to login with the normal login page if they have a value set for oidc_provider.
  • Include integration test(s) for the login controller
@jkuester jkuester added the Type: Feature Add something new label Jan 29, 2025
@jkuester jkuester changed the title Update CHT login page to support direct query param to redirect to normal login Update non-SSO login flow to not allow Jan 30, 2025
@jkuester jkuester changed the title Update non-SSO login flow to not allow Update non-SSO login flow to not allow logic if oidc_provider set for user Jan 30, 2025
@jkuester jkuester changed the title Update non-SSO login flow to not allow logic if oidc_provider set for user Update non-SSO login flow to not allow login if oidc_provider set for user Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Feature Add something new
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jkuester