RockteMQ-AI commented on issue #4223:
URL: 
https://github.com/apache/rocketmq-dashboard/issues/4223#issuecomment-5632193381

   **Issue Evaluation**
   
   Category: `bug` | Status: **Confirmed**
   
   The reported issue has been verified against the described codebase.
   
   **Root Cause:** `AuthService.listUsers` passes the raw search string 
directly to MyBatis-Plus `.like(\"username\", normalizedSearch)` without 
escaping SQL LIKE wildcards (`\`, `%`, `_`). This causes `_` and `%` in user 
input to be interpreted as SQL wildcards rather than literal characters.
   
   **Impact:** Incorrect search results when usernames or search queries 
contain `_` or `%`. This is a correctness issue and a minor security concern 
(information leakage through wildcard matching).
   
   **Severity:** Medium — affects search accuracy; consistent with similar bugs 
already fixed in #4192–#4194.
   
   **Fix path:** Escape `\`, `%`, and `_` in the search string before passing 
to `.like()`, following the same pattern already applied to instance, 
credential, and operation-audit search paths. Add a focused unit regression 
test.
   
   An automated fix proposal can be generated. Reply `/approve` to proceed with 
PR generation.
   
   ---
   *Automated evaluation by github-manager*


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