Skip to content

Conversation

@FaisalMisbah23
Copy link

@FaisalMisbah23 FaisalMisbah23 commented Jan 1, 2026

Fixes #393

What is the purpose of the change

This PR fixes multiple issues with Dead Letter Queue (DLQ) message functionality in the RocketMQ Dashboard as reported in issue #393:

  1. DLQ messages not visible when searching by consumer group
  2. Message Detail view failing with "Failed to query message by Id: undefined"
  3. Resend Message hanging/failing
  4. Export functionality producing empty Excel files or throwing exceptions

Brief changelog

Backend:

  • DlqMessageServiceImpl.java: Implemented DLQ-specific querying using DefaultMQPullConsumer, fixed resend to use DefaultMQProducer, added system property filtering, fixed pagination total count
  • DlqMessageController.java: Added /resendDlqMessage.do endpoint, improved error handling
  • MessageController.java: Fixed viewMessage to return MessageView directly
  • DlqMessageExcelModel.java: Added null safety checks
  • pom.xml: Upgraded EasyExcel to 3.3.4 for Java 17 compatibility

Frontend:

  • dlqmessage.jsx: Fixed message detail modal, resend, export, pagination; added BornHost/BornTimestamp fields
  • DlqMessageDetailViewDialog.jsx: Added BornHost and BornTimestamp display
  • remoteApi.js: Fixed API parameter passing and response handling

Verifying this change

All functionality manually tested:

  • Query DLQ messages by consumer group (messages appear correctly)
  • View message details (modal opens, all fields display, no undefined errors)
  • Single and batch resend (works correctly, shows success notifications)
  • Single and batch export (Excel files download with correct data)
  • Pagination (correct total count display, page navigation works)
  • Date range filtering (only messages in range are displayed)

Testing environment: RocketMQ Dashboard 2.1.1-SNAPSHOT, RocketMQ 5.3.4, Java 17, Linux

@FaisalMisbah23 FaisalMisbah23 force-pushed the fix-issue-393-dlq-message branch from 40d46d7 to 050d409 Compare January 3, 2026 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

死信消息查不出来

1 participant