Skip to content

feat: send client_uri and logo_uri in OAuth client registration - #1877

Merged
Aaronontheweb merged 2 commits into
netclaw-dev:devfrom
Aaronontheweb:feat/oauth-registration-client-metadata
Aug 11, 2026
Merged

feat: send client_uri and logo_uri in OAuth client registration#1877
Aaronontheweb merged 2 commits into
netclaw-dev:devfrom
Aaronontheweb:feat/oauth-registration-client-metadata

Conversation

@Aaronontheweb

Copy link
Copy Markdown
Collaborator

What

RFC 7591 dynamic client registration (McpOAuthClientRegistrar) now includes two optional metadata fields in the registration request:

  • client_urihttps://netclaw.dev
  • logo_urihttps://raw.githubusercontent.com/netclaw-dev/netclaw-brand/dev/logo/netclaw-icon-purple.png (square PNG icon)

Why

Authorization servers display these on the consent screen. Without them, Netclaw shows up as a bare client_name: netclaw with no link or logo — indistinguishable from any other OAuth client. Providers that support RFC 7591 (e.g. TextForge) will now render a recognizable Netclaw identity.

How

  • Added ClientUri / LogoUri constants to McpOAuthClientRegistrar
  • Included both fields in the registration request dictionary
  • Extended the integration test fake DCR server to capture client_name / client_uri / logo_uri and assert them

Validation

  • dotnet build — 0 warnings, 0 errors
  • McpSdkOAuthFlowIntegrationTests — 23/23 passed
  • Both URLs verified reachable (200, image/png)

RFC 7591 dynamic client registration now advertises the Netclaw home page
and logo so authorization servers can display a recognizable consent
screen. client_uri points at netclaw.dev and logo_uri at the square
Netclaw icon in the public brand repo.
@Aaronontheweb Aaronontheweb added mcp Model context protocol server / client issues. providers Provider integrations and capability detection across OpenAI-compatible backends. labels Aug 11, 2026

@Aaronontheweb Aaronontheweb left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

LGTM

@Aaronontheweb Aaronontheweb removed the providers Provider integrations and capability detection across OpenAI-compatible backends. label Aug 11, 2026
@Aaronontheweb

Copy link
Copy Markdown
Collaborator Author

This change only impacts our client registration metadata with MCP servers

@Aaronontheweb
Aaronontheweb merged commit a33dacc into netclaw-dev:dev Aug 11, 2026
17 checks passed
@Aaronontheweb
Aaronontheweb deleted the feat/oauth-registration-client-metadata branch August 11, 2026 20:22
@Aaronontheweb Aaronontheweb mentioned this pull request Aug 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mcp Model context protocol server / client issues.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant