agent-Specialization/modules/file_manager/crud_mixin.py
JOJO 1a4e5baa23 fix(security): Docker 多用户模式全面安全加固(安全审计修复)
威胁模型:拥有普通账号的已登录用户攻击服务器。完整审计报告见
_experiments/security_audit_2026-09-02/(不入库),含实测复现记录。

严重/高危修复:
- 彻底移除文件夹打包下载(/api/download/folder 端点删除,gui/api_v1
  目录下载分支改 410):文件型符号链接会被宿主进程解析,实测可读宿主
  任意文件(含 settings.json 全部 LLM 密钥),删除比补校验更彻底
- XFF 伪造防护:get_client_ip 仅信任 ASTRION_TRUSTED_PROXIES
  (默认空=不信 XFF);登录新增账号维度锁定(5 次失败锁 300s);
  注册邀请码加爆破锁定;限流桶/失败表加 2 万键上限+GC 回收
- auth_debug.log 接入大小轮转;/api/client_debug_log 加限流+长度截断
- /host-login 增加 LINUX_SAFETY 检查且仅回环地址可用

容器加固:
- 默认 cpus=1 / memory=1g,新增 pids-limit=512、memory-swap、
  no-new-privileges
- 新增每用户容器配额 MAX_ACTIVE_CONTAINERS_PER_USER(默认 3),
  防止单用户占满全局容器池

api_v1:
- prompts/personalizations 的 name 加白名单校验(^[A-Za-z0-9_-]{1,64}$),
  修复路径穿越写入;对话元数据中的引用名同步加白名单
- workspaces/conversations/messages/upload 四端点加用户维度限流
- 消息体加 MAX_MESSAGE_CHARS 上限(默认 200000,/api/tasks 同步)
- _path_within 加分隔符边界(修复 startswith 前缀碰撞)

其他:
- SVG 预览强制 application/octet-stream(修存储型 XSS 漏网)
- monitor_snapshot 缓存键加 username 维度(修跨用户快照读取)
- ensure/delete_workspace 加 workspace_id 白名单+父目录二次核验
- delete_folder 拒绝删除工作区根(crud_mixin 与容器代理同步修)
- GuiFileManager 死代码加名称校验防复活
- admin_dashboard 静态壳非 admin 访问一律 404
- /api/app/apk/latest 加登录校验+限流(原未认证可拉 133MB)

有意未修(见报告 7.3 遗留清单):容器 egress 过滤(部署层)、
--cap-drop ALL(怕破坏容器内工作流,已先上 no-new-privileges)、
API 用户/子智能体 LLM 按 token 计费(待产品决策)、str(exc) 收口、
WebSocket 限流、network_permission=restricted 容器语义。

验证:全部文件 py_compile 通过;冒烟测试 6/6 通过;关键新逻辑
(名称校验/路径边界/容器代理删除防护/限流回收)已单测级验证;
端点级行为待服务重启后实测复核(清单见报告 7.5)。

Co-authored-by: Astrion powered by Kimi-K3 <astrion-agent@users.noreply.github.com>
2026-09-02 18:31:00 +08:00

346 lines
13 KiB
Python
Raw Blame History

This file contains ambiguous Unicode characters

This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.

