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 373 from issue
id
373
repo_id
22
index
39
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：app-260616 - 111
content
## 自动代码审查报告 **分支**: app-260616 **提交**: `a23ae826b
## 自动代码审查报告 **分支**: app-260616 **提交**: `a23ae826b8c28744cf9938993cd3e4a0ee99a6e2` **提交人**: zhangjunnan (121158035@qq.com) **时间**: 2026-05-27 16:26:24 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：代码实现了较为完整的业务闭环（预订、支付、退款、Redis缓存、OSS操作等），但存在较多历史遗留问题。核心风险集中在**硬编码敏感凭证**、**缺失事务保障导致的数据不一致**、**SQL注入隐患**以及**PHP 8 兼容性**。代码结构偏向“过程式”，Helper 中混杂大量 DB 逻辑，Model 方法过长且职责不清，需进行架构级重构。 - **风险等级**：🔴 高（存在资金对账风险、凭证泄露风险及并发竞态条件） > 📌 **框架说明**：基于 `$CI =& get_instance()`、`system/` 目录结构及 `Simple_model` 继承关系，判定当前代码基于 **CodeIgniter 3.x** 架构。若 `phpci` 为内部定制框架，以下基于 CI3/PSR 标准的建议同样适用。 --- ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `common_helper.php`<br>`alioss_*` 函数 | **硬编码阿里云 OSS 密钥**。`accessId` 与 `accessKey` 直接写死在代码中，极易通过 Git 泄露或反编译获取，导致存储桶被恶意删除/盗刷。 | 移至 `application/config/oss.php` 或 `.env` 环境变量，通过 `$CI->config->item()` 读取。生产环境严禁硬编码。 | `$param = $CI->config->item('aliyun_oss');` | | 🔴 严重 | `Ahead_pay_log_model.php`<br>`update_refund_amount`<br>`get_refundable_pay_log` | **SQL 注入风险**。使用字符串拼接构造 `WHERE` 条件（如 `'_relation_id="' . $relation_id . '"...'`），绕过 CI Query Builder 的自动转义机制。 | 统一使用数组传参，利用框架底层 PDO/MySQLi 预处理防注入。 | `$where = ['_relation_id' => $relation_id, '_status' => [1,4]];` | | 🔴 严重 | `Ahead_book_order_change_pay_log_model.php`<br>`refund()` | **缺失数据库事务，资金状态不一致**。循环中先执行 `$this->update(['_status'=>4])`，若后续 `wx_refund()` 失败直接 `return`，已更新的记录无法回滚。 | 使用 `$this->db->trans_start()` 包裹整个退款流程。外部 API 失败时抛出异常或显式 `trans_rollback()`。 | `try { $this->db->trans_start(); ... $this->db->trans_complete(); } catch(\Exception $e) { $this->db->trans_rollback(); }` | | 🔴 严重 | `common_helper.php`<br>`decodeUnicode()` | **使用已废弃的 `create_function()`**。PHP 7.2 起报 `Deprecated`，PHP 8.0+ 直接致命错误。 | 替换为现代匿名函数（Closure）。 | `preg_replace_callback('/\\\\u([0-9a-f]{4})/i', fn($m) => mb_convert_encoding(pack("H*", $m[1]), "UTF-8", "UCS-2BE"), $str);` | | 🟠 警告 | `common_helper.php`<br>`get_aliyun_redis_conn()` | **频繁创建/关闭 Redis 连接**。每次调用都 `new Redis()` 并 `$obj->close()`，TCP 握手开销极大，高并发下易耗尽文件描述符。 | 使用 CI 内置 Redis 驱动，或实现单例/连接池复用。Helper 中不应直接管理底层连接生命周期。 | `$CI->load->driver('cache', ['adapter' => 'redis']); $redis = $CI->cache->redis;` | | 🟠 警告 | `Ahead_book_order_change_pay_log_model.php`<br>`wx_refund()` | **退款单号 `out_refund_no` 冲突风险**。仅使用 `商户号+YmdHis`，同一秒内多笔退款会触发微信 `OUT_TRADE_NO_USED` 错误。 | 追加业务唯一标识（如流水号 ID 或 `uniqid()`）。 | `$input->SetOut_refund_no($mchid . date('YmdHis') . '_' . $refund_info['log_id']);` | | 🟠 警告 | `common_helper.php`<br>`doRedisLockRelease()` | **分布式锁释放存在竞态条件**。`GET` 校验后 `DEL` 非原子操作，高并发下可能误删其他请求持有的锁。 | 使用 Lua 脚本保证 `GET` 与 `DEL` 的原子性。 | `if redis.call("get",KEYS[1]) == ARGV[1] then return redis.call("del",KEYS[1]) else return 0 end` | | 🟠 警告 | `common_helper.php`<br>`rs_set()` / `rs_zAdd()` | **逻辑缺陷与返回值误导**。`rs_set` 中 `!$obj->exists()` 导致行为变为 `SETNX` 而非覆盖；`rs_zAdd` 在 `$member` 为空时仍返回 `true`。 | 移除多余的 `exists` 判断，严格校验参数并返回 Redis 原生结果。 | `return $obj->set($keyName, $string, $expireTime > 0 ? ['EX' => $expireTime] : null);` | | 🟡 建议 | `Ahead_shop_config_second_model.php`<br>`get_one()` | **参数拼写错误 & 静态缓存无失效机制**。`$fileds` 应为 `$fields`；`self::$shop_config` 在单次请求内有效，但跨请求或配置变更时会产生脏数据。 | 修正拼写；引入 CI Cache 驱动或设置请求级缓存失效策略。 | `public function get_one($where, $fields = "*", $order = "")` | | 🟡 建议 | 全局多个 Model | **魔法数字泛滥**。支付状态、类型、场景等大量使用 `1, 2, 3, 4, 13, 14` 等硬编码，可读性差且维护成本高。 | 提取为类常量或独立配置类（如 `PayStatus::PAID = 1`）。 | `const STATUS_PAID = 1; const TYPE_BOOK = 5;` | | 🟡 建议 | `common_helper.php`<br>`get_mac_by_url()` 等 | **职责越界**。Helper 中直接 `$CI->load->model()` 并执行复杂 DB 查询，违反 MVC 分层原则，难以单元测试。 | 将解析逻辑下沉至 `QrCodeService` 或对应 Model，Helper 仅保留纯函数逻辑。 | 迁移至 `application/services/QrCodeResolver.php` | --- ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题（P0/P1） 1. **立即移除硬编码凭证**：将 OSS、Redis、短信等第三方密钥全部迁移至配置文件或环境变量，并检查 Git 历史是否已泄露。 2. **补全退款事务**：所有涉及资金状态变更（`_status=4`）与外部支付网关调用的流程，必须包裹在 `$this->db->trans_start()` 中，确保“状态更新”与“退款请求”的原子性。 3. **修复 SQL 注入隐患**：全局搜索 `'_field="' . $var . '"'` 类字符串拼接，统一替换为 CI Query Builder 的数组语法或参数绑定。 4. **PHP 8 兼容性升级**：替换所有 `create_function()`，检查 `srand()`、`ereg()` 等已移除函数，确保代码可平滑升级至 PHP 8.1+。 ### 🛠 后续重构与优化方向 1. **架构分层解耦**： - 当前 `common_helper.php` 已演变为“上帝文件”（超 800 行）。建议按业务域拆分为 `RedisHelper`、`OssService`、`SmsService`、`LockService` 等独立类，并引入依赖注入（DI）替代全局 `$CI`。 - Model 中避免直接处理支付网关逻辑，应抽象出 `PaymentGateway` 接口，通过策略模式对接微信、银联、会员余额等渠道。 2. **缓存策略规范化**： - 废弃 `self::$shop_config` 等请求级静态缓存，改用 CI 的 `Cache` 驱动（Redis/Memcached）并设置合理 TTL。 - Redis 连接改为单例复用，避免频繁 `connect/close`。 3. **代码规范与可维护性**： - 统一命名风格（如 `setTablename` vs `set_table_name`），严格遵循 PSR-12。 - 为长方法（如 `refund_by_notify`、`openRoomByCommunityOrder`）提取私有方法，控制单个方法圈复杂度（Cyclomatic Complexity）< 10。 - 补充关键业务逻辑的注释与 PHPDoc 类型声明，便于后续团队协作与静态分析工具（如 PHPStan）接入。 > 💡 **提示**：若当前系统处于稳定运行期，建议采用**渐进式重构**：优先修复安全与事务漏洞，随后逐步将 Helper 中的 DB 逻辑抽离至 Service 层，最后进行 PHP 版本升级与规范统一。如需针对某个模块（如支付退款链路）输出详细重构方案，可提供更完整的调用上下文。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779870384
updated_unix
1779870384
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel