Skip to content

Fix Next auth preflight proxy#270

Merged
zbeyens merged 2 commits into
mainfrom
codex/next-auth-options-preflight
May 22, 2026
Merged

Fix Next auth preflight proxy#270
zbeyens merged 2 commits into
mainfrom
codex/next-auth-options-preflight

Conversation

@zbeyens
Copy link
Copy Markdown
Member

@zbeyens zbeyens commented May 22, 2026

  • Auto release

🐛 Fixes ➖ N/A
🟢 95-100% confidence

Phase 🧪 Tests 🌐 Browser
Reproduced 🔴 missing handler.OPTIONS regression ➖ N/A
Verified 🟢 bun check ➖ N/A

✅ Outcome

  • kitcn/auth/nextjs forwards OPTIONS preflight to Convex.
  • Generated Next auth route exports GET, POST, OPTIONS.
  • Docs, fixture, skill reference, changeset, and solution note synced.

⚠️ Caveat

  • Deployment-wide generic Convex HTTP 500 before handler execution is platform/internal, not fixed here.

🏗️ Design

  • Chosen layer: Next auth proxy + route template.
  • Why not app-local wrappers: duplicated CORS policy and cookie handling.
  • Why not broader Convex route work: registerRoutes already owns CORS preflight.

🧪 Verified

  • bun check

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
better-convex Ready Ready Preview, Comment May 22, 2026 1:09pm

Request Review

…ns-preflight

# Conflicts:
#	.changeset/convex-auth-provider-query-client.md
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 22, 2026

🦋 Changeset detected

Latest commit: ba20144

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
kitcn Patch
@kitcn/resend Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@zbeyens zbeyens merged commit 2703bf0 into main May 22, 2026
5 checks passed
@zbeyens zbeyens deleted the codex/next-auth-options-preflight branch May 22, 2026 13:23
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.

1 participant