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 330 from issue
id
330
repo_id
22
index
35
poster_id
1
original_author
original_author_id
0
name
🔍 代码审查报告：app-260519 - 需求 订单回执
content
## 自动代码审查报告 **分支**: app-260519 **提交**: `bb7c35b05
## 自动代码审查报告 **分支**: app-260519 **提交**: `bb7c35b05eb2515e2a121fc31bf84e7d1c9d9947` **提交人**: chenjunfeng (developer.jeff.c@gmail.com) **时间**: 2026-05-26 15:31:55 --- ## 1. 审查摘要 - **代码质量评分**：4 / 10 分 - **总体评价**：代码呈现典型的早期 CodeIgniter 3 架构风格（注：项目结构及 `get_instance()`、`$this->load->model()` 等用法高度契合 CI3，若 `phpci` 为内部定制框架，核心规范仍适用）。整体存在较多历史技术债务，包含**硬编码敏感凭证、已废弃 PHP 语法、非原子 Redis 操作、性能反模式及逻辑缺陷**。代码可读性与可维护性较低，未遵循现代 PHP (PSR-12) 规范，需进行系统性重构。 - **风险等级**：🔴 高 ## 2. 问题详情 | 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) | | :--- | :--- | :--- | :--- | :--- | | 🔴 严重 | `common_helper.php`<br>`alioss_deleteObject` / `alioss_addObject` | **硬编码云厂商 AccessKey/SecretKey**，极易导致 OSS 资源被盗刷、数据泄露或恶意篡改。 | 立即将凭证迁移至配置文件或环境变量，通过框架配置加载。 | `'accessId' => getenv('ALI_OSS_ACCESS_KEY') ?? config_item('ali_oss.access_id')` | | 🔴 严重 | `common_helper.php`<br>`decodeUnicode` | 使用 `create_function()`，该函数在 PHP 7.2 已废弃，**PHP 8.0+ 将直接抛出致命错误**。 | 替换为现代匿名函数（Closure）。 | `return preg_replace_callback('/\\\\u([0-9a-f]{4})/i', function($m) { return mb_convert_encoding(pack("H*", $m[1]), "UTF-8", "UCS-2BE"); }, $str);` | | 🔴 严重 | `common_helper.php`<br>`doBatchFromTable` | **逻辑缺陷**：`foreach` 中 `$data` 为值拷贝，修改 `$data[$k]` 不会同步回 `$list`，导致函数实际无效。 | 直接赋值回原数组对应键。 | `foreach ($list as $k => $data) { $list[$k] = doSingleFromTable($data); } return $list;` | | 🔴 严重 | `common_helper.php`<br>`getFirstCharter` | 使用 `$str{0}` 语法，PHP 7.4 已废弃，**PHP 8.0 将报 Fatal Error**。 | 统一改为方括号数组访问 `$str[0]`。 | `$fchar = ord($str[0]);` | | 🟠 警告 | `common_helper.php`<br>`rs_*` 系列函数 | **严重性能瓶颈**：每次调用都通过 `get_aliyun_redis_conn()` 新建连接并 `close()`，高频调用将耗尽连接池并拖慢响应。 | 采用单例模式或复用 CI 内置 Redis 驱动，保持长连接/连接池复用。 | `static $redis = null; if(!$redis) $redis = get_aliyun_redis_conn(); return $redis->sMembers($keyName);` | | 🟠 警告 | `common_helper.php`<br>`rs_set` | **竞态条件**：先 `exists()` 再 `set()` 非原子操作，高并发下可能覆盖其他进程写入的值。 | 直接使用 Redis 的 `NX` 选项保证原子性。 | `$obj->set($keyName, $string, ['nx' => true, 'ex' => $expireTime]);` | | 🟠 警告 | `common_helper.php`<br>`doRedisLockRelease` | **非原子解锁**：先 `GET` 校验再 `DEL`，校验与删除之间存在时间窗口，可能误删其他客户端持有的锁。 | 使用 Lua 脚本保证 `GET` 与 `DEL` 的原子性。 | `$lua = 'if redis.call("get",KEYS[1]) == ARGV[1] then return redis.call("del",KEYS[1]) else return 0 end'; return $redis->eval($lua, [$key, $value], 1);` | | 🟠 警告 | `Ahead_shop_config_second_model.php`<br>`get_shop_setting` | **缓存键冲突**：`self::$shop_config[$shop_id]` 仅以门店ID为键，未结合商户ID，多租户架构下极易串数据。 | 使用复合键缓存。 | `$cacheKey = "{$merchant_id}_{$shop_id}"; if (empty(self::$shop_config[$cacheKey])) { ... self::$shop_config[$cacheKey] = ... }` | | 🟠 警告 | `common_helper.php`<br>`curlRequest` | `CURLOPT_SSLVERSION` 设为 `1` (SSLv3) 已被现代 TLS 协议废弃，新版 cURL 可能报错或降级为不安全协议。 | 强制使用 TLSv1.2+。 | `curl_setopt($curl, CURLOPT_SSLVERSION, CURL_SSLVERSION_TLSv1_2);` | | 🟡 建议 | `common_helper.php`<br>`get_mac_by_url` | Helper 中频繁调用 `get_instance()` 加载 Model 违反 MVC 分层原则，且 URL 解析逻辑冗长脆弱。 | 将业务查询逻辑下沉至 Model/Service 层，Helper 仅保留纯字符串处理。 | 提取 `UrlResolverService::resolveMac($url)` | | 🟡 建议 | `common_helper.php`<br>`passport_encrypt/decrypt` | 自定义 XOR 加密算法强度极低，且使用已废弃的 `srand()`。不符合现代安全标准。 | 替换为 `openssl_encrypt()` 或 CI 内置 `Encryption` 库。 | `openssl_encrypt($str, 'AES-256-CBC', $key, 0, $iv);` | | 🟡 建议 | `Ahead_shop_config_second_model.php`<br>类外部 `$CI` | 在类外部定义 `$CI = &get_instance();` 不符合 CI 模型实例化机制，可能导致上下文污染。 | 移除全局 `$CI`，在方法内按需 `$this->load->model()` 或通过构造函数注入。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` | | 🟡 建议 | `common_helper.php`<br>`parseQrcodeContent` | 依赖外部 `zxing.org` 且使用正则解析 HTML，存在 **SSRF 风险** 且极易因目标站改版失效。 | 引入本地二维码解析扩展（如 `endroid/qr-code` 或 `chillerlan/php-qrcode`）。 | 使用 Composer 引入成熟库替代 HTTP 请求 | > 📝 **局限性说明**：`common_helper.php` 末尾的 `import($file)` 函数代码被截断，无法完整评估其文件加载逻辑与安全性。建议补充完整代码以便进一步审查。 ## 3. 总结与行动建议 ### 🚨 优先修复项（P0） 1. **移除硬编码凭证**：立即将阿里云 AK/SK、数据库密码、固定 IP 等敏感信息迁移至 `.env` 或框架配置中心，杜绝泄露风险。 2. **兼容 PHP 8+**：全局替换 `create_function`、`$str{0}`、`srand()` 等已废弃语法，确保系统可平滑升级至现代 PHP 版本。 3. **修复数据操作 Bug**：修正 `doBatchFromTable` 的无效赋值逻辑，避免业务层数据转换静默失败。 ### 🛠 架构与性能优化方向 1. **Redis 连接复用**：废弃“每次调用新建/关闭连接”的模式。建议封装 `RedisClient` 单例类，或使用 CI 的 `Cache` 驱动统一管理，降低网络开销。 2. **分层重构**：`common_helper.php` 已演变为“上帝文件”，混杂了 URL 解析、加密、OSS、短信、Redis、日志、排序等数十种职责。建议按领域拆分为 `services/` 目录下的独立类，Helper 仅保留轻量级工具函数。 3. **原子性保障**：所有分布式锁、缓存写入操作必须保证原子性。优先使用 Redis 原生命令（如 `SET NX EX`）或 Lua 脚本，避免 `GET` + `SET/DEL` 组合。 4. **规范与类型安全**：逐步引入 PHP 7.4+/8.0 类型声明（`declare(strict_types=1);`、参数类型、返回值类型），统一遵循 PSR-12 编码规范，提升代码可测试性与 IDE 支持度。 ### 📖 框架适配提示 当前代码结构高度依赖 **CodeIgniter 3** 的生命周期（如 `BASEPATH` 守卫、`get_instance()`、`$this->load->model()`）。若项目确为 `phpci` 定制框架，请确认其是否完全兼容 CI3 的 Loader 机制。对于不确定的框架特定组件（如 `Simple_model` 基类行为、`throwError` 全局函数），建议查阅 `phpci` 官方文档或内部架构规范，确保异常处理与模型加载符合框架设计哲学。 --- *此 Issue 由代码审查服务自动创建*
...
milestone_id
0
priority
0
is_closed
0
is_pull
0
num_comments
0
ref
deadline_unix
0
created_unix
1779780715
updated_unix
1779780715
closed_unix
0
is_locked
0
content_version
0
time_estimate
0
Delete
Cancel