Add init password hints. #13348
Add init password hints. #13348
Conversation
|
Thanks for your this PR. 🙏 感谢您提交的PR。 🙏 |
增强密码初始化提示及UI布局调整变更文件
💡 小贴士与 lingma-agents 交流的方式📜 直接回复评论
📜 在代码行处标记
📜 在讨论中提问
|
There was a problem hiding this comment.
🔍 代码评审报告
🎯 评审意见概览
| 严重度 | 数量 | 说明 |
|---|---|---|
| 🔴 Blocker | 1 | 阻断性问题，需立即修复。例如：系统崩溃、关键功能不可用或严重安全漏洞。 |
| 🟠 Critical | 0 | 严重问题，高优先级修复。例如：核心功能异常或性能瓶颈影响用户体验。 |
| 🟡 Major | 0 | 主要问题，建议修复。例如：非核心功能缺陷或代码维护性较差。 |
| 🟢 Minor | 1 | 次要问题，酌情优化。例如：代码格式不规范或注释缺失。 |
总计： 2 个问题
📋 评审意见详情
💡 单文件建议
以下是文件级别的代码建议，聚焦于代码的可读性、可维护性和潜在问题。
⚛️ console-ui/src/pages/Register/Register.jsx (1 💬)
- 直接展示明文密码存在安全风险 (L75)
🎨 console-ui/src/pages/Register/index.scss (1 💬)
- CSS类命名不符合BEM规范 (L66-L72)
🚀 跨文件建议
以下是对代码架构和设计的综合分析，聚焦于跨文件交互、系统一致性和潜在优化空间。
🔍 1. 新增密码相关功能缺乏测试覆盖
新增的初始化密码提示功能（如密码明文展示、保存提示等）未在单元测试或E2E测试中体现。当前PR中未包含任何测试文件的更新，可能导致未来维护时功能可靠性无法保障。建议为密码提示展示、用户交互流程等核心场景补充测试用例。
🔍 2. CSS样式定义存在冗余风险
在src/locales/zh-CN.js和en-US.js中新增的internalSysTip3/4提示信息，其对应样式在src/pages/Register/index.scss和console/src/main/resources/static/css/main.css中均存在同名类定义（.internal-sys-tip2）。需确认两个样式文件的加载顺序是否会导致样式覆盖或冲突，建议统一CSS维护规范。
📌 关键代码：
.internal-sys-tip2 { width: 100%; line-height: 15px; font-size: 15px; margin-top: 15px; text-align: center; font-weight: 800; }
.home-page .top-section .login-panel .internal-sys-tip2{width:100%;line-height:15px;font-size:15px;margin-top:15px;text-align:center;font-weight:800}🔍 3. 国际化字符串管理分散
新增的初始化密码相关提示（如internalSysTip3、internalSysTip4、hintSavePassword）分散在多个locales文件中，未形成独立的密码管理模块。建议将密码相关提示集中到独立的password.i18n模块，提升国际化字符串的可维护性
🔍 4. 静态资源版本控制风险
console/src/main/resources/static/index.html中main.css和main.js的版本参数（52d7f4f6947bc2e25f20）与旧版本（fafb9812f71694525187）存在差异，但未提供版本更新说明。需确认该改动是否影响静态资源缓存策略，是否存在版本回滚风险
📌 关键代码：
<link href="./css/main.css?52d7f4f6947bc2e25f20" rel="stylesheet">💡 小贴士
与 lingma-agents 交流的方式
📜 直接回复评论
直接回复本条评论，lingma-agents 将自动处理您的请求。例如：
-
在当前代码中添加详细的注释说明。
-
请详细介绍一下你说的 LRU 改造方案，并使用伪代码加以说明。
📜 在代码行处标记
在文件的特定位置创建评论并 @lingma-agents。例如：
-
@Lingma-Agent 分析这个方法的性能瓶颈并提供优化建议。
-
@Lingma-Agent 对这个方法生成优化代码。
📜 在讨论中提问
在任何讨论中 @lingma-agents 来获取帮助。例如：
-
@Lingma-Agent 请总结上述讨论并提出解决方案。
-
@Lingma-Agent 请根据讨论内容生成优化代码。
| @@ -72,7 +72,7 @@ class Register extends React.Component { | |||
| localStorage.setItem('token', JSON.stringify(res)); | |||
| Dialog.alert({ | |||
| title: locale.Login.initPassword + locale.ListeningToQuery.success, | |||
| content: locale.Password.newPassword + '：' + res.password, | |||
| content: (<div><div>{locale.Password.newPassword}：{res.password}</div><div>{locale.Password.hintSavePassword}</div></div>), | |||
There was a problem hiding this comment.
直接展示明文密码存在安全风险
🔴 Blocker | 🔓 Vulnerabilities
📋 问题详情
在初始化密码成功后，直接在页面上以明文形式展示用户密码（res.password），可能导致密码泄露风险。攻击者可通过截图、网络监控等手段获取用户密码，严重违反安全规范。
💡 解决方案
应将密码值替换为星号或其他安全占位符，并通过其他安全方式通知用户（如邮件/短信）。修改示例：
- content: (<div><div>{locale.Password.newPassword}：{res.password}</div><div>{locale.Password.hintSavePassword}</div></div>),
+ content: (<div><div>{locale.Password.newPassword}：••••••••</div><div>{locale.Password.hintSavePassword}</div></div>),您的反馈对我们很重要！(建议右键在新标签页中打开以下链接)
| .internal-sys-tip2 { | ||
| width: 100%; | ||
| line-height: 15px; | ||
| font-size: 15px; | ||
| margin-top: 15px; | ||
| text-align: center; | ||
| font-weight: 800; |
|
|
Please do not create a Pull Request without creating an issue first.
What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily:
[ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=trueto make sure basic checks pass. Runmvn clean install -DskipITsto make sure unit-test pass. Runmvn clean test-compile failsafe:integration-testto make sure integration-test pass.