sqlite-web 0.7.2
gitea.db
issue
Create
Query
access
access_token
action
action_artifact
action_run
action_run_index
action_run_job
action_runner
action_runner_token
action_schedule
action_schedule_spec
action_task
action_task_output
action_task_step
action_tasks_version
action_variable
app_state
attachment
auth_token
badge
branch
collaboration
comment
commit_status
commit_status_index
commit_status_summary
commit_sync_log
commit_sync_status
dbfs_data
dbfs_meta
deploy_key
email_address
email_hash
external_login_user
follow
gpg_key
gpg_key_import
hook_task
issue
issue_assignees
issue_content_history
issue_dependency
issue_index
issue_label
issue_pin
issue_user
issue_watch
label
language_stat
lfs_lock
lfs_meta_object
login_source
milestone
mirror
notice
notification
oauth2_application
oauth2_authorization_code
oauth2_grant
org_user
package
package_blob
package_blob_upload
package_cleanup_rule
package_file
package_property
package_version
project
project_board
project_issue
protected_branch
protected_tag
public_key
pull_auto_merge
pull_request
push_mirror
reaction
release
renamed_branch
repo_archiver
repo_hidden_file
repo_indexer_status
repo_license
repo_redirect
repo_topic
repo_transfer
repo_unit
repository
review
review_state
secret
session
sqlite_sequence
star
stopwatch
system_setting
task
team
team_invite
team_repo
team_unit
team_user
topic
tracked_time
two_factor
upload
user
user_badge
user_blocking
user_open_id
user_redirect
user_setting
version
watch
webauthn_credential
webhook
Toggle helper tables
Structure
Content
Query
Insert
Drop
Import
Export
Delete row 437 from issue
id
437
repo_id
21
index
152
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：pay-260616 - 切换支付方式时更新页面优惠券列表
content
## 自动代码审查报告 **分支**: pay-260616 **提交**: `f32131c65
## 自动代码审查报告 **分支**: pay-260616 **提交**: `f32131c65de891d3fc755a6e9511b716d7eec12d` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-01 18:07:49 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：代码实现了较为完整的小程序首页、包厢预订、订单支付等核心业务链路，但存在明显的架构耦合、安全漏洞与逻辑隐患。大量硬编码、全局实例属性污染、未严格校验的输入参数以及原始 SQL 拼接降低了系统的可维护性与安全性。整体处于“能跑但脆弱”的状态，亟需安全加固与规范化重构。 - **风险等级**：🔴 高 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `Ahead_yc_order_model.php`<br>`get_bill_goods_info` 方法内 | **SQL 注入漏洞**：使用字符串拼接构造查询条件 `$sql = '_unique_key="' . $unique_key . '" ...'`，若 `$unique_key` 未严格过滤，将导致数据库被恶意查询或篡改。 | 废弃手动拼接 SQL，全面改用 CI Query Builder 或预处理语句。 | `$this->db->where('_unique_key', $unique_key)->where_in('_status', [1,4])->get($this->table_name)->result_array();` | | 🔴 严重 | `Index.php`<br>`getRoomInfo` 方法 | **逻辑分支未终止**：`if (empty($open_room_log))` 块内调用 `$this->success_response()` 后未 `return` 或 `exit`。若响应函数未中断脚本，后续代码将尝试访问空数组键值导致 `Fatal Error`。 | 在条件分支末尾显式添加 `return;`，确保控制流正确终止。 | `if (empty($open_room_log)) { ... $this->success_response(); return; }` | | 🔴 严重 | `Index.php`<br>`__construct` 方法 | **横向越权风险 (IDOR)**：直接信任客户端传入的 `$this->param['spe_merchant_id']` 并赋值给核心业务变量，未校验当前用户是否有权访问该商家数据。 | 增加权限校验中间件或模型层校验，确保 `spe_merchant_id` 与当前登录用户/会话绑定。 | `if (!$this->auth->check_merchant_access($this->spe_merchant_id)) { throwError('无权访问该商家'); }` | | 🟠 警告 | `Index.php`<br>`getConfig` 方法 | **重复加载模型**：构造函数已加载 `ahead_merchant_wx_min_set_model`，方法内再次 `$this->load->model()` 造成冗余调用（虽 CI 会缓存，但违背规范）。 | 移除方法内的重复加载，直接使用构造函数已加载的实例。 | 删除 `$this->load->model('Ahead_merchant_wx_min_set_model');` | | 🟠 警告 | `Ahead_billiards_model.php`<br>`check_room` 方法 | **全局实例属性污染**：直接修改 `$CI->merchant_id`、`$CI->operational_scene` 等全局属性，破坏封装性，易引发并发请求下的状态串扰。 | 将状态数据封装至 Model 自身属性或通过方法参数/返回值传递，避免污染 CI 超全局实例。 | `private $merchant_id; private $operational_scene; // 使用 $this->merchant_id 替代 $CI->xxx` | | 🟠 警告 | `Ahead_shop_book_time_info_model.php`<br>多处静态属性 | **静态缓存生命周期风险**：大量使用 `self::$book_days_info`、`self::$shop_data` 等静态变量缓存数据。在 PHP-FPM 长连接或 CLI 模式下可能导致跨请求脏数据，且难以清理。 | 改用 CI 内置缓存组件 `$this->cache->save()` 或实例属性，并在请求结束时重置。 | `if (empty($this->cache->get('book_days_'.$merchant_id))) { ... $this->cache->save(...); }` | | 🟡 建议 | 全局多处 | **魔法数字/硬编码泛滥**：大量使用 `'-1'`、`'1'`、`'2'`、`33424` 等字面量，业务含义不透明，后期维护成本极高。 | 提取为类常量或独立配置类，统一命名管理。 | `class OrderStatus { const PENDING = -1; const PAID = 1; }` | | 🟡 建议 | `Index.php` / `Ahead_billiards_model.php` | **命名规范不一致**：方法名混用驼峰 (`getRoomInfo`)、下划线 (`get_room_info`) 及自定义前缀 (`public_getIndex`)，不符合 PSR-12。 | 统一采用驼峰命名法，路由前缀建议通过路由配置文件处理，而非硬编码在方法名中。 | `public function getIndex()`, `public function getRoomInfo()` | | 🟡 建议 | 全局 | **框架适配说明**：代码特征（`get_instance()`, `$this->load->model()`, `BASEPATH`）高度符合 **CodeIgniter 3** 架构。若 `phpci` 为内部定制框架或笔误，请确认其生命周期与 CI3 是否一致。 | 建议查阅 `phpci` 官方文档确认模型加载、路由解析及请求生命周期差异，必要时适配对应框架的依赖注入容器。 | 无 | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **立即修复 SQL 注入**：替换 `Ahead_yc_order_model.php` 中所有手动拼接的 `$sql` 字符串，全面启用 CI Query Builder (`$this->db->where()`, `$this->db->select()`) 或 PDO 预处理。 2. **阻断逻辑穿透**：在 `Index.php::getRoomInfo` 的空结果判断块末尾补充 `return;`，防止空指针异常。 3. **实施商家权限校验**：在控制器层或中间件层增加 `spe_merchant_id` 的归属权校验，严禁直接信任前端传入的商户标识。 ### 🛠 后续重构与优化方向 1. **解耦全局状态**：将 `Ahead_billiards_model` 中对 `$CI` 实例属性的直接读写改为依赖注入或方法参数传递。Model 应保持无状态或仅维护自身实例属性，提升单元测试可行性。 2. **统一配置与常量管理**：建立 `config/constants.php` 或独立 `Enum` 类，集中管理订单状态、支付场景、运营场景等魔法值。例如：`const SCENE_KTV = '1'; const SCENE_BILLIARDS = '2';`。 3. **规范模型加载与缓存策略**： - 将频繁使用的 Model 移至控制器 `__construct` 中加载。 - 替换 `self::$` 静态缓存为 CI 原生缓存驱动（Redis/Memcached/File），并设置合理的 TTL，避免内存泄漏或脏读。 4. **强化输入验证层**：在控制器入口统一使用 CI 的 `form_validation` 或自定义 `RequestValidator` 对 `$this->param` 进行类型转换、范围校验与 XSS 过滤，避免业务代码中散落 `intval()`、`empty()` 判断。 5. **框架适配确认**：若项目确为 `phpci` 框架，请核对其是否兼容 CI3 的 `load` 机制。若为自研框架，建议逐步迁移至现代 PHP 架构（如 PSR-4 自动加载、依赖注入容器、中间件管道），以提升长期可维护性。 > 💡 **提示**：本次审查基于提供的代码片段。若涉及核心支付、用户鉴权或高并发预订场景，建议补充单元测试覆盖率，并引入静态分析工具（如 PHPStan / Psalm）进行自动化质量门禁。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780308469
updated_unix
1780308469
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel