[Plugin] astrbot_plugin_deobfuscation

[qingyuan114514]

插件信息

{
  "name": "astrbot_plugin_deobfuscation",
  "display_name": "小番茄混淆",
  "desc": "小番茄图片混淆/解混淆插件，基于Gilbert 2D空间填充曲线算法",
  "author": "Tairitsu_nya",
  "repo": "https://github.com/qingyuan114514/astrbot_plugin_deobfuscation",
  "tags": [],
  "social_link": ""
}

插件检查清单

• 我的插件经过完整的测试
• 我的插件不包含恶意代码
• 我已阅读并同意遵守该项目的 行为准则。

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_deobfuscation

您好！我已经对你提交的插件代码进行了初步自动化审核，作为初步参考:

main.py

1. SSRF 防护存在 DNS Rebinding 绕过风险（高优先级）
   _validate_url() 先解析域名并校验 IP，但真正发请求时 session.get(url) 仍按原 hostname 再次解析。
   攻击者可利用 DNS 重绑定在“校验后、请求前”切换到内网地址，绕过当前检查。
   建议：将“校验后的目标 IP”与实际连接目标绑定（如自定义 resolver / 连接器固定解析结果），避免二次解析漂移。

2. 本地路径读取缺乏可信目录约束（高优先级）
   _get_image_source() + _load_local_image() 对 img_comp.file 的本地路径直接读取，只要不是 http(s) 就当成本地文件。
   若消息组件可被伪造或污染，存在任意本地文件探测/读取尝试风险（即使最终不是图片，也会泄露存在性与错误信息）。
   建议：限制仅允许读取白名单目录（如框架上传临时目录），并做 realpath + 前缀校验。

3. 错误回传可能泄露服务器文件系统信息（中优先级）
   FileNotFoundError 分支把异常文本返回给用户，异常中包含本地绝对路径。
   建议：对外统一返回通用错误（如“文件不存在或不可访问”），详细路径只写日志。

4. 图片下载未校验 MIME 类型（中优先级）
   _download_image() 仅按体积限制，不校验 Content-Type。非图片内容会进入 PIL 解析，增加异常路径与资源消耗。
   建议：至少做 Content-Type 白名单（image/*）+ PIL 解码失败快速处理。

5. _do_process 职责偏重，可维护性一般（低优先级）
   该函数同时承担来源解析、下载/读取、安全异常分流、处理、回包拼装与清理，分支较多。
   建议：拆分为“获取字节”“处理字节”“发送结果”三个子流程，减少复杂度，便于测试。

---

tomato_scramble.py

1. 缓存并发下存在重复构建开销（中优先级）
   _get_curve() 在第一次加锁检查后释放锁再计算，多个线程并发请求同一尺寸时会重复执行 _build_gilbert2d()，造成 CPU 浪费。
   建议：可用“每 key 的构建锁/占位 Future”避免同 key 重复计算。

2. 缓存返回的是可变 numpy 数组引用（中优先级）
   _curve_cache[key] 直接返回；若后续代码意外修改该数组，会污染全局缓存并影响后续加解密正确性。
   建议：返回只读视图（setflags(write=False)）或返回副本（有性能成本，需权衡）。

---

init.py

（未发现需要报告的问题）

---

代码评审团

人物一：Linus Torvalds（直接、硬核）
“你这 SSRF 校验像是‘做了作业但没交卷’：检查 DNS 是对的，但连接时又重新解析，等于把门锁好后把窗户打开。安全逻辑要闭环，不是‘看起来安全’。”

人物二：Steve Jobs（产品与体验导向）
“给用户返回服务器本地路径？这不是‘透明’，这是把后台机房地图印在说明书上。用户只需要知道失败，不需要知道你的目录结构。”

人物三：钢铁侠 Tony Stark（工程效率派）
“曲线缓存做得不错，但并发重复构建像让三台反应堆同时给一个灯泡供电——炫但浪费。加个按尺寸去重构建机制，性能和优雅都能拿下。”

Disclaimer: 以上评审内容由 AI 自动生成，所涉及人物形象与现实无关，不代表真实人物观点。如果给出的建议无关痛痒请忽略。

---

🔍 审核摘要

统计信息

• 仓库文件总数: 3 个 Python 文件
• 已选择审核文件: 3 / 3
  已发送至 AI 审核的文件清单

main.py
__init__.py
tomato_scramble.py

---

此报告由AI自动生成，旨在提供初步反馈和改进建议，不能完全替代人工审核。最终决策以社区维护者的人工审核为准。目前自动审核（仓库地址）处于试验阶段，如遇问题请向维护者反馈。

[astrbot-plugin-copybara]

插件信息已同步到集合仓库。
PR: AstrBotDevs/AstrBot_Plugins_Collection#291
已自动合并 PR #291

该插件已上架。
This plugin is now published.