youngkermit8-coder opened a new pull request, #2046:
URL: https://github.com/apache/rocketmq-dashboard/pull/2046

   ## What changed
   
   - require administrator permission for `GET /api/acl/remote/rules`
   - preserve reader access to the local ACL rule list
   - cover normalized trailing-slash and matrix-parameter paths to prevent 
authorization bypasses
   
   ## Why
   
   The remote ACL discovery endpoint uses the server's stored administrative 
RocketMQ credentials to connect to every broker and returns live ACL 2.0 
subjects, resources, source IPs, and broker addresses. It was classified as a 
regular read-only endpoint, so any authenticated reader could invoke this 
privileged discovery operation.
   
   ## Validation
   
   - confirmed the new non-admin regression cases fail before the production 
change
   - `mvn 
-Dtest=AuthInterceptorTest,AuthServiceTest,AuthControllerTest,AuthCorsIntegrationTest
 test` — 55 tests passed
   - `mvn test` — 1,059 tests passed, 0 failures; Checkstyle reported 0 
violations
   
   Fixes #2045
   


-- 
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]

Reply via email to