return useful information about encountered errors #5354

pull heath wants to merge 1 commits into bitcoin:master from heath:fix/useful-errors-for-bitrpc changing 1 files +36 −39

heath commented at 7:54 AM on November 23, 2014: contributor

Only generic messages are printed to the screen currently.
return useful information about encountered errors f69e421a39
laanwj commented at 11:50 AM on November 24, 2014: member

I agree that it makes sense to show a more detailed error message, but letting exceptions hang out with raise is not an acceptable way to show errors to the user IMO. Also, now you're catching the exceptions just to raise them again :rabbit:
kanzure commented at 6:37 PM on November 24, 2014: contributor

So it seems to me that you want a --verbose option for the script, and the default is to swallow detailed error messages or detailed log messages.
heath commented at 6:48 PM on November 24, 2014: contributor

please close this ticket or merge #5360, sorry it wasn't an update to this PR that new PR addresses your issue @laanwj
sipa commented at 6:50 PM on November 24, 2014: member

You can close this pull request yourself, no?
heath commented at 8:07 PM on November 24, 2014: contributor

@sipa sure
heath closed this on Nov 24, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors