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 468 from issue
id
468
repo_id
22
index
55
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：app-260519 - 1
content
## 自动代码审查报告 **分支**: app-260519 **提交**: `cba73c16a
## 自动代码审查报告 **分支**: app-260519 **提交**: `cba73c16a1126fa6360fe68b6349e638ce336128` **提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com) **时间**: 2026-06-02 15:51:41 --- ## 1. 审查摘要 - **代码质量评分**：5.5 / 10 分 - **总体评价**：代码实现了商家端核心业务路由与鉴权逻辑，但存在严重的安全硬编码、密码明文比对隐患、控制器职责过重（God Class）及大量魔法数字。整体架构偏向过程式堆砌，缺乏面向对象封装与配置化管理，可维护性与扩展性较差。 - **风险等级**：🔴 高 > 📌 **框架说明**：代码结构、加载方式（`$this->load->`、`$this->db->`、`defined('BASEPATH')`）高度契合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，部分生命周期与组件行为请以官方文档为准。本审查基于 CI3 最佳实践进行。 > ⚠️ **局限性说明**：代码在 `case "1001"` 处被截断，后续逻辑无法评估。以下审查基于已提供片段。 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `~330行` | **敏感信息硬编码**：科大讯飞 TTS 的 `APISecret`、`APIKey` 直接写在控制器中，极易随代码库泄露。 | 移至环境变量或受保护的配置文件（如 `application/config/secrets.php`），通过 `getenv()` 或 `$this->config->item()` 读取。 | `// config/secrets.php<br>'xfyun' => [<br> 'app_id' => getenv('XFYUN_APPID'),<br> 'api_key' => getenv('XFYUN_APIKEY'),<br> 'api_secret' => getenv('XFYUN_APISECRET')<br>]<br>// 控制器中<br>$config = $this->config->item('xfyun');` | | 🔴 严重 | `~480行` | **密码明文比对**：`$data['_discount_pwd'] != $_old_password` 暗示密码可能以明文或弱哈希存储，违反安全基线。 | 数据库必须存储 `password_hash()` 生成的哈希值，验证时使用 `password_verify()`。 | `if (!password_verify($_old_password, $data['_discount_pwd'])) {<br> $this->error_response('旧密码不正确');<br>}<br>$new_hash = password_hash($_new_password1, PASSWORD_DEFAULT);<br>$this->ahead_yc_merchant_user_model->update(['_discount_pwd' => $new_hash], ['_id' => $uid]);` | | 🔴 严重 | `全局` | **响应方法未终止执行**：`$this->error_response()` / `$this->success_response()` 调用后未 `exit` 或 `return`，导致后续代码继续执行，可能引发重复响应或逻辑越权。 | 确保响应方法内部调用 `exit()`，或在调用处使用 `return $this->error_response(...)`。 | `// 推荐写法<br>return $this->error_response("参数错误");<br>// 或在 error_response 方法末尾添加<br>exit(json_encode($response));` | | 🟠 警告 | `~35行` | **CORS 策略过于宽松**：`Access-Control-Allow-Origin: *` 允许任意域名跨域请求商家接口，易被恶意站点利用。 | 根据实际业务限制来源域名，或使用动态白名单校验 `Origin` 头。 | `$origin = $_SERVER['HTTP_ORIGIN'] ?? '';<br>$allowed = ['https://merchant.yourdomain.com'];<br>if (in_array($origin, $allowed)) {<br> header("Access-Control-Allow-Origin: $origin");<br>}` | | 🟠 警告 | `__construct` | **条件跳过初始化导致状态不一致**：`if ($this->router->fetch_method() !== "uploadPic")` 跳过了数据库、配置、模型的加载。若 `uploadPic` 依赖这些组件将直接报错。 | 将公共初始化移至 `__construct` 顶部，仅将鉴权逻辑放入条件判断，或为 `uploadPic` 单独创建控制器。 | `parent::__construct();<br>$this->load->database('default');<br>$this->config->load('config_talent');<br>// 鉴权逻辑独立<br>if ($this->router->fetch_method() !== 'uploadPic') {<br> $this->_check_auth();<br>}` | | 🟠 警告 | `index()` | **巨型路由方法违反单一职责**：`index()` 方法超千行，包含登录、个人中心、打印机、支付配置等数十个业务分支，难以测试与维护。 | 按业务模块拆分为独立控制器（如 `AuthController`, `ProfileController`, `PrinterController`），或使用框架路由映射。 | `// 路由配置示例<br>$route['merchant/(:any)'] = 'MerchantApp/$1';<br>// 拆分后<br>class AuthController extends AplicationController { public function login() { ... } }` | | 🟠 警告 | `~380,410行` | **冗余获取 CI 实例**：在控制器内部使用 `$CI = &get_instance();` 赋值属性，控制器本身已是实例，此操作多余且易引发作用域混淆。 | 直接使用 `$this` 访问属性或方法。 | `// 移除<br>$CI = &get_instance();<br>$CI->token = $result['token'];<br>// 改为<br>$this->token = $result['token'];` | | 🟡 建议 | `全局` | **大量魔法数字与硬编码 ID**：如 `'0005'`, `694`, `849`, `533` 等散落在代码中，业务规则变更时需全局搜索替换。 | 提取为类常量或独立配置文件（如 `config/menu_ids.php`）。 | `const FUNC_LOGIN = '0005';<br>const MENU_FILTER_IDS = [694, 849, 824, 1250];<br>if (in_array($v['id'], self::MENU_FILTER_IDS)) { ... }` | | 🟡 建议 | `类属性` | **属性全部声明为 `public`**：破坏封装性，外部可直接修改内部状态，增加不可预知的副作用风险。 | 改为 `protected` 或 `private`，通过 getter/setter 或构造函数初始化。 | `protected $stream;<br>protected $expire_sms_time = 600;<br>protected $current_version = "2.0";` | | 🟡 建议 | `~35行` | **拼写错误**：基类名 `AplicationController` 缺少字母 `p`，若为框架核心类名不一致将导致致命错误。 | 修正为 `ApplicationController`（需确认框架实际基类名称）。 | `class MerchantAppServer extends ApplicationController` | ## 3. 总结与行动建议 ### 🚨 优先修复项（P0） 1. **移除硬编码密钥**：立即将 `xfyun_tts_config` 中的 `APISecret` 和 `APIKey` 迁移至环境变量或服务器安全配置中，并检查 Git 历史是否已泄露。 2. **修复密码验证逻辑**：全面排查 `_discount_pwd` 字段存储方式，强制实施 `password_hash()` / `password_verify()` 标准流程。 3. **阻断响应后代码执行**：全局检查 `error_response()` 与 `success_response()` 实现，确保输出 JSON 后立即 `exit`，或统一改为 `return` 模式。 ### 🛠 重构与优化方向 1. **控制器瘦身（SRP 原则）**：当前 `MerchantAppServer` 承担了路由分发、鉴权、登录、个人中心、打印机管理、支付配置等职责。建议按业务域拆分为 4~5 个独立控制器，`index()` 仅保留路由分发或废弃改用框架原生路由。 2. **配置与常量集中化**：将菜单过滤 ID、权限 ID、支付平台映射、API 版本号等提取至 `application/config/merchant_rules.php`，避免业务逻辑与配置耦合。 3. **性能优化**： - `case '0005'` 中存在大量串行 DB 查询（`get_admin_menu`, `get_one`, `get_modules` 等）。建议对高频读取的配置类数据引入 Redis 缓存（框架已提供 `Cache` 组件）。 - 菜单过滤逻辑使用 `foreach` + `unset` + `array_values` 效率较低，可改用 `array_filter` 或预定义白名单映射。 4. **规范与可维护性**： - 统一属性可见性为 `protected`。 - 清理过期注释（如 `//edit by nan 17.10.25`），改用 Git 提交记录追溯变更。 - 补充输入校验：`json_decode` 后应检查 `json_last_error()`，防止畸形 JSON 导致后续数组访问报错。 > 💡 **后续建议**：在重构前，建议为当前核心接口补充 PHPUnit 单元测试或集成测试，确保拆分与优化过程中业务逻辑不发生退化。若需针对 `case "1001"` 截断部分进行深度审查，请提供完整代码片段。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1780386701
updated_unix
1780386701
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel