Skip to content

fix(auth): defer resource binding for dynamic clients#146

Merged
stefan-ernst merged 1 commit into
mainfrom
feat/docmost-public-oauth
Jul 23, 2026
Merged

fix(auth): defer resource binding for dynamic clients#146
stefan-ernst merged 1 commit into
mainfrom
feat/docmost-public-oauth

Conversation

@stefan-ernst

Copy link
Copy Markdown
Contributor

Summary

  • make RFC 7591 registrations resource-neutral instead of binding every public client to /mcp
  • let each authorization request select its resource through RFC 8707
  • use provider-neutral names and slugs for dynamically registered clients

This enables Docmost to use a per-user public OAuth flow without an MCP audience. Omni is unchanged and continues to send /mcp during authorization, token exchange, and refresh.

Tests

  • full internal/handlers overlay suite (passed)
  • focused dynamic-registration regression after rebasing onto main (passed)
  • Omni Windshift connector: 3 passed
  • Omni connector OAuth integration: 5 passed

The matching regression test is on Windshiftapp/core-tests branch feat/docmost-public-oauth.

@github-actions

Copy link
Copy Markdown


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@stefan-ernst stefan-ernst changed the title fix(oauth): defer resource binding for dynamic clients fix(auth): defer resource binding for dynamic clients Jul 23, 2026
@stefan-ernst
stefan-ernst merged commit 909bc8e into main Jul 23, 2026
20 of 22 checks passed
@stefan-ernst
stefan-ernst deleted the feat/docmost-public-oauth branch July 23, 2026 11:06
@stefan-ernst
stefan-ernst restored the feat/docmost-public-oauth branch July 23, 2026 21:18
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.

1 participant