Skip to content

Issue #9 follow-up#10

Merged
kentcdodds merged 2 commits intomainfrom
cursor/issue-9-follow-up-c413
Feb 26, 2026
Merged

Issue #9 follow-up#10
kentcdodds merged 2 commits intomainfrom
cursor/issue-9-follow-up-c413

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented Feb 26, 2026

Strip resource from OAuth request params before completeAuthorization to avoid local audience mismatch failures on /db-api.

Closes #9


Open in Web Open in Cursor 


Note

Medium Risk
Touches OAuth authorization/token request handling; while small, it changes how audiences may be derived and relies on request cloning semantics that could affect compatibility with some clients.

Overview
Prevents audience-bound tokens from being created from MCP clients’ resource parameter by stripping resource before calling completeAuthorization in app/routes/oauth/authorize.tsx.

Fixes token endpoint compatibility by reading the /oauth/token form body via request.clone() before removing resource, avoiding consuming the original request body in workers/app.ts.

Written by Cursor Bugbot for commit 5557ea9. This will update automatically on new commits. Configure here.

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@cursor
Copy link
Copy Markdown

cursor bot commented Feb 26, 2026

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

@kentcdodds kentcdodds marked this pull request as ready for review February 26, 2026 19:25
@kentcdodds kentcdodds merged commit 0e98ed3 into main Feb 26, 2026
6 checks passed
@kentcdodds kentcdodds deleted the cursor/issue-9-follow-up-c413 branch February 26, 2026 22:26
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: invalid_token / Invalid audience from DB API in exercises 4 & 5

2 participants