fix: return clean auth failures for invalid MCP bearer tokens#620
fix: return clean auth failures for invalid MCP bearer tokens#620ArshyaAI wants to merge 1 commit into
Conversation
|
AStack/OpenClaw remote-MCP readiness follow-up evidence for this PR:
Live downstream canary evidence from AStack Remote MCP service after applying the same InvalidTokenError semantics in its wrapper:
Why this matters for downstream readiness: without this upstream fix, invalid/expired MCP bearer tokens can surface as |
b3e4f25 to
444fb2d
Compare
|
Rebased onto upstream master Targeted verification after rebase:
This remains the upstream counterpart for the AStack Remote MCP canary requirement where invalid/expired bearer tokens must return clean 401/403 rather than surfacing as a 500. |
444fb2d to
c6a3f95
Compare
|
Rebased onto upstream master |
|
Live downstream evidence from AStack/OpenClaw dogfood cut on 2026-05-05:
This PR remains one of the upstream-clean blockers for AStack FULL PASS; current dogfood status is intentionally custom/non-upstream-clean until merged and consumed. |
|
2026-05-06 maintainer merge packet / downstream readiness refresh:
Why this PR matters: it removes the AStack-local Remote MCP auth semantics patch so invalid/expired bearer tokens are clean OAuth failures upstream, not custom runtime behavior. |
|
Thank you for this work @ArshyaAI — closing as already in master at Verified during the v0.41.3.0 fix wave scope challenge (#1403). The fix is in production; closing for queue cleanup. |
Summary
Closes #616.
Verification
Note: this avoids changing GBrain auth semantics in downstream AStack wrappers; the fix aligns the provider with the MCP SDK bearerAuth error contract.
Need help on this PR? Tag
@codesmithwith what you need.