yyqdbngt opened a new pull request, #1132: URL: https://github.com/apache/rocketmq-dashboard/pull/1132
The restart/upgrade/delete endpoints for brokers, NameServers and proxies wrapped a boolean `success` flag in `Result.ok`, so a failed operation (if a service ever returned `false` instead of throwing) would reach the client as HTTP 200 with `success=false` — a success envelope masking a failure. The controllers now throw a `BusinessException` when the operation reports failure, so a failed operation is an actual error response, and the success-flag field is gone from the payload. Updated the affected controller tests. -- 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]
