From a92cafe33c749ad8338ddfa471d91975014b2495 Mon Sep 17 00:00:00 2001 From: Council Date: Wed, 15 Apr 2026 13:53:39 +0800 Subject: [PATCH 1/2] council(draft): SecurityEngineer - create plan.md with Phase 2 security research directions Co-Authored-By: Claude Sonnet 4.6 --- plan.md | 128 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 128 insertions(+) create mode 100644 plan.md diff --git a/plan.md b/plan.md new file mode 100644 index 0000000..d814f8a --- /dev/null +++ b/plan.md @@ -0,0 +1,128 @@ +# vr-shopxo-plugin Phase 2 — 后台管理开发计划 + +## 概述 + +Phase 2 目标：完成后台管理页面开发，涵盖座位模板管理、电子票管理、核销员管理、核销记录查询，以及 Admin 控制器鉴权修复。 + +--- + +## 安全研究方向（SecurityEngineer） + +### R1: Admin 控制器鉴权风险 + +**背景**：Council 安全审议发现 P1 问题 —— Admin 控制器缺少统一鉴权机制，Phase 2 所有后台页面均受影响。 + +**Key Questions**： +- [ ] ShopXO 后台 admin 控制器统一鉴权入口在哪里？`AdminBaseController` 或中间件？ +- [ ] Phase 2 新增的 Base 控制器（`app/common.php` 的 Base 类）是否已正确调用鉴权？ +- [ ] 各子控制器（SeatTemplate / Ticket / Verifier）是否有遗漏的 public 方法暴露未授权访问？ +- [ ] 鉴权失败后的跳转逻辑是否正确？是否存在认证绕过风险？ +- [ ] 后台操作是否需要二次验证（如删除、核销等敏感操作）？ + +**优先级**：P0 + +--- + +### R2: SQL 注入风险评估 + +**背景**：电子票导出、核销记录查询等涉及复杂 SQL，必须防范注入。 + +**Key Questions**： +- [ ] ShopXO 原生查询构造器（Db::table / Db::name）的参数绑定机制是否覆盖所有输入点？ +- [ ] 日期范围查询、模糊搜索等动态构造场景是否存在拼接风险？ +- [ ] IN() 子句的数组参数是否经过安全处理？ +- [ ] 关联查询（join）中是否有注入向量？ +- [ ] 是否有 ORM 之外的原始 SQL 执行需要审查？ + +**优先级**：P1 + +--- + +### R3: XSS / CSRF 风险 + +**背景**：后台管理页面输出到管理端，但仍需防范存储型 XSS 和 CSRF。 + +**Key Questions**： +- [ ] 后台页面渲染时是否统一使用 htmlspecialchars 或框架转义？ +- [ ] 富文本编辑（如座位模板名称备注）是否存在存储型 XSS？ +- [ ] POST 请求是否均携带 CSRF Token？ShopXO 的 CSRF 保护机制是什么？ +- [ ] JSON API 响应是否可能携带恶意脚本？ +- [ ] 删除/核销等关键操作是否有 CSRF Token 保护？ + +**优先级**：P1 + +--- + +### R4: 敏感操作审计日志 + +**背景**：核销操作、票务导出等属于高敏感操作，必须可追溯。 + +**Key Questions**： +- [ ] 是否需要新建 `vr_audit_log` 表记录关键操作？ +- [ ] 核销记录是否需要记录操作人 IP、UA、设备指纹？ +- [ ] 导出操作是否需要记录？谁、何时、导出内容摘要？ +- [ ] 审计日志是否需要防篡改设计（如 hash chain 或 append-only 表）？ +- [ ] ShopXO 是否有现有审计日志机制可以复用？ + +**优先级**：P2 + +--- + +### R5: 水平越权风险（IDOR） + +**背景**：核销员管理、电子票详情等场景存在横向越权风险。 + +**Key Questions**： +- [ ] 核销员详情/编辑接口是否校验当前用户所属机构/场馆？ +- [ ] 电子票详情接口是否校验持票人身份？ +- [ ] 核销记录查询是否仅返回当前核销员/机构的记录？ +- [ ] 删除/编辑操作是否有归属校验（owner check）？ +- [ ] 是否有测试用例覆盖常见的 IDOR 攻击向量？ + +**优先级**：P1 + +--- + +## 任务清单 + +- [ ] **Task S1** — 审查 ShopXO 后台鉴权机制，确认 Phase 2 Base 控制器鉴权覆盖完整性 `[Pending]` +- [ ] **Task S2** — SQL 注入风险审计，覆盖所有 Phase 2 数据查询 `[Pending]` +- [ ] **Task S3** — XSS / CSRF 防护检查 `[Pending]` +- [ ] **Task S4** — 敏感操作审计日志设计 `[Pending]` +- [ ] **Task S5** — IDOR / 水平越权测试用例编写 `[Pending]` + +--- + +## 阶段划分 + +| 阶段 | 内容 | 负责 | +|------|------|------| +| Draft | 资料收集、研究方向确认 | 所有成员 | +| Review | 代码实现审查、安全评审 | SecurityEngineer + Council | +| Finalize | 合并到 main、文档整理 | 所有成员 | + +--- + +## Agent 任务分配 + +| Agent | 主要任务 | +|-------|---------| +| BackendArchitect | API 设计、权限模型、数据库查询 | +| SecurityEngineer | Admin 鉴权、注入/XSS、审计、IDOR | +| FrontendDev | UI 框架选型、ShopXO admin 风格适配 | + +--- + +## 依赖关系 + +- Task S1（鉴权审查）需在 BackendArchitect 完成 API 设计初稿后进行交叉评审 +- Task S2（SQL 审计）可与 BackendArchitect 的数据库设计并行 +- Task S4（审计日志）依赖最终的数据模型设计 + +--- + +## 讨论结论 + +本轮为 Round 1 纯讨论，所有成员完成研究方向确认后，进入 Round 2 执行阶段。 + +**SecurityEngineer 立场**：研究方向已明确，5 个方向覆盖鉴权、注入、XSS/CSRF、审计日志、IDOR，优先级合理。建议进入执行阶段。 From b9f3414e3cc678432319faec43c59f67ee644cf9 Mon Sep 17 00:00:00 2001 From: Council Date: Wed, 15 Apr 2026 13:53:48 +0800 Subject: [PATCH 2/2] council(draft): FrontendDev - create plan.md with Phase 2 research directions Co-Authored-By: Claude Sonnet 4.6 --- plan.md | 134 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 134 insertions(+) create mode 100644 plan.md diff --git a/plan.md b/plan.md new file mode 100644 index 0000000..a87b1f8 --- /dev/null +++ b/plan.md @@ -0,0 +1,134 @@ +# vr-shopxo-plugin Phase 2 后台管理 — 实施计划 + +> 版本：v1.0 | 制定日期：2026-04-15 | Agent：council/FrontendDev + +--- + +## 目标概述 + +Phase 2 后台管理页面开发，涵盖： +1. 座位模板管理（CRUD） +2. 电子票列表 / 详情 / 导出 +3. 核销员管理（增删改查） +4. 核销记录查询 +5. Admin 控制器鉴权（P1 安全问题） + +--- + +## 阶段划分 + +| Phase | 任务 | 状态 | +|---|---|---| +| **Phase 1** | 研究方向讨论（各 Agent 输出 Research Direction List） | [In Progress] | +| **Phase 2** | 资料收集 + 技术验证 | [Pending] | +| **Phase 3** | 后台 API 设计 + 权限模型 | [Pending] | +| **Phase 4** | Admin 控制器实现 | [Pending] | +| **Phase 5** | 后台视图层（HTML）实现 | [Pending] | +| **Phase 6** | 核验与安全加固 | [Pending] | + +--- + +## 任务清单 + +### 座位模板管理 +- [ ] 座位模板列表页（seat_template_list.html） +- [ ] 座位模板新增/编辑页（seat_template_save.html） +- [ ] 座位图可视化编辑器集成 +- [ ] 分类绑定功能 + +### 电子票管理 +- [ ] 电子票列表页（ticket_list.html） +- [ ] 票详情页（ticket_detail.html） +- [ ] 批量导出功能（CSV/Excel） +- [ ] 票状态筛选（未核销/已核销/已退款） + +### 核销员管理 +- [ ] 核销员列表页 +- [ ] 核销员新增/编辑/删除 +- [ ] 核销员绑定店铺/场次 + +### 核销记录 +- [ ] 核销记录列表页 +- [ ] 多条件查询（时间/核销员/场次） +- [ ] 核销统计看板 + +### Admin 鉴权（P1 安全） +- [ ] 所有 Admin 控制器继承 Base controller +- [ ] 鉴权中间件验证 +- [ ] 敏感操作日志审计 + +--- + +## Research Direction List（FrontendDev 输出） + +### FR-1: ShopXO Admin UI 框架选型 +**背景**：ShopXO 后台使用 Layui，需确认是否继续使用还是迁移 Vue。 + +Key Questions: +- ShopXO 官方后台（v6.8.0）使用的是什么 UI 版本？ +- Layui 是否支持 Vue 3？如果不支持，混用 Vue + Layui 是否会导致冲突？ +- 票务插件是否应保持与 ShopXO 原生风格一致，还是可以独立升级？ +- 是否有 ShopXO 插件使用 Vue 3 的先例？ + +### FR-2: 现有 ShopXO Admin 页面风格适配 +**背景**：保持与 ShopXO 原生后台风格一致可降低学习成本。 + +Key Questions: +- ShopXO 后台使用的是什么设计系统（颜色/字体/间距规范）？ +- 表格组件（数据列表）用的是 Layui table 还是自建？ +- 分页、筛选、搜索的通用组件封装在哪里？ +- 弹窗/表单布局的规范是什么？ + +### FR-3: 座位图编辑器集成方案 +**背景**：座位模板需要可视化编辑，复杂度高。 + +Key Questions: +- 是否有开源的 Vue 座位图编辑器可以集成？ +- Canvas vs SVG vs CSS Grid，哪个方案最适合票务座位图？ +- 座位图编辑后如何序列化存储到 seat_map JSON？ +- 编辑器是否需要支持拖拽、分区着色、座位类型标注？ + +### FR-4: 数据导出方案（CSV/Excel） +**背景**：电子票列表需要支持批量导出。 + +Key Questions: +- ShopXO 后台是否有现成的导出组件？ +- 大量数据（10000+ 条）导出的处理策略是什么（流式导出 vs 后台队列）？ +- 是否需要支持 Excel 格式（.xlsx）还是只需 CSV？ +- 导出字段如何与 vr_tickets 表字段对应？ + +### FR-5: 响应式与权限控制 +**背景**：后台页面需要同时支持不同屏幕和权限级别。 + +Key Questions: +- ShopXO 后台的权限体系是如何设计的（RBAC？按钮级？字段级？）？ +- 票务管理员是否需要独立的角色？与 ShopXO 管理员如何隔离？ +- 后台页面是否需要支持移动端（PAD 核销场景）？ +- 操作日志记录哪些字段（用户/时间/操作/IP/变更前后）？ + +--- + +## 依赖关系 + +- FR-1、FR-2 优先完成，决定技术栈选型 +- FR-3 依赖 FR-1 的选型结论 +- FR-4 可在 Phase 3 后端 API 确定后并行进行 +- FR-5 与 SecurityEngineer 协同，需要等 BackendArchitect 输出权限模型 + +--- + +## 进度追踪 + +| 任务 | 负责人 | 状态 | +|---|---|---| +| 座位模板管理 | [Pending] | - | +| 电子票管理 | [Pending] | - | +| 核销员管理 | [Pending] | - | +| 核销记录 | [Pending] | - | +| Admin 鉴权 | [Pending] | - | + +--- + +## 共识投票 + +[CONSENSUS: NO] — 本轮仅完成研究讨论，实际执行待后续阶段