dbcli / mssql-cli

A command-line client for SQL Server with auto-completion and syntax highlighting
BSD 3-Clause "New" or "Revised" License
1.35k stars 191 forks source link

JSON RPC exception suppressing #359

Open ellbosch opened 4 years ago

ellbosch commented 4 years ago

There are quite a few areas in the JSON RPC module where exceptions are being suppressed. I believe we should be raising exceptions here, or at the very least providing more information to the user that their request encountered a failure.

I'm currently testing exception raising in this branch, and have also created a test build which runs every day to see if new exceptions are raised. As of the time of this writing, no exceptions have been raised.