Update example of deniedMcpServers - #45479
Conversation
How to review these changes 👓Thank you for your contribution. To review these changes, choose one of the following options: A Hubber will need to deploy your changes internally to review. Table of review linksNote: Please update the URL for your staging server or codespace. The table shows the files in the Key: fpt: Free, Pro, Team; ghec: GitHub Enterprise Cloud; ghes: GitHub Enterprise Server 🤖 This comment is automatically generated. |
There was a problem hiding this comment.
Pull request overview
Updates the denied MCP server example to demonstrate blocking root filesystem access.
Changes:
- Replaces the Microsoft Learn URL deny rule with an exact filesystem server command rule.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues ⚡ |
Why:
Closes:
What's being changed (if available, include any code snippets, screenshots, or gifs):
A better security-focused example is denying an MCP filesystem server configured to expose the root filesystem:
This demonstrates a common enterprise concern: preventing broad filesystem access. I would avoid calling any specific server an “industry standard” deny target because deny policies depend on organizational risk. Also note that serverCommand matching is exact, so this rule blocks only this precise command and argument sequence.
The original value gives customers, including enterprise customers, that the official documentation for Microsoft is a risk and should be denied, which isn't true.
Check off the following: