Skip to content

Add OIDC auto-redirect config to Core#1339

Open
johnmaguire wants to merge 27 commits intomoghtech:2.2.0from
johnmaguire:oidc-auto-redirect
Open

Add OIDC auto-redirect config to Core#1339
johnmaguire wants to merge 27 commits intomoghtech:2.2.0from
johnmaguire:oidc-auto-redirect

Conversation

@johnmaguire
Copy link
Copy Markdown

@johnmaguire johnmaguire commented Apr 8, 2026

When KOMODO_OIDC_AUTO_REDIRECT=true, the login options response includes oidc_auto_redirect, enabling the UI (via mogh-lib) to automatically redirect unauthenticated users to the OIDC provider. Users can bypass by appending ?disableAutoLogin to the login URL.

Depends on moghtech/lib UI change adding auto-redirect to LoginPage: moghtech/lib#3

Closes #311

When KOMODO_OIDC_AUTO_REDIRECT=true, the login options response
includes oidc_auto_redirect, enabling the UI (via mogh-lib) to
automatically redirect unauthenticated users to the OIDC provider.
Users can bypass by appending ?disableAutoLogin to the login URL.

Depends on moghtech/lib UI change adding auto-redirect to LoginPage.

Closes moghtech#311
@mbecker20
Copy link
Copy Markdown
Member

mbecker20 commented Apr 15, 2026

The user dropdown has an Add Account button too, this links to login page and shouldn't auto oidc redirect from this, ?disableAutoLogin=true needs to be passed to link in this case as well

@johnmaguire
Copy link
Copy Markdown
Author

@mbecker20 Will do - just curious, moving forward, what's the best branch for me to target?

@mbecker20
Copy link
Copy Markdown
Member

mbecker20 commented Apr 16, 2026

@johnmaguire i think main, I can rebase PR to version branch to include. V2 was big change, but won't be another like this for foreseeable future.

These conflicts just require rebase, sometimes I have to release a previous version patch version off main underneath the next version branch, after merging patch to main the version branch was rebased on main as well to keep history linear, this can mess up the PRs.

@johnmaguire
Copy link
Copy Markdown
Author

@mbecker20 Yes, no worries, happy to rebase. I only ask because I think Github forces me to reopen the PR to switch target branch, so wanted to check before opening a new one. 👍

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.

2 participants