Skip to content

docs: remove nonexistent MSALApp from rc1 notes#3146

Merged
jlowin merged 1 commit intomainfrom
docs/fix-rc1-msalapp-reference
Feb 11, 2026
Merged

docs: remove nonexistent MSALApp from rc1 notes#3146
jlowin merged 1 commit intomainfrom
docs/fix-rc1-msalapp-reference

Conversation

@jlowin
Copy link
Copy Markdown
Member

@jlowin jlowin commented Feb 11, 2026

The RC1 notes referenced MSALApp as a dependency alongside EntraOBOToken, but MSALApp was never implemented — only EntraOBOToken exists in src/fastmcp/server/auth/providers/azure.py. Removes the dangling reference.

@jlowin jlowin merged commit 3e79572 into main Feb 11, 2026
10 of 12 checks passed
@jlowin jlowin deleted the docs/fix-rc1-msalapp-reference branch February 11, 2026 15:19
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 11, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

Documentation in the v3 features notes has been updated to revise the description of the Azure/Entra extra. The change modifies how the extra's dependencies are described, shifting from mentioning both EntraOBOToken and MSALApp dependencies to focusing on EntraOBOToken as the mechanism for handling On-Behalf-Of token exchange declaratively. The update clarifies the dependency surface by removing explicit mention of MSALApp from the documentation. No code changes are included in this pull request.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch docs/fix-rc1-msalapp-reference

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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