Loyal-Young opened a new pull request, #6909: URL: https://github.com/apache/rocketmq-dashboard/pull/6909
## What is the purpose of the change Fixes #5998. On the current `rocketmq-studio` branch, mock DLQ search still compares the raw term case-sensitively. The real `DLQService` trims the term and `RocketMQDLQProvider` compares the group name and DLQ topic case-insensitively. Operators using the demo cannot find a group with the same query that works against a broker. The earlier PR #5999 addressed this issue but was automatically closed after inactivity without merging. This patch was reproduced and verified independently on the current branch. ## Brief changelog - Normalize the mock search term once by trimming and lowercasing it. - Match both group name and DLQ topic with the same case-insensitive behavior as the backend. - Add regressions for padded uppercase group and topic terms plus an all-whitespace term. ## Verification - New focused test failed before the fix (`ORDER` returned no groups). - `vitest run src/services/messageService.test.ts`: 13 passed after the fix. - TypeScript `tsc --noEmit`: passed. - ESLint on the two changed files: passed without warnings. - `git diff --check`: passed. ## AI assistance AI assistance helped compare the demo service with the backend search contract and draft the regression cases. I reviewed the current branch behavior and ran the checks above. -- 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]
