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

   ## Summary
   
   - authenticate the logout request in `AuthControllerTest` so the interceptor 
allows the controller path under test
   - provide the newly required `ApacheAclReadService` collaborator in the 
credential authorization MVC slice
   - preserve production authentication behavior while restoring the focused 
regression suites
   
   This fixes #1812.
   
   ## Root cause
   
   The authentication interceptor now participates in the controller slice 
whenever `AuthService` is present, but the logout fixture did not mark its 
token as authenticated. Separately, `AclController` gained an 
`ApacheAclReadService` constructor dependency without the focused authorization 
slice adding that collaborator.
   
   ## Verification
   
   ```text
   mvn -Dtest=AuthControllerTest,AuthCredentialAuthorizationIntegrationTest test
   Tests run: 11, Failures: 0, Errors: 0, Skipped: 0
   
   mvn -Dtest=org.apache.rocketmq.studio.auth.*Test test
   Tests run: 56, Failures: 0, Errors: 0, Skipped: 0
   
   Checkstyle violations: 0
   git diff --check: passed
   ```
   
   Before the change, the exact upstream head produced one 401 assertion 
failure and three application-context errors in these two suites.


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