# modules/file_manager.py - 文件管理模块(添加行编辑功能)
import os
import shutil
from pathlib import Path
import re
from bisect import bisect_right
from typing import Any, Optional, Dict, List, Set, Tuple, TYPE_CHECKING
from datetime import datetime
try:
from config import (
MAX_FILE_SIZE,
FORBIDDEN_PATHS,
FORBIDDEN_ROOT_PATHS,
OUTPUT_FORMATS,
READ_TOOL_MAX_FILE_SIZE,
PROJECT_MAX_STORAGE_BYTES,
TERMINAL_SANDBOX_MODE,
LINUX_SAFETY,
)
except ImportError: # 兼容全局环境中存在同名包的情况
import sys
from pathlib import Path
project_root = Path(__file__).resolve().parents[1]
if str(project_root) not in sys.path:
sys.path.insert(0, str(project_root))
from config import (
MAX_FILE_SIZE,
FORBIDDEN_PATHS,
FORBIDDEN_ROOT_PATHS,
OUTPUT_FORMATS,
READ_TOOL_MAX_FILE_SIZE,
PROJECT_MAX_STORAGE_BYTES,
TERMINAL_SANDBOX_MODE,
LINUX_SAFETY,
)
from modules.container_file_proxy import ContainerFileProxy
from modules.host_sandbox_policy import get_macos_writable_paths, get_macos_readable_paths
from utils.logger import setup_logger
from modules.i18n import tr
if TYPE_CHECKING:
from modules.user_container_manager import ContainerHandle
# 临时禁用长度检查
DISABLE_LENGTH_CHECK = True
logger = setup_logger(__name__)
class CrudMixin:
"""FileManager crud mixin 能力 mixin。"""
def create_file(self, path: str, content: str = "", file_type: str = "txt") -> Dict:
"""创建文件"""
valid, error, full_path = self._validate_path(path)
if not valid:
return {"success": False, "error": error}
# 添加文件扩展名
if not full_path.suffix:
full_path = full_path.with_suffix(f".{file_type}")
ok, msg = self._ensure_host_access(full_path, "write")
if not ok:
return {"success": False, "error": msg}
relative_path = self._relative_path(full_path)
try:
if self._use_container():
result = self._container_call("create_file", {
"path": relative_path,
"content": ""
})
if result.get("success"):
print(f"{OUTPUT_FORMATS['file']} 创建文件: {relative_path}")
return result
# 创建父目录
full_path.parent.mkdir(parents=True, exist_ok=True)
with open(full_path, 'w', encoding='utf-8') as f:
f.write("")
print(f"{OUTPUT_FORMATS['file']} 创建文件: {relative_path}")
return {
"success": True,
"path": relative_path,
"size": 0
}
except Exception as e:
return {"success": False, "error": str(e)}
def delete_file(self, path: str) -> Dict:
"""删除文件"""
valid, error, full_path = self._validate_path(path)
if not valid:
return {"success": False, "error": error}
if not full_path.exists():
return {"success": False, "error": tr("file_manager.file_not_found")}
if not full_path.is_file():
return {"success": False, "error": tr("file_manager.not_a_file")}
ok, msg = self._ensure_host_access(full_path, "write")
if not ok:
return {"success": False, "error": msg}
try:
relative_path = self._relative_path(full_path)
if self._use_container():
result = self._container_call("delete_file", {
"path": relative_path
})
if result.get("success"):
print(f"{OUTPUT_FORMATS['file']} 删除文件: {relative_path}")
return result
full_path.unlink()
print(f"{OUTPUT_FORMATS['file']} 删除文件: {relative_path}")
# 删除文件备注(如果存在)
# 这需要通过context_manager处理但file_manager没有直接访问权限
# 所以返回相对路径,让调用者处理备注删除
return {
"success": True,
"path": relative_path,
"action": "deleted"
}
except Exception as e:
return {"success": False, "error": str(e)}
def rename_file(self, old_path: str, new_path: str) -> Dict:
"""重命名文件"""
valid_old, error_old, full_old_path = self._validate_path(old_path)
if not valid_old:
return {"success": False, "error": error_old}
valid_new, error_new, full_new_path = self._validate_path(new_path)
if not valid_new:
return {"success": False, "error": error_new}
if not full_old_path.exists():
return {"success": False, "error": tr("file_manager.original_not_found")}
if full_new_path.exists():
return {"success": False, "error": tr("file_manager.target_exists")}
ok_old, msg_old = self._ensure_host_access(full_old_path, "write")
if not ok_old:
return {"success": False, "error": msg_old}
ok_new, msg_new = self._ensure_host_access(full_new_path, "write")
if not ok_new:
return {"success": False, "error": msg_new}
try:
old_relative = self._relative_path(full_old_path)
new_relative = self._relative_path(full_new_path)
if self._use_container():
result = self._container_call("rename_file", {
"old_path": old_relative,
"new_path": new_relative
})
if result.get("success"):
print(f"{OUTPUT_FORMATS['file']} 重命名: {old_relative} -> {new_relative}")
return result
full_old_path.rename(full_new_path)
print(f"{OUTPUT_FORMATS['file']} 重命名: {old_relative} -> {new_relative}")
return {
"success": True,
"old_path": old_relative,
"new_path": new_relative,
"action": "renamed"
}
except Exception as e:
return {"success": False, "error": str(e)}
def create_folder(self, path: str) -> Dict:
"""创建文件夹"""
valid, error, full_path = self._validate_path(path)
if not valid:
return {"success": False, "error": error}
if full_path.exists():
return {"success": False, "error": tr("file_manager.folder_exists")}
ok, msg = self._ensure_host_access(full_path, "write")
if not ok:
return {"success": False, "error": msg}
try:
relative_path = self._relative_path(full_path)
if self._use_container():
result = self._container_call("create_folder", {"path": relative_path})
if result.get("success"):
print(f"{OUTPUT_FORMATS['file']} 创建文件夹: {relative_path}")
return result
full_path.mkdir(parents=True, exist_ok=True)
print(f"{OUTPUT_FORMATS['file']} 创建文件夹: {relative_path}")
return {"success": True, "path": relative_path}
except Exception as e:
return {"success": False, "error": str(e)}
def delete_folder(self, path: str) -> Dict:
"""删除文件夹"""
valid, error, full_path = self._validate_path(path)
if not valid:
return {"success": False, "error": error}
# 安全:禁止删除工作区根目录本身(空路径/点路径会解析为根;
# docker 模式下该目录是宿主 bind mountrmtree 会真实删除宿主文件)
try:
if full_path.resolve() == Path(self.project_path).resolve():
return {"success": False, "error": tr("file_manager.cannot_delete_workspace_root")}
except Exception:
pass
if not full_path.exists():
return {"success": False, "error": tr("file_manager.folder_not_found")}
if not full_path.is_dir():
return {"success": False, "error": tr("file_manager.not_a_folder")}
try:
relative_path = self._relative_path(full_path)
if self._use_container():
result = self._container_call("delete_folder", {"path": relative_path})
if result.get("success"):
print(f"{OUTPUT_FORMATS['file']} 删除文件夹: {relative_path}")
return result
shutil.rmtree(full_path)
print(f"{OUTPUT_FORMATS['file']} 删除文件夹: {relative_path}")
return {"success": True, "path": relative_path}
except Exception as e:
return {"success": False, "error": str(e)}
def write_file(self, path: str, content: str, mode: str = "w") -> Dict:
"""
写入文件
Args:
path: 文件路径
content: 内容
mode: 写入模式 - "w"(覆盖), "a"(追加)
"""
valid, error, full_path = self._validate_path(path)
if not valid:
return {"success": False, "error": error}
ok, msg = self._ensure_host_access(full_path, "write")
if not ok:
return {"success": False, "error": msg}
# === 新增:内容预处理和验证 ===
if content:
# 长度检查
if not DISABLE_LENGTH_CHECK and len(content) > 9999999999: # 100KB限制
return {
"success": False,
"error": tr("file_manager.content_too_long", length=len(content)),
"suggestion": "请分块处理或使用部分修改方式"
}
# 检查潜在的JSON格式问题
if content.count('"') % 2 != 0:
print(f"{OUTPUT_FORMATS['warning']} 检测到奇数个引号,可能存在格式问题")
# 检查大量转义字符
if content.count('\\') > len(content) / 20:
print(f"{OUTPUT_FORMATS['warning']} 检测到大量转义字符,建议检查内容格式")
try:
relative_path = self._relative_path(full_path)
if PROJECT_MAX_STORAGE_BYTES and self._is_docker_mode():
current_size = self._get_project_size()
existing_size = full_path.stat().st_size if full_path.exists() else 0
if mode == "a":
projected_total = current_size + len(content)
else:
projected_total = current_size - existing_size + len(content)
if projected_total > PROJECT_MAX_STORAGE_BYTES:
return {
"success": False,
"error": tr("file_manager.storage_quota_exceeded"),
"limit_bytes": PROJECT_MAX_STORAGE_BYTES,
"project_size_bytes": current_size,
"attempt_size_bytes": len(content)
}
# 记录写入前的原始内容(用于版本控制和前端 diff 显示)
original_file: Optional[str] = None
try:
if full_path.exists():
original_file = full_path.read_text(encoding='utf-8')
except Exception:
original_file = None
if self._use_container():
result = self._container_call("write_file", {
"path": relative_path,
"content": content,
"mode": mode
})
if result.get("success"):
action = "覆盖" if mode == "w" else "追加"
print(f"{OUTPUT_FORMATS['file']} {action}文件: {relative_path}")
result["original_file"] = original_file
return result
# 创建父目录
full_path.parent.mkdir(parents=True, exist_ok=True)
with open(full_path, mode, encoding='utf-8') as f:
f.write(content)
action = "覆盖" if mode == "w" else "追加"
print(f"{OUTPUT_FORMATS['file']} {action}文件: {relative_path}")
# 读取写入后的实际内容(追加模式需要从文件读取)
try:
new_file = full_path.read_text(encoding='utf-8')
except Exception:
new_file = content
return {
"success": True,
"path": relative_path,
"size": len(content),
"mode": mode,
"original_file": original_file,
"new_file": new_file,
}
except Exception as e:
return {"success": False, "error": str(e)}
def append_file(self, path: str, content: str) -> Dict:
"""追加内容到文件"""
return self.write_file(path, content, mode="a")
def clear_file(self, path: str) -> Dict:
"""清空文件内容"""
return self.write_file(path, "", mode="w")