Skip to content

<fix>[vip]: add checkAccount to l3NetworkUuid in APICreateVipMsg#3356

Open
ZStack-Robot wants to merge 1 commit into5.5.6from
sync/ye.zou/fix/ZSTAC-77017
Open

<fix>[vip]: add checkAccount to l3NetworkUuid in APICreateVipMsg#3356
ZStack-Robot wants to merge 1 commit into5.5.6from
sync/ye.zou/fix/ZSTAC-77017

Conversation

@ZStack-Robot
Copy link
Collaborator

Resolves: ZSTAC-77017

Target: 5.5.6

sync from gitlab !9186

…project admin access

Resolves: ZSTAC-77017

Change-Id: Id4ade94464e906de33b916b5aac3037afd289414
@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

演示说明

该拉取请求修改了 APICreateVipMsg 类中的 l3NetworkUuid 参数注解,为 APIParam 添加了 checkAccount = true 属性,以启用 L3 网络 UUID 参数的账户验证功能。

变更详情

变更群组 / 文件 总结
VIP API 参数验证
plugin/vip/src/main/java/org/zstack/network/service/vip/APICreateVipMsg.java
l3NetworkUuid 字段的 @APIParam 注解中添加 checkAccount = true 参数,启用账户级别的资源验证检查。

代码审查工作量

🎯 1 (微不足道) | ⏱️ ~3 分钟

诗歌

🐰 参数一改添验证,
账户检查把关把,
VIP 网络更安全,
小小改动大保障。

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 拉取请求标题完全符合格式要求,长度64字符(≤72),遵循[scope]: 格式,清楚地总结了主要变更。
Description check ✅ Passed 拉取请求描述与变更集相关,提供了JIRA问题号和目标分支信息,与APICreateVipMsg中添加checkAccount验证的变更相关联。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into 5.5.6

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sync/ye.zou/fix/ZSTAC-77017

No actionable comments were generated in the recent review. 🎉


Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants