Skip to content

Pass request headers into auth session resolution#2203

Merged
elie222 merged 1 commit intomainfrom
codex/mobile-request-header-fix
Apr 10, 2026
Merged

Pass request headers into auth session resolution#2203
elie222 merged 1 commit intomainfrom
codex/mobile-request-header-fix

Conversation

@elie222
Copy link
Copy Markdown
Owner

@elie222 elie222 commented Apr 10, 2026

User description

Summary

  • pass route request headers into auth session resolution instead of relying on ambient headers
  • update mobile-sensitive API and linking routes to resolve sessions from the incoming request
  • align middleware auth checks with the same request-header based session handling

Testing

  • cd apps/web && pnpm exec vitest run 'app/api/user/me/route.test.ts' 'utils/webhook/validate-webhook-account.test.ts'

Generated description

Below is a concise technical summary of the changes proposed in this PR:
Update session resolution to pass incoming headers into the auth helper so downstream components derive user context from the specific request instead of ambient state. Ensure linking callbacks, user lifecycle endpoints, and middleware gates call auth(request.headers) so every flow checks authorization based on the request at hand.

Latest Contributors(2)
UserCommitDate
elie222oauth: dedupe account ...April 01, 2026
josh@jshwrnr.comBackend fixes for mobi...March 09, 2026
This pull request is reviewed by Baz. Review like a pro on (Baz).

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 10, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
inbox-zero Ignored Ignored Apr 10, 2026 7:50am

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 6 files

@elie222 elie222 merged commit 6560d78 into main Apr 10, 2026
17 checks passed
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