Skip to content

Comments

[bug] Redirect existing users on new login#654

Merged
KCarretto merged 1 commit intomainfrom
fix/653
Feb 23, 2024
Merged

[bug] Redirect existing users on new login#654
KCarretto merged 1 commit intomainfrom
fix/653

Conversation

@KCarretto
Copy link
Collaborator

What type of PR is this?

/kind bug

What this PR does / why we need it:

Our /oauth/authorize endpoint is setup to create new user accounts (based on oauth state) and redirect users to the application. However, it was not redirecting users with existing user accounts signing in for the first time from a new device. This PR addresses that issue.

Which issue(s) this PR fixes:

Fixes #653

@KCarretto KCarretto requested a review from Cictrone February 23, 2024 21:01
@KCarretto KCarretto merged commit 0f80eb2 into main Feb 23, 2024
@KCarretto KCarretto deleted the fix/653 branch February 23, 2024 21:02
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.

[bug] Redirect appears to not work for /oauth/authorize

1 participant