Skip to content

Conversation

@lovasoa
Copy link
Collaborator

@lovasoa lovasoa commented Nov 28, 2025

Change OIDC auth redirects to use HTTP 303 (SeeOther) instead of 307 (TemporaryRedirect).

This ensures that POST requests are converted to GET requests during the redirect process, which is the standard and expected behavior for OIDC authentication flows. HTTP 307 preserves the original POST method and body, which is not suitable for these redirects.


Open in Cursor Open in Web

Co-authored-by: contact <contact@ophir.dev>
@cursor
Copy link

cursor bot commented Nov 28, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@lovasoa
Copy link
Collaborator Author

lovasoa commented Nov 28, 2025

@cursoragent briefly describe the change and the bug it fixes in the changelog for v0.40

@cursor
Copy link

cursor bot commented Nov 28, 2025

Added a note under 0.40.0 (unreleased) in CHANGELOG.md explaining that OIDC login redirects now return HTTP 303 so POST submissions flip to GET before hitting the identity provider, resolving the previous 307 behavior that kept the POST method/body and broke standard OIDC flows.


Open in Cursor Open in Web

Learn more about Cursor Agents

@lovasoa lovasoa marked this pull request as ready for review November 28, 2025 15:04
Co-authored-by: contact <contact@ophir.dev>
@lovasoa lovasoa merged commit b808ccf into main Nov 28, 2025
11 checks passed
@lovasoa lovasoa deleted the cursor/use-302-or-303-for-oidc-redirects-gpt-5.1-codex-edd3 branch November 28, 2025 16:22
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.

3 participants