-
Notifications
You must be signed in to change notification settings - Fork 109
Adding documentation for Vault MCP Server #948
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Vercel Previews Deployed
|
Broken Link CheckerNo broken links found! 🎉 |
gsantos-hc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
General comment: Can we make the path to the docs /vault/docs/mcp-server/...? No need to have **vault-**mcp-server.
content/vault/v1.20.x/content/docs/vault-mcp-server/reference.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/vault-mcp-server/reference.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/vault-mcp-server/reference.mdx
Outdated
Show resolved
Hide resolved
| Learn how to understand security implications while working with the Vault MCP Server. | ||
| --- | ||
|
|
||
| # Security model for Vault MCP server |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Who can help us develop this a bit further? Steven?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deniz and I are working on this doc
content/vault/v1.20.x/content/docs/vault-mcp-server/overview.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/vault-mcp-server/overview.mdx
Outdated
Show resolved
Hide resolved
Add a Beta release disclaimer to the Vault MCP server docs and update the legal disclaimer title and paragraph breaks.
* docs: vault mcp security model * remove : * better ops * remove infra keyword
content/vault/v1.20.x/content/docs/mcp-server/security-model.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/mcp-server/security-model.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/mcp-server/security-model.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/mcp-server/security-model.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/mcp-server/security-model.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/mcp-server/security-model.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/mcp-server/security-model.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/mcp-server/security-model.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/mcp-server/security-model.mdx
Outdated
Show resolved
Hide resolved
content/vault/v1.20.x/content/docs/mcp-server/security-model.mdx
Outdated
Show resolved
Hide resolved
gsantos-hc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@gautambaghel I created #987 to make minor style fixes. This should fix the build issue as well. |
DO NOT MERGE UNTIL SEPT 25th FOR HASHICONF