mirror of
https://github.com/moghtech/komodo.git
synced 2026-05-08 21:28:16 -05:00
[GH-ISSUE #567] Invalid OIDC endpoint causes Rust panic #4998
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @atakiya on GitHub (May 31, 2025).
Original GitHub issue: https://github.com/moghtech/komodo/issues/567
Unsure if this would be a bug report or feature request or something else entirely..
So if Komodo at startup cannot set up OIDC correctly when enabled per config, a certain code path that calls
unwrap()causes the entire application to panic.I do not think Komodo should be designed like this - in my instance I have both GitHub and a self-hosted OIDC provider set up. As long as at least one way to authenticate is available, it should simply disable them.
Alternatively, config option to make sure they are all available or vice versa?
Here's the Rust error just for clarity:
For the reason why I encountered this error: I'm moving some subdomains around and currently my OIDC provider only returns 400s. Yes this is on me, but I assumed I could still use Komodo and log in via the GitHub IdP or username+password instead of plucking around in the CLI more.
In the case this is intended design, I'd like to understand why exactly.
@mbecker20 commented on GitHub (Jun 7, 2025):
good point, its not necessary to panic in this case. It should just disable the oidc provider as an option and still allow alternatives