yuluo-yx opened a new pull request, #1767:
URL: https://github.com/apache/rocketmq-dashboard/pull/1767

   ## What is the purpose of the change
   
   Avoid NullPointerException when trace decoding receives a null message or a 
message without a body. Such input now produces the same empty trace result as 
an empty payload.
   
   Fixes #1763
   
   ## Brief changelog
   
   - Check the message and raw body before constructing the trace-data String.
   - Add regression tests for null messages and null bodies.
   
   ## Verifying this change
   
   - JDK: Temurin 17.0.19
   - Backend-only Maven verification: `mvn -q compiler:compile 
compiler:testCompile -Dtest=<target> surefire:test`
   - `MessageTraceViewTest`: 2 tests, 0 failures, 0 errors
   - `mvn validate`: passed
   - `git diff --check`: passed
   - PR scope check: passed (2 files, 45 changed lines, one commit, base 
`master`)
   - Full lifecycle, integration, E2E, and container checks were not run 
because this five-PR workflow requires lightweight local verification and 
leaves heavyweight checks to upstream CI.
   
   Follow this checklist to help us incorporate your contribution quickly and 
easily.
   
   - [x] Make sure there is a Github issue filed for the change. This PR 
addresses only that issue.
   - [x] Format the pull request title like `[ISSUE #123] Fix UnknownException 
when host config not exist`. Each commit has a meaningful subject and body.
   - [x] Write a pull request description that explains what, how, and why.
   - [x] Write necessary unit tests to verify the logic correction.
   - [ ] Run the repository full basic, install, and integration command 
matrix. The focused backend checks above pass; heavyweight checks are delegated 
to upstream CI.
   - [ ] If this contribution is large, file an Apache Individual Contributor 
License Agreement.


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