mcp-dbutils 1.0.1__tar.gz → 1.0.2__tar.gz

{mcp_dbutils-1.0.1 → mcp_dbutils-1.0.2}/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [1.0.2](https://github.com/donghao1393/mcp-dbutils/compare/v1.0.1...v1.0.2) (2025-05-05)
+
+
+### Bug Fixes
+
+* 修复表名大小写敏感性问题 ([#95](https://github.com/donghao1393/mcp-dbutils/issues/95)) ([2986101](https://github.com/donghao1393/mcp-dbutils/commit/2986101656d3f6d8d70ee21c051e4d140c0c9eee)), closes [#92](https://github.com/donghao1393/mcp-dbutils/issues/92)
+
 ## [1.0.1](https://github.com/donghao1393/mcp-dbutils/compare/v1.0.0...v1.0.1) (2025-05-04)
 
 

{mcp_dbutils-1.0.1 → mcp_dbutils-1.0.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mcp-dbutils
-Version: 1.0.1
+Version: 1.0.2
 Summary: MCP Database Utilities Service
 Author: Dong Hao
 License-Expression: MIT

{mcp_dbutils-1.0.1 → mcp_dbutils-1.0.2}/docs/write-operations.md

@@ -41,6 +41,28 @@ connections:
   - `default_policy`：默认策略，可选值为 `read_only`（只读）或 `allow_all`（允许所有写操作）。
   - `tables`：表级权限配置，可以为每个表指定允许的操作类型。
 
+### 表名大小写处理
+
+MCP-DBUtils 在处理表名时采用大小写不敏感的比较策略：
+
+- 在配置文件中，表名可以使用任意大小写（如 `users`、`Users` 或 `USERS`）。
+- 在 SQL 语句中，表名同样可以使用任意大小写。
+- 系统会自动将表名转换为小写进行比较，确保大小写不影响权限检查。
+
+例如，以下配置和 SQL 语句都能正确匹配：
+
+```yaml
+# 配置文件中使用小写表名
+tables:
+  users:
+    operations: ["INSERT", "UPDATE"]
+```
+
+```sql
+-- SQL 语句中使用大写表名，仍然能正确匹配权限
+INSERT INTO USERS (id, name) VALUES (1, 'test');
+```
+
 ## 使用写操作工具
 
 MCP-DBUtils 提供了 `dbutils-execute-write` 工具用于执行写操作：

{mcp_dbutils-1.0.1 → mcp_dbutils-1.0.2}/docs/zh/configuration-write-operations.md

@@ -92,6 +92,28 @@ tables:
 
 如果未指定`operations`，则默认允许所有写操作（INSERT、UPDATE、DELETE）。
 
+### 2.4 表名大小写处理
+
+MCP-DBUtils 在处理表名时采用大小写不敏感的比较策略：
+
+- 在配置文件中，表名可以使用任意大小写（如 `users`、`Users` 或 `USERS`）。
+- 在 SQL 语句中，表名同样可以使用任意大小写。
+- 系统会自动将表名转换为小写进行比较，确保大小写不影响权限检查。
+
+例如，以下配置和 SQL 语句都能正确匹配：
+
+```yaml
+# 配置文件中使用小写表名
+tables:
+  users:
+    operations: [INSERT, UPDATE]
+```
+
+```sql
+-- SQL 语句中使用大写表名，仍然能正确匹配权限
+INSERT INTO USERS (id, name) VALUES (1, 'test');
+```
+
 ## 3. 完整配置示例
 
 以下是一个包含多种配置场景的完整示例：
@@ -103,7 +125,7 @@ connections:
     type: sqlite
     database: ":memory:"
     # 未指定writable，默认为false（只读）
-  
+
   # 示例2：可写连接，无细粒度控制
   simple_writable_mysql:
     type: mysql
@@ -114,7 +136,7 @@ connections:
     password: secret
     writable: true
     # 未指定write_permissions，所有表都可写
-  
+
   # 示例3：可写连接，有表级和操作级控制
   controlled_postgres:
     type: postgres
@@ -124,7 +146,7 @@ connections:
     user: postgres
     password: postgres
     writable: true
-    
+
     write_permissions:
       # 表级权限
       tables:
@@ -134,10 +156,10 @@ connections:
           operations: [INSERT]  # 只允许插入
         temp_data:
           operations: [INSERT, UPDATE, DELETE]  # 允许所有写操作
-      
+
       # 默认策略
       default_policy: read_only  # 未指定的表默认只读
-  
+
   # 示例4：可写连接，只有默认策略
   all_writable_postgres:
     type: postgres
@@ -147,7 +169,7 @@ connections:
     user: postgres
     password: postgres
     writable: true
-    
+
     write_permissions:
       default_policy: allow_all  # 所有表默认可写
 ```
@@ -210,6 +232,7 @@ connections:
 | "Connection is not configured for write operations" | 连接未设置`writable: true` | 在配置文件中添加`writable: true` |
 | "No permission to perform [操作] on table [表名]" | 表未被允许执行该操作 | 在`write_permissions.tables`中添加表和操作 |
 | "Operation not confirmed" | 未提供正确的确认参数 | 在工具调用中添加`confirmation: "CONFIRM_WRITE"` |
+| "No permission to perform [操作] on table [大写表名]" | SQL语句中使用了大写表名，但配置中使用小写表名 | 系统已支持大小写不敏感的表名比较，此问题在v1.0.1及以上版本已修复 |
 
 ## 6. 配置迁移
 

{mcp_dbutils-1.0.1 → mcp_dbutils-1.0.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mcp-dbutils"
-version = "1.0.1"
+version = "1.0.2"
 description = "MCP Database Utilities Service"
 readme = "README.md"
 license = "MIT"

{mcp_dbutils-1.0.1 → mcp_dbutils-1.0.2}/src/mcp_dbutils/base.py

@@ -625,6 +625,9 @@ class ConnectionServer:
             # 没有细粒度权限控制，默认允许所有写操作
             return
 
+        # 将表名转换为小写，用于大小写不敏感的比较
+        table_name_lower = table_name.lower()
+
         # 检查表级权限
         tables = write_permissions.get("tables", {})
         if not tables:
@@ -638,9 +641,12 @@ class ConnectionServer:
                     operation=operation_type, table=table_name
                 ))
 
-        # 检查特定表的权限
-        if table_name in tables:
-            table_config = tables[table_name]
+        # 创建表名到配置的映射，支持大小写不敏感的比较
+        tables_lower = {k.lower(): v for k, v in tables.items()}
+
+        # 检查特定表的权限（大小写不敏感）
+        if table_name_lower in tables_lower:
+            table_config = tables_lower[table_name_lower]
             operations = table_config.get("operations", ["INSERT", "UPDATE", "DELETE"])
             if operation_type in operations:
                 return

{mcp_dbutils-1.0.1 → mcp_dbutils-1.0.2}/tests/unit/test_write_permissions.py

@@ -108,3 +108,18 @@ class TestWritePermissions:
         await connection_server._check_write_permission("test_conn_allow_all", "any_table", "INSERT")
         await connection_server._check_write_permission("test_conn_allow_all", "any_table", "UPDATE")
         await connection_server._check_write_permission("test_conn_allow_all", "any_table", "DELETE")
+
+    @pytest.mark.asyncio
+    async def test_check_write_permission_case_insensitive(self, connection_server):
+        """Test _check_write_permission with different table name case"""
+        # Table 'users' in config, but using 'USERS' in query
+        await connection_server._check_write_permission("test_conn_with_permissions", "USERS", "INSERT")
+        await connection_server._check_write_permission("test_conn_with_permissions", "USERS", "UPDATE")
+
+        with pytest.raises(ConfigurationError, match="No permission to perform DELETE operation on table USERS"):
+            await connection_server._check_write_permission("test_conn_with_permissions", "USERS", "DELETE")
+
+        # Table 'products' in config, but using mixed case 'Products' in query
+        await connection_server._check_write_permission("test_conn_with_permissions", "Products", "INSERT")
+        await connection_server._check_write_permission("test_conn_with_permissions", "Products", "UPDATE")
+        await connection_server._check_write_permission("test_conn_with_permissions", "Products", "DELETE")