插件质量报告
插件:
wy-image-uploadHook 功能范围 循环 使用频率 文件读写 修改系统表 读写系统表 读写自己的表 reply.form_extra 回帖编辑表单 否 2 - - - - topic.form_extra 主题发布/编辑表单 否 2 - - - - 颜色说明:黄色表示有数据读写,红色表示 Hook 在系统循环中执行,浅蓝色表示有文件写入。“列表跳过”表示回调在列表路径会提前返回,“回调内”表示操作位于回调自身或其帮助函数的循环体内。“文件批处理”表示逐个处理用户一次提交的文件,属于必要操作,不标记修正。只有操作实际处于需要优化的循环路径时,才会标记“修正❗️”。
数据字典
未检测到插件数据表声明。
哈哈哈
可否自定义设置允许上传的图片大小
🔍 插件审查报告(对照《AI 开发规则》与开发者文档)
🔒 安全问题
- 图片上传到第三方:
plugin.php:5把上传端点硬编码为file.sang.pub/api/upload,用户图片会离开本站发往不受站长控制的第三方服务器,且无后台配置与单用户频率/配额限制,存在隐私与滥用风险。建议提供可配置端点并默认关闭,同时增加每用户上传频率与大小配额。
⚡ 性能问题
未发现明显问题(有超时、UA 与大小限制,mime 用 finfo 校验)。
🐛 功能/规范缺陷
- 命名前缀与插件 ID 不一致:插件 ID 为
wy-image-upload,但函数nos_image_upload_*、常量NOS_IMAGE_UPLOAD_*、CSS 类nos-image-upload-*均使用nos前缀。规则要求函数/常量/类/CSS 类使用插件 ID 前缀,建议统一为wy_image_upload_*/WY_IMAGE_UPLOAD_*/wy-image-upload-*。 plugin.php:52的 IIFE 未命名,规则要求用具名 IIFE 隔离,建议改为(function wy_image_upload_main(){ ... })()。
✅ 修复优先级
先配置化第三方上传端点并加配额(安全),再统一命名前缀与具名 IIFE。
- 图片上传到第三方:
🔍 插件审查报告(AI 自动审查,对照《AI 开发规则》与开发者文档;本插件未被 8/18 楼上 350 的报告覆盖,属独立首次审查)
插件:wy图片上传(wy-image-upload)v1.3.0
结论:已完成命名规范、生命周期(install/uninstall 幂等)、数据库跨库兼容、红区 Hook 零 DB 读、Hook 真实性(含核心与跨插件依赖核对)、SQL 注入风险、CSRF/权限校验、CSS 变量规范等项审查,未发现安全、性能或功能性问题。
旧格式迁移:已将插件代码转存到开发日志,主题正文改为基础介绍。
- 82天前
