pilichoumao opened a new issue, #6304:
URL: https://github.com/apache/rocketmq-dashboard/issues/6304

   ### Bug
   
   The text output paths of `rmqctl config set-context`, `use-context`, 
`delete-context`, and `current-context` ignore the errors returned by 
`fmt.Fprintf/Fprintln`. When stdout rejects output, these commands report exit 
0 despite producing no result. JSON output and `get-contexts` correctly exit 
nonzero for the same failure.
   
   ### Reproduction
   
   Build the unchanged `rocketmq-studio` base 
`5e4c39b053c35a5598cc79ff331b8a54e649d7b1`. With a synthetic temporary config, 
redirect a command's stdout file descriptor to a read-only file, which returns 
EBADF. All four text commands exit 0; their JSON equivalents exit 1. No server 
requests or real credentials are involved.
   
   ### Expected behavior and scope
   
   Propagate text output errors so automation receives a nonzero exit and the 
underlying output failure. Configuration mutations occur before their 
confirmation output: preserve the successfully saved configuration and do not 
roll back or replay it merely because confirmation output failed. Keep normal 
output and structured formats compatible.
   
   Related #5209 fixes only the `version` command; #5144 adds config argument 
completion. Neither covers these four output sites. This report targets the 
configuration command lifecycle, without changing the global error taxonomy or 
terminal cell formatting.
   


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