Skip to content

Add support for reauthentication against OIDC to activate sudo_mode #7

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jirutka
Copy link
Contributor

@jirutka jirutka commented Mar 5, 2023

Currently, if the sudo_mode is enabled and the user tries to perform any sensitive action (e.g. change application settings), they have to enter their local password. However, if the user is logged in via OIDC, they typically don't know their local password (it has been autogenerated on the first login via OIDC).

This commit implements an opt-in feature that replaces this local password verification with reauthentication on the OIDC provider, i.e. even if the user is still logged in, they are forced to re-enter their password (or any other kind of authentication) on the OIDC provider.

I don’t speak German, so please update the German localization.

Currently, if the sudo_mode is enabled and the user tries to perform any
sensitive action (e.g. change application settings), they have to enter
their *local* password. However, if the user is logged in via OIDC, they
typically don't know their local password (it has been autogenerated on
the first login via OIDC).

This commit implements an opt-in feature that replaces this local
password verification with reauthentication on the OIDC provider, i.e.
even if the user is still logged in, they are forced to re-enter their
password (or any other kind of authentication) on the OIDC provider.
jirutka added a commit to jirutka/redmine_oidc that referenced this pull request Apr 5, 2023
@joachimmathes joachimmathes self-requested a review September 5, 2023 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant