Skip to content

Add SQL query tool security documentation#211

Merged
guillaq merged 2 commits intomainfrom
pierre/add-sql-security-docs
Sep 17, 2025
Merged

Add SQL query tool security documentation#211
guillaq merged 2 commits intomainfrom
pierre/add-sql-security-docs

Conversation

@pierrevalade
Copy link
Copy Markdown
Contributor

Summary

  • Added comprehensive security documentation for the SQL query tool
  • Documents ClickHouse RBAC implementation and read-only permissions
  • Explains security guarantees and data isolation for customers

Changes

  • Created new docs/content/docs/security.mdx page
  • Added security page to navigation in meta.json

Context

A customer inquired about the security of the MCP SQL query tool. This documentation explains the existing security measures to help users understand how their data is protected.

Closes #91

-- Claude Code

/cc @guillaq for review

- Created new security.mdx page documenting SQL query tool security measures
- Explains ClickHouse RBAC implementation and read-only permissions
- Details security guarantees and data isolation
- Added security page to navigation in meta.json

Closes #91

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 12, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
anotherai Ready Ready Preview Comment Sep 15, 2025 9:17pm
anotherai-docs Ready Ready Preview Comment Sep 15, 2025 9:17pm

Comment thread docs/content/docs/security.mdx Outdated
Comment thread docs/content/docs/security.mdx Outdated
Comment thread docs/content/docs/security.mdx
- Change "Role-based access control (RBAC)" to "Tenant Isolation" with more specific wording about dedicated ClickHouse users
- Add "Resource limitations" security measure for CPU and memory usage limits
- Add resource exhaustion prevention to the "What you CANNOT do" list

Co-Authored-By: Guillaume <guillaq@users.noreply.github.com>

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@pierrevalade
Copy link
Copy Markdown
Contributor Author

@guillaq I've applied all your review suggestions:

✅ Changed "Role-based access control (RBAC)" to "Tenant Isolation" with the more specific wording you suggested about dedicated ClickHouse users
✅ Added "Resource limitations" as a new security measure for CPU and memory usage limits
✅ Added "Execute queries that would require too much resources" to the "What you CANNOT do" list

The changes have been committed and pushed. Please review again when you have a chance!

-- Claude Code

@guillaq guillaq merged commit 8b7f25e into main Sep 17, 2025
14 checks passed
@guillaq guillaq deleted the pierre/add-sql-security-docs branch September 17, 2025 08:49
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.

Document SQL query tool security measures

2 participants