didi / KnowStreaming

一站式云原生实时流数据平台,通过0侵入、插件化构建企业级Kafka服务,极大降低操作、存储和管理实时流数据门槛
https://knowstreaming.com
GNU Affero General Public License v3.0
6.99k stars 1.28k forks source link

[Bugfix]允许为不存在的Topic添加ACL权限(#1051) #1072

Closed HZSUZJ closed 1 year ago

HZSUZJ commented 1 year ago

请不要在没有先创建Issue的情况下创建Pull Request。

变更的目的是什么

需要允许为不存在的Topic添加ACL权限,可能出现Topic还没有,但是想提前增加权限的动作;

简短的更新日志

允许为不存在的Topic添加ACL权限

验证这一变化

所有的topic

02813ae4e3d95eedc9762f129a03ae6

改变前:

2ac3f514cbdfe1358191e5df00a6c3a

改变后:

174e3b88297aa650d955e83324417fe

请遵循此清单,以帮助我们快速轻松地整合您的贡献:

ZQKC commented 1 year ago

请不要在没有先创建Issue的情况下创建Pull Request。

变更的目的是什么

需要允许为不存在的Topic添加ACL权限,可能出现Topic还没有,但是想提前增加权限的动作;

简短的更新日志

允许为不存在的Topic添加ACL权限

验证这一变化

所有的topic

02813ae4e3d95eedc9762f129a03ae6

改变前:

2ac3f514cbdfe1358191e5df00a6c3a

改变后:

174e3b88297aa650d955e83324417fe

请遵循此清单,以帮助我们快速轻松地整合您的贡献:

  • [x] 一个 PR(Pull Request的简写)只解决一个问题,禁止一个 PR 解决多个问题;
  • [x] 确保 PR 有对应的 Issue(通常在您开始处理之前创建),除非是书写错误之类的琐碎更改不需要 Issue ;
  • [x] 格式化 PR 及 Commit-Log 的标题及内容,例如 [Bugfix]修复用户信息修改后,用户列表会抛出空指针异常的问题(#860) #861 。PS:Commit-Log 需要在 Git Commit 代码时进行填写,在 GitHub 上修改不了;
  • [x] 编写足够详细的 PR 描述,以了解 PR 的作用、方式和原因;
  • [x] 编写必要的单元测试来验证您的逻辑更正。如果提交了新功能或重大更改,请记住在 test 模块中添加 integration-test;
  • [x] 确保编译通过,集成测试通过;

1、后续commit-log里面不需要加issue的id,直接就是:[Bugfix]修复消费组Offset重置后,提示重置成功,但是前端不刷新数据,Offset无变化的问题,就可以了; 2、一个pr可以存在多个commit; 3、原先的逻辑不要修改掉,仅仅是在prefixed的这个模式下,不需要校验Topic是否存在;