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

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

@@ -1,3 +1,17 @@
+## [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)
+
+
+### Bug Fixes
+
+* 提高代码覆盖率以解决SonarCloud检查失败问题 ([#90](https://github.com/donghao1393/mcp-dbutils/issues/90)) ([adbc9f2](https://github.com/donghao1393/mcp-dbutils/commit/adbc9f2e6fb54ed446d96775c65edf67611637ed))
+
 # [1.0.0](https://github.com/donghao1393/mcp-dbutils/compare/v0.23.1...v1.0.0) (2025-05-04)
 
 

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

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

{mcp_dbutils-1.0.0 → 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.0 → 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.0 → mcp_dbutils-1.0.2}/pyproject.toml

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

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

@@ -188,8 +188,8 @@ class ConnectionHandler(ABC):
                 if "row" in result and "affected" in result:
                     # 从结果字符串中提取受影响的行数
                     import re
-                    # 使用更安全的正则表达式，避免回溯问题
-                    match = re.search(r"(\d+) rows?", result)
+                    # 限制数字长度，避免DoS风险
+                    match = re.search(r"(\d{1,10}) rows?", result)
                     if match:
                         affected_rows = int(match.group(1))
             except Exception:
@@ -601,7 +601,7 @@ class ConnectionServer:
         # Default fallback
         return "unknown_table"
 
-    async def _check_write_permission(self, connection: str, table_name: str, operation_type: str) -> bool:
+    async def _check_write_permission(self, connection: str, table_name: str, operation_type: str) -> None:
         """检查写操作权限
 
         Args:
@@ -609,9 +609,6 @@ class ConnectionServer:
             table_name: 表名
             operation_type: 操作类型 (INSERT, UPDATE, DELETE)
 
-        Returns:
-            bool: 是否有权限执行写操作
-
         Raises:
             ConfigurationError: 如果连接不可写或没有表级权限
         """
@@ -626,7 +623,10 @@ class ConnectionServer:
         write_permissions = db_config.get("write_permissions", {})
         if not write_permissions:
             # 没有细粒度权限控制，默认允许所有写操作
-            return True
+            return
+
+        # 将表名转换为小写，用于大小写不敏感的比较
+        table_name_lower = table_name.lower()
 
         # 检查表级权限
         tables = write_permissions.get("tables", {})
@@ -634,19 +634,22 @@ class ConnectionServer:
             # 没有表级权限配置，检查默认策略
             default_policy = write_permissions.get("default_policy", "read_only")
             if default_policy == "allow_all":
-                return True
+                return
             else:
                 # 默认只读
                 raise ConfigurationError(WRITE_OPERATION_NOT_ALLOWED_ERROR.format(
                     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 True
+                return
             else:
                 raise ConfigurationError(WRITE_OPERATION_NOT_ALLOWED_ERROR.format(
                     operation=operation_type, table=table_name
@@ -655,15 +658,13 @@ class ConnectionServer:
             # 表未明确配置，检查默认策略
             default_policy = write_permissions.get("default_policy", "read_only")
             if default_policy == "allow_all":
-                return True
+                return
             else:
                 # 默认只读
                 raise ConfigurationError(WRITE_OPERATION_NOT_ALLOWED_ERROR.format(
                     operation=operation_type, table=table_name
                 ))
 
-        return False
-
     def _create_handler_for_type(
         self, db_type: str, connection: str
     ) -> ConnectionHandler: