yuluo-yx opened a new pull request, #4396: URL: https://github.com/apache/rocketmq-dashboard/pull/4396
<!-- Make sure the base branch is `master`: that is the RocketMQ Studio trunk. --> ### Which Issue(s) This PR Fixes - Fixes #3136 - Replaces #3137, closed by the branch switch in #4379. ### Brief Description This is a resubmission of #3137 against `master`, following the branch switch notice in #4379. The fork branch is unchanged; `master` is now the RocketMQ Studio trunk. ## Summary - add a read-only cross-instance ACL policy comparison drawer - walk complete bounded user and rule inventories for both selected instances - compare stable user permissions and rule semantics while retaining duplicate logical policies - provide field-level drift evidence, kind/status/search filters, source/target swap, and filtered CSV export - guarantee that credential reveal endpoints and secret material are outside the comparison and export path - add Chinese and English UI text plus focused utility and component coverage ## Design boundaries Policy identity is based on username for users and principal/resource coordinates for rules. Database ids, timestamps, access-key display variants, and secrets are excluded. The workflow does not synchronize or mutate ACL state. ### How Did You Test This Change? ## 原提交验证记录 - `npm test -- --run src/utils/aclPolicyComparison.test.ts src/components/__tests__/AclPolicyComparisonDrawer.test.tsx`: 2 files, 14 tests passed - `npm run lint`: 0 errors; 10 pre-existing warnings outside this change - `npm run build`: TypeScript and Vite production build passed; 8040 modules transformed - `git diff --check`: passed - common base `36126024cbf75ae2e0f27cc074b2f39ceb6c8b6e`: 6 files, 1096 additions, 8 deletions Fixes #3136 ### Checklist - [x] One coherent change; unrelated modifications are not bundled in - [x] Commit subject follows Conventional Commits (`feat:` / `fix:` / `refactor:` / `chore:` / `docs:` / `perf:`) - [x] Tests added or updated for non-trivial changes, test methods named `...Test` - [x] New UI text has both Chinese and English entries under `web/src/i18n/`, or the change does not add UI text - [x] Architecture constraints stay green (`mvn test` runs the ArchUnit checks), or the original verification scope is stated above - [x] New source files carry the ASF license header, or no new source files were added - [x] Documentation touched where behaviour changed (README / `docs/` / in-app help), or no documentation change was required -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
