yyqdbngt opened a new pull request, #2883:
URL: https://github.com/apache/rocketmq-dashboard/pull/2883

   ## Summary
   - parse stored outbox statuses through a guarded helper in 
`NotificationOutboxService.listDeliveries(alertId)`
   - trim and root-locale uppercase the stored value; unknown or blank statuses 
surface as `FAILED`
   
   ## Why
   `listDeliveries` called `NotificationOutboxStatus.valueOf(row.getStatus())` 
directly. A legacy or partially written row with an unrecognized status (or 
whitespace/case drift) made `valueOf` throw `IllegalArgumentException`, turning 
the whole per-alert delivery listing into a 500. Unknown values now map to 
`FAILED` so they are visible and need manual attention, matching how the retry 
endpoint already treats non-`FAILED` rows as untouched.
   
   ## Testing
   - `cd server && mvn -q -Dtest=NotificationOutboxServiceTest test` — all 
tests pass, including the new 
`mapsUnknownStoredOutboxStatusesToFailedWhenListingTest` which lists a row 
storing `BOUNCED` (→ `FAILED`) and one storing ` pending ` (→ `PENDING`)
   


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