mirror of
https://github.com/hpd840321/starRiverProperty.git
synced 2026-06-09 08:20:31 +08:00
hpd840321
7b2bd307f1
- backend/: 13 Maven modules (cw-elevator-application, cloudwalk-cloud, intelligent-cwoscomponent, ninca-crk, etc.) - frontend/: 4 Vue projects (elevator-front, cwos-portal, alarm-front, front_acs) + decompiled + scripts - scripts/: build, test-env, tools (Docker Compose, service templates, API parity) - docs/: AGENTS.md, superpowers specs, architecture docs - .gitignore: standard Java/Maven exclusions Moved from legacy maven-*/ root layout to backend/ organized structure.
1.6 KiB
1.6 KiB
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 资源管理 为首要整改；其次统一 异常与降级 语义。