yyqdbngt opened a new pull request, #2217: URL: https://github.com/apache/rocketmq-dashboard/pull/2217
## Summary - deep-copy broker, proxy, and NameServer objects when returning cluster snapshots - copy incoming cluster configuration before storing it - cover nested read mutation, TPS history isolation, and update-input aliasing with regression tests ## Root cause `ClusterRepositoryImpl` created fresh list containers but reused their mutable elements. It also assigned the caller-owned `ClusterConfigVO` directly during updates. Callers could therefore change cached repository state without invoking another repository operation. This complements #1528/#1531, which isolated configuration objects returned by reads; it does not change that earlier fix. ## Validation - `mvn -f server/pom.xml -Dtest=ClusterRepositoryImplTest test` — 4 tests passed - Maven validate lifecycle — 0 Checkstyle violations Closes #2212 -- 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]
