mirror of
https://github.com/hpd840321/starRiverProperty.git
synced 2026-06-09 08:20:31 +08:00
反编译工作区
21 lines
1.6 KiB
Markdown
21 lines
1.6 KiB
Markdown
# 04 `maven-cloudwalk-intelligent-davinci-manager` — 阿里巴巴规范走查（整理版）
|
||
|
||
**范围**：共 **13** 个 `*.java`（`davinci-manager-common` + `davinci-manager-storage`）。
|
||
|
||
## 问题清单（节选）
|
||
|
||
| 严重度 | 位置 | 类别 | 说明 | 建议 |
|
||
|--------|------|------|------|------|
|
||
| P0 | `FileStorageManagerImpl.java` | 安全 | `fileDownLoadWithAbsoluteUrl` 将用户传入 URL 作为 Feign `target`，**SSRF** 风险 | URL 白名单、协议限制、禁止内网/metadata |
|
||
| P0 | `FileStorageManagerImpl.java` | 资源 | `feign.Response` 未在 `finally`/`try-with-resources` 中 **close**，易连接泄漏 | 确保关闭 Response |
|
||
| P1 | `FileStorageManagerImpl.java` | 健壮性 | `DavinciResult` 未系统判空 | 统一空安全/错误模型 |
|
||
| P1 | `FileStorageManagerImpl.java` | 异常 | `catch (Exception)` 过宽；错误码空串、未链 cause | 收窄、记录日志、`initCause` |
|
||
| P1 | `FileManagerFeignClientFallback.java` | 容错 | Fallback 一律 `RuntimeException`，无结构化降级 | 返回失败 `DavinciResult` 或统一异常类型 + 日志 |
|
||
| P1 | `OuterCallFeignClient.java` | Feign | `@RequestLine("GET ")` 路径为空，易误用 | 明确相对路径或文档约束 |
|
||
| P2 | `DavinciServiceException.java` | API | 继承受检 `Exception` | 评估改为运行时异常 + 错误码 |
|
||
| P2 | 接口参数名 | 命名 | `paramString` 等反编译式命名 | 语义化重命名（与调用方协同） |
|
||
|
||
## 小结
|
||
|
||
**P0** 以 **SSRF + Feign Response 资源管理** 为首要整改；其次统一 **异常与降级** 语义。
|