Skip to content

fix: oidc failure catch and unused urls redirects #1173

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
Merged
merged 2 commits into from
Oct 24, 2025
Merged

fix: oidc failure catch and unused urls redirects #1173

merged 2 commits into from
Oct 24, 2025

Conversation

@ChibiBlasphem
Copy link
Contributor

@ChibiBlasphem ChibiBlasphem commented Oct 23, 2025 •
edited
Loading

  • Redirect /sign-in-email and /create-password to /sign-in when auth provider is OIDC
  • Redirect to /sign-in when /oidc/callback is erroring (avoiding a raw error message for now)

@ChibiBlasphem ChibiBlasphem requested a review from apognu October 23, 2025 10:00
@ChibiBlasphem ChibiBlasphem force-pushed the fix/add-oidc-failure-catch-unused-url-redirects branch from cf4f948 to b47464c Compare October 24, 2025 08:01
@ChibiBlasphem ChibiBlasphem force-pushed the fix/add-oidc-failure-catch-unused-url-redirects branch from b47464c to 0ea356e Compare October 24, 2025 08:03
@ChibiBlasphem ChibiBlasphem merged commit 69d6645 into main Oct 24, 2025
6 checks passed
@ChibiBlasphem ChibiBlasphem deleted the fix/add-oidc-failure-catch-unused-url-redirects branch October 24, 2025 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@siiick siiick siiick approved these changes

@apognu apognu Awaiting requested review from apognu

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ChibiBlasphem @siiick