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 617 from issue
id
617
repo_id
22
index
77
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：app-260616 - Merge pull request '开房套餐
🔍 代码审查报告：app-260616 - Merge pull request '开房套餐转房增加房态判断。' (#75) from app
...
content
## 自动代码审查报告 **分支**: app-260616 **提交**: `48af25d1e
## 自动代码审查报告 **分支**: app-260616 **提交**: `48af25d1eb8d8e97ce1b9fc107d08baf2a9eb74e` **提交人**: zhangjunnan (121158035@qq.com) **时间**: 2026-06-09 10:36:26 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：代码实现了较为完整的商家端业务逻辑，但整体架构偏向传统的“胖控制器+贫血模型”模式。存在**硬编码敏感密钥**、**SQL注入风险**、**N+1查询性能瓶颈**及**事务处理不规范**等关键问题。代码规范与 PSR-12 存在较大偏差，魔法数字泛滥，可维护性较低。 - **风险等级**：🔴 高 > 📌 *注：提示中提及 `phpci` 框架，但实际代码结构、加载方式（`$this->load->`、`get_instance()`、`BASEPATH`）均明确指向 **CodeIgniter 3.x**。以下审查将基于 CI3 最佳实践与通用 PHP 安全规范进行。* ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `MerchantAppServer.php`<br>(约 150-160 行) | **硬编码敏感 API 密钥**：科大讯飞 `APPID`、`APISecret`、`APIKey` 直接暴露在控制器中，极易导致密钥泄露与盗刷。 | 将敏感配置移至 `application/config/` 或环境变量中，通过 `$this->config->item()` 读取。生产环境严禁硬编码。 | `// config/xfyun.php<br>return ['appid'=>getenv('XFYUN_APPID'), ...];<br>// 控制器中<br>$config = $this->config->item('xfyun');` | | 🔴 严重 | `Ahead_yc_merchant_user_model.php`<br>`get_user_shop_page()` 等方法 | **SQL 注入漏洞**：直接使用字符串拼接构造 `WHERE` 与 `LIKE` 条件（如 `"... like '%" . $params['name'] . "%'"`），未使用查询构建器转义。 | 全面改用 CI3 Query Builder 的 `like()`、`where()` 方法，或手动使用 `$this->db->escape()`。 | `$this->db->like('user._name', $params['name'], 'both');<br>// 替代原有的字符串拼接` | | 🔴 严重 | `Ahead_yc_merchant_user_model.php`<br>`check_token()` | **弱加密验证 Token**：使用 `md5()` 拼接静态盐值验证 Token，存在碰撞风险且无过期机制，易被重放攻击。 | 升级为 `hash_hmac('sha256')` 或引入 JWT。Token 生成时应绑定时间戳并校验有效期。 | `hash_hmac('sha256', $payload, $secret_key);<br>// 结合 Redis 设置 TTL 实现会话管理` | | 🟠 警告 | `Ahead_book_order_model.php`<br>`get_list()` | **N+1 查询性能瓶颈**：在 `foreach` 循环中频繁调用 `$this->ahead_yc_order_model->get_one()` 获取关联数据，导致数据库连接数激增。 | 使用 `WHERE IN` 批量查询或 `JOIN` 关联表一次性获取数据，或使用 CI 缓存机制。 | `$ids = array_column($order_info, 'relation_order_id');<br>$orders = $this->ahead_yc_order_model->get_by_ids($ids);<br>// 循环中通过数组映射获取` | | 🟠 警告 | `Ahead_book_order_model.php`<br>`invalid_book()` | **事务回滚机制不健全**：在 `$this->db->trans_start()` 后直接调用自定义 `throwError()`，若该函数未抛出标准异常，可能导致事务挂起或数据不一致。 | 使用 `try-catch` 包裹事务逻辑，确保异常时强制 `trans_rollback()`，并依赖 CI 的 `trans_status()` 判断。 | `try { $this->db->trans_start(); ... } catch(\Exception $e) { $this->db->trans_rollback(); throw $e; }` | | 🟠 警告 | `MerchantAppServer.php`<br>`index()` | **控制器职责过重 (Fat Controller)**：单方法超 300 行，混杂权限校验、菜单过滤、配置加载、数据组装，违反单一职责原则。 | 将业务逻辑下沉至 `Service` 或 `Repository` 层，控制器仅负责路由分发与请求/响应转换。 | `// 控制器<br>$data = $this->login_service->handleLogin($this->stream);<br>$this->success_response($data);` | | 🟡 建议 | 全局多处 | **魔法数字与硬编码 ID 泛滥**：如 `permission_id = 534`、`$unsetMenuIds = [694, 849...]` 等散落在代码中，后期维护成本极高。 | 提取为常量类（如 `PermissionConst::RECHARGE_INVALID = 534`）或写入配置文件。 | `class MenuConst { const FILTER_IDS = [694, 849, 824, 1250]; }` | | 🟡 建议 | 全局多处 | **命名规范与 PSR-12 不符**：类名使用下划线（`Ahead_book_order_model`），变量混用驼峰与下划线，注释含过期人员标记（`edit by nan 17.10.25`）。 | 遵循 PSR-12：类名 `PascalCase`，方法/属性 `camelCase`，清理历史注释，使用 PHPDoc 规范。 | `class AheadBookOrderModel extends Simple_model { ... }` | | 🟡 建议 | `Ahead_yc_merchant_user_model.php` | **频繁使用 `get_instance()`**：在 Model 中多次调用 `$CI = &get_instance();` 获取 CI 实例，增加耦合且影响性能。 | CI3 的 Model 已继承 `CI_Model`，可直接使用 `$this->load->model()` 或 `$this->config->item()`。 | `// 移除 $CI = &get_instance();<br>$this->load->model('ahead_yc_merchant_model');` | ## 3. 总结与行动建议 ### 🔑 优先修复的关键问题 1. **立即移除硬编码密钥**：将 `MerchantAppServer.php` 中的讯飞 API 密钥、加密盐值等迁移至配置文件或 `.env`，并检查版本控制历史是否已泄露。 2. **修复 SQL 注入漏洞**：全面审查 `Ahead_yc_merchant_user_model.php` 中所有动态拼接 SQL 的地方，替换为 CI Query Builder 或参数化查询。 3. **升级 Token 验证机制**：废弃 MD5 静态盐方案，引入带过期时间的 HMAC-SHA256 签名或标准 JWT，并在 Redis 中维护会话黑名单。 ### 🛠 后续重构与优化方向 1. **架构分层改造**： - 将 `MerchantAppServer::index()` 拆分为 `AuthService`、`MenuService`、`PermissionService`。 - 控制器仅保留：参数接收 -> 调用 Service -> 格式化响应。 2. **数据库查询优化**： - 消除 `get_list()` 等方法的 N+1 查询，改用 `JOIN` 或批量 `WHERE IN`。 - 对高频读取的配置数据（如权限列表、菜单树）引入 Redis 缓存，设置合理 TTL。 3. **规范化与工程化**： - 统一类名与文件命名规范，清理 `//edit by xxx` 等无效注释。 - 将散落的权限 ID、菜单 ID、状态码提取至 `config/constants.php` 或独立常量类。 - 引入静态分析工具（如 PHPStan / Psalm）与代码规范检查（PHP_CodeSniffer PSR-12），纳入 CI/CD 流程。 > ⚠️ **局限性说明**：`MerchantAppServer.php` 代码在 `case "1001"` 处被截断，未能完整审查该分支的结账取消逻辑。若该分支涉及资金扣减或订单状态变更，请补充完整代码以便进行事务与并发安全审查。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780972586
updated_unix
1780972586
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel