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 616 from issue
id
616
repo_id
22
index
76
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：app - Merge pull request 'Merge pull
🔍 代码审查报告：app - Merge pull request 'Merge pull request '合并0519' (#
...
content
## 自动代码审查报告 **分支**: app **提交**: `518d00cf2a78cb28
## 自动代码审查报告 **分支**: app **提交**: `518d00cf2a78cb28e24dab21ba3d2f2827974f81` **提交人**: zhangjunnan (121158035@qq.com) **时间**: 2026-06-09 10:36:17 --- ## 1. 审查摘要 - **代码质量评分**：4.5 / 10 - **总体评价**：代码已实现核心业务链路，但存在**严重的安全隐患**（硬编码凭证、非原子锁、SQL拼接）、**架构反模式**（Model内加载Model、巨型Switch、全局状态污染）及**性能瓶颈**（频繁建连Redis、循环内N+1查询）。整体工程化程度较低，不符合现代 PHP 及 CI3 最佳实践。 - **风险等级**：🔴 高 > 📌 **框架说明**：提交代码实际基于 **CodeIgniter 3** 架构（使用 `CI_Controller`、`get_instance()`、`$this->load->` 等）。若 `phpci` 为内部定制分支，部分加载机制可对照官方文档微调，但底层安全与性能原则通用。 --- ## 2. 问题详情 | 严重程度 | 文件/位置 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `common_helper.php` / `alioss_*` 函数 | **硬编码云厂商 AccessKey/Secret**。直接暴露在公共 Helper 中，极易被泄露导致云资源被恶意操作。 | 移至 `application/config/oss.php` 或环境变量，通过 CI 配置加载。禁止在代码中明文存储密钥。 | `$config = $CI->config->item('aliyun_oss');`<br>`$param = $config['credentials'];` | | 🔴 严重 | `common_helper.php` / `doRedisLockRelease` | **Redis 分布式锁非原子释放**。`GET` 校验后 `DEL` 存在竞态条件，可能误删其他请求持有的锁。 | 使用 Lua 脚本保证 `GET` 与 `DEL` 的原子性，或使用 `Redis::eval`。 | `lua = "if redis.call('get',KEYS[1]) == ARGV[1] then return redis.call('del',KEYS[1]) else return 0 end"`<br>`$redis->eval($lua, [$key, $value], 1);` | | 🔴 严重 | `Ahead_book_order_model.php` / `refund_by_notify` | **字符串拼接构造 SQL 条件**：`$log_where = '_relation_id="' . $order_data['_id'] . '"...'` 存在 SQL 注入风险，且绕过 CI 查询构造器。 | 全面使用 CI Query Builder 或参数绑定，禁止手动拼接 WHERE 子句。 | `$this->db->where('_relation_id', $order_data['_id'])`<br>`->where('_status', 1)`<br>`->where_in('_type', [5,13]);` | | 🟠 警告 | `Api.php` / `selfChangeRoom` | **全局状态污染**：直接修改 `$CI->merchant_id`、`$CI->admin_data`。在并发请求或 CLI 环境下会导致数据串扰。 | 通过方法参数传递上下文，或使用独立的 `Context` 对象/Session 存储，避免修改 CI 超全局实例。 | `// 移除 $CI->admin_data = ...`<br>`$orderService->handle($merchant_id, $admin_data, $params);` | | 🟠 警告 | `MerchantAppServer.php` / `index()` | **上帝方法 (God Method)**：单方法超 500 行，包含巨型 `switch`，违反单一职责原则，极难维护与测试。 | 按 `function` 路由拆分至独立 Controller 或 Service 类。主方法仅负责鉴权与路由分发。 | `// 路由分发示例`<br>`$action = $request['function'];`<br>`$service = new MerchantService($this);`<br>`return $service->$action($request);` | | 🟠 警告 | `common_helper.php` / `get_aliyun_redis_conn` | **频繁创建/关闭 Redis 连接**。每次调用都 `connect()` + `close()`，高并发下将耗尽文件描述符并引发严重延迟。 | 使用 CI 内置 Redis 驱动或单例模式复用连接。生产环境建议启用 `persistent` 连接。 | `$CI->load->driver('cache', ['adapter' => 'redis']);`<br>`$redis = $CI->cache->redis;` | | 🟠 警告 | `Ahead_book_order_model.php` / `invalid_book` | **事务中断风险**：`throwError()` 直接 `exit/die`，导致 `$this->db->trans_start()` 未执行 `trans_complete()`，连接池可能泄漏或状态不一致。 | 改用抛出 `Exception` 或返回错误数组，在 `try-catch` 中统一处理事务回滚。 | `try { $this->db->trans_start(); ... $this->db->trans_complete(); } catch(\Exception $e) { $this->db->trans_rollback(); throw $e; }` | | 🟡 建议 | 多个 Model 文件 | **Model 内部加载 Model**：`$CI = &get_instance(); $CI->load->model(...)` 破坏依赖注入，增加耦合。 | 模型应在 Controller 层加载，或通过 CI `autoload.php` 预加载。若需跨模型调用，应在 `__construct` 中注入。 | `// 移除内部 $CI->load->model()`<br>`// 在 Controller 中统一加载`<br>`$this->load->model('Ahead_yc_merchant_model');` | | 🟡 建议 | `common_helper.php` / `decodeUnicode` | **使用已废弃函数**：`create_function` 在 PHP 7.2+ 已废弃，存在安全风险且性能差。 | 替换为匿名函数（闭包）。 | `return preg_replace_callback('/\\\\u([0-9a-f]{4})/i', fn($m) => mb_convert_encoding(pack('H*', $m[1]), 'UTF-8', 'UCS-2BE'), $str);` | | 🟡 建议 | `Api.php` / `jsonEcho` | **冗余输出缓冲控制**：`ob_end_clean(); ob_start(); ... ob_end_flush();` 易引发 Header 已发送警告，且逻辑冗余。 | 直接设置 Header 后输出 JSON 并终止，CI 已处理基础缓冲。 | `header('Content-Type: application/json; charset=utf-8');`<br>`echo json_encode($result, JSON_UNESCAPED_UNICODE);`<br>`exit;` | --- ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题（P0） 1. **立即移除硬编码凭证**：将 `common_helper.php` 中的 OSS Key/Secret 迁移至配置文件或环境变量，并轮换已泄露的密钥。 2. **修复分布式锁竞态条件**：将 `doRedisLockRelease` 改为 Lua 脚本原子操作，防止业务数据错乱。 3. **消除 SQL 注入隐患**：全面审查 `Ahead_book_order_model.php` 及其他 Model 中的字符串拼接 SQL，统一替换为 CI Query Builder 或 PDO 参数绑定。 4. **规范事务处理**：移除 `throwError()` 中的 `exit()`，改用异常机制确保 `$this->db->trans_rollback()` 必执行。 ### 🛠 后续重构与优化方向 1. **架构解耦**： - 拆分 `MerchantAppServer::index()` 巨型方法，采用 **路由分发器 + 独立 Service 类** 模式。 - 禁止在 Model 内部使用 `get_instance()->load->model()`，改为构造函数依赖注入或 CI 自动加载。 2. **性能提升**： - 替换 `get_aliyun_redis_conn` 为 CI 官方 Redis 驱动，启用长连接。 - 解决 `get_list` / `get_detail` 中的 **N+1 查询问题**：使用 `JOIN` 或批量 `WHERE IN` 预加载关联数据。 3. **规范与安全**： - 遵循 PSR-12 规范：统一类名/文件名大小写、移除重复函数定义（如 `rs_hmset`）、补充类型声明与 PHPDoc。 - 替换 `passport_encrypt` 弱加密算法为 `openssl_encrypt` 或 `sodium`。 - 限制 `Access-Control-Allow-Origin: *`，改为白名单校验或动态反射 Origin。 > ⚠️ **局限性说明**：部分文件（如 `MerchantAppServer.php`、`common_helper.php`、`Ahead_book_order_model.php`）末尾被截断，未完整覆盖所有分支逻辑与异常处理路径。建议补充完整代码后进行二次深度审查。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780972577
updated_unix
1780972577
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel