majialoong opened a new issue, #9809:
URL: https://github.com/apache/rocketmq/issues/9809

   ### Before Creating the Enhancement Request
   
   - [x] I have confirmed that this should be classified as an enhancement 
rather than a bug/feature.
   
   
   ### Summary
   
   When the getAcl method receives a null subject (for example, due to 
incorrect user input or a non-existent subjectType), it currently returns a raw 
NPE error message. This is not user-friendly. We should avoid NPE and provide a 
clear, helpful error message instead.
   
   <img width="1818" height="222" alt="Image" 
src="https://github.com/user-attachments/assets/8661acb0-911f-4c37-83e7-03b62c88f99c";
 />
   
   ### Motivation
   
   Avoid returning raw NPE error messages directly.
   
   ### Describe the Solution You'd Like
   
   Check whether the subject is null and handle the exception accordingly.This 
is also similar to other methods in `AuthorizationMetadataManagerImpl`, such as 
`createAcl` and `updateAcl`.
   
   The improved message prompt is as follows:
   
   <img width="833" height="178" alt="Image" 
src="https://github.com/user-attachments/assets/f557e26b-e08e-4748-835a-01fe66772967";
 />
   
   ### Describe Alternatives You've Considered
   
   No
   
   ### Additional Context
   
   _No response_


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