Skip to content

fix(oauth): anunciar registro dinâmico nos metadados OAuth - #104

Merged
hugoalves-dev merged 1 commit into
mainfrom
fix/issue-103-mcp-oauth-metadata
Nov 6, 2026
Merged

hugoalves-dev merged 1 commit into
mainfrom
fix/issue-103-mcp-oauth-metadata

Conversation

@hugoalves-dev

Copy link
Copy Markdown
Owner

Alterações

O documento OAuth Authorization Server passa a anunciar o endpoint de registro dinâmico quando habilitado, usando o customizador do Spring. O registro rejeita URIs de redirecionamento inválidas e métodos de autenticação de cliente não suportados. Os testes cobrem os documentos de descoberta com DCR habilitado e desabilitado e o fluxo de um cliente novo até a listagem de ferramentas MCP.

Relacionado à #103. A issue deve permanecer aberta até a validação no OpenClaw.

Verificação

  • 27 testes direcionados passaram sobre a main atualizada: DynamicClientRegistrationIT, DcrDisabledIT, ChatGptConnectorOAuthE2ETest e McpAuthIT.
  • Na execução anterior da suíte completa, 451 testes foram executados e houve quatro erros no setup de McpAuthIT por restrição de chave estrangeira em dados WebAuthn compartilhados. A classe passou em execução direcionada.

Validação após o merge

  • Testar no OpenClaw um cliente novo, incluindo login e descoberta de ferramentas; a hipótese de interoperabilidade ainda não foi confirmada no cliente real.
  • Confirmar que o proxy de produção remove ou substitui cabeçalhos X-Forwarded-For fornecidos por clientes, pois o limite de registros depende desse endereço.
  • Verificar a URI de callback usada pelo OpenClaw: o registro aceita HTTP apenas para o host literal localhost, não para 127.0.0.1 ou [::1].

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

Qodana for JVM

It seems all right 👌

No new problems were found according to the checks applied

View the detailed Qodana report

To be able to view the detailed Qodana report, you can either:

To get *.log files or any other Qodana artifacts, run the action with upload-result option set to true,
so that the action will upload the files as the job artifacts:

      - name: 'Qodana Scan'
        uses: JetBrains/qodana-action@v2025.3.2
        with:
          upload-result: true
Contact Qodana team

Contact us at qodana-support@jetbrains.com

@hugoalves-dev
hugoalves-dev merged commit 919c89c into main Nov 6, 2026
6 checks passed
@hugoalves-dev
hugoalves-dev deleted the fix/issue-103-mcp-oauth-metadata branch November 6, 2026 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(oauth): anunciar DCR no metadata RFC 8414 para compatibilidade MCP com OpenClaw

1 participant