tju-yxq commented on PR #1918: URL: https://github.com/apache/rocketmq-dashboard/pull/1918#issuecomment-5269734106
I reviewed PR #1918 against this issue and ran the focused tests locally. Backend verification: ```bash mvn -q "-Dtest=ProxyAddressServiceTest,ProxyCompatControllerTest,ProxyControllerTest" test ``` Result: `BUILD SUCCESS`. Frontend verification: ```bash npm ci NODE_OPTIONS=--no-experimental-webstorage npx vitest run src/api/proxy.test.ts src/pages/studio/__tests__/Proxy.test.tsx ``` Result: `2 passed` test files, `11 passed` tests. The fix looks broader than the original report: it persists proxy address preferences, scopes them by cluster/instance, retains legacy compatibility endpoints, and covers backend/API/frontend behavior. I also narrowed the issue wording to clarify that the original report was based on code inspection rather than a live restart reproduction. -- 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]
