Skip to content

context_server: Mirror authorization server grant_types_supported#53501

Merged
tomhoule merged 1 commit into
mainfrom
tomhoule-vuxpwotzwltu
May 18, 2026
Merged

context_server: Mirror authorization server grant_types_supported#53501
tomhoule merged 1 commit into
mainfrom
tomhoule-vuxpwotzwltu

Conversation

@tomhoule

@tomhoule tomhoule commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

In MCP OAuth, mirror the authorization server's grant_types_supported in the DCR registration body instead of hardcoding just authorization_code. Logfire's auth server requires both authorization_code and refresh_token in grant_types, and we already uses refresh tokens, so the only issue was not advertising the capability during registration. The DCR body now intersects our supported grant types with what the server advertises, or sends all of ours when the server metadata omits grant_types_supported.

Without this change, the Pydantic Logfire MCP auth server refuses our client registration.

Release Notes:

  • MCP: Improve selection of the grant_types we send during OAuth dynamic client registration.

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Apr 9, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Apr 9, 2026
@tomhoule tomhoule force-pushed the tomhoule-vuxpwotzwltu branch from c536183 to 121cac8 Compare April 9, 2026 09:50
@tomhoule tomhoule marked this pull request as ready for review April 9, 2026 10:52
@tomhoule tomhoule force-pushed the tomhoule-vuxpwotzwltu branch 2 times, most recently from 30c9342 to 413b588 Compare April 16, 2026 11:47
@tomhoule tomhoule force-pushed the tomhoule-vuxpwotzwltu branch from 413b588 to 31e0d22 Compare April 21, 2026 12:21
In MCP OAuth, mirror the authorization server's grant_types_supported in the
DCR registration body instead of hardcoding just authorization_code.
Logfire's auth server requires both authorization_code and refresh_token
in grant_types, and we already uses refresh tokens, so the only issue
was not advertising the capability during registration. The DCR body now
intersects our supported grant types with what the server advertises, or
sends all of ours when the server metadata omits grant_types_supported.

Without this change, the Pydantic Logfire MCP auth server refuses our client registration.
@tomhoule tomhoule force-pushed the tomhoule-vuxpwotzwltu branch from 31e0d22 to 8e1463b Compare May 18, 2026 11:49
@tomhoule tomhoule added this pull request to the merge queue May 18, 2026
Merged via the queue into main with commit 79f998a May 18, 2026
32 checks passed
@tomhoule tomhoule deleted the tomhoule-vuxpwotzwltu branch May 18, 2026 12:51
TomPlanche pushed a commit to TomPlanche/zed that referenced this pull request May 20, 2026
…d-industries#53501)

In MCP OAuth, mirror the authorization server's grant_types_supported in
the DCR registration body instead of hardcoding just authorization_code.
Logfire's auth server requires both authorization_code and refresh_token
in grant_types, and we already uses refresh tokens, so the only issue
was not advertising the capability during registration. The DCR body now
intersects our supported grant types with what the server advertises, or
sends all of ours when the server metadata omits grant_types_supported.

Without this change, the Pydantic Logfire MCP auth server refuses our
client registration.

Release Notes:

- MCP: Improve selection of the `grant_types` we send during OAuth
dynamic client registration.
TomPlanche pushed a commit to TomPlanche/zed that referenced this pull request Jun 2, 2026
…d-industries#53501)

In MCP OAuth, mirror the authorization server's grant_types_supported in
the DCR registration body instead of hardcoding just authorization_code.
Logfire's auth server requires both authorization_code and refresh_token
in grant_types, and we already uses refresh tokens, so the only issue
was not advertising the capability during registration. The DCR body now
intersects our supported grant types with what the server advertises, or
sends all of ours when the server metadata omits grant_types_supported.

Without this change, the Pydantic Logfire MCP auth server refuses our
client registration.

Release Notes:

- MCP: Improve selection of the `grant_types` we send during OAuth
dynamic client registration.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants