more information and care on KeyboardInterrupt

mycli is already _much_ better at cancelling queries on
KeyboardInterrupt, compared to the vendor client.  But not perfect --
and sometimes the result is an unfortunate runaway query.

Here we:
 * catch the case that "ok" is not in the status string, and emit a
   warning in red.  Previously we only caught Exceptions.
 * add an echo to the debugged path in which cancellation is skipped.
 * downgrade the routine cancellation message to blue and add the query
   id number.

The first and second changes are intended to help the user notice the
cases in which we fail to interrupt, and know that we failed.  The
third change is intended to help in unknown cases, in which we believe
that we succeeded, but didn't interrupt.

Author: rolandwalker

changelog.md

@@ -5,6 +5,7 @@ Features
 --------
 
 * Support SSL query parameters on DSNs.
+* More information and care on KeyboardInterrupt.
 
 Internal
 --------

mycli/main.py

@@ -774,11 +774,15 @@ def one_iteration(text=None):
                             status_str = str(status).lower()
                             if status_str.find("ok") > -1:
                                 logger.debug("cancelled query, connection id: %r, sql: %r", connection_id_to_kill, text)
-                                self.echo("cancelled query", err=True, fg="red")
+                                self.echo(f"Cancelled query id: {connection_id_to_kill}", err=True, fg="blue")
+                            else:
+                                logger.debug("Failed to confirm query cancellation, connection id: %r, sql: %r", connection_id_to_kill, text)
+                                self.echo(f"Failed to confirm query cancellation, id: {connection_id_to_kill}", err=True, fg="red")
                     except Exception as e:
                         self.echo("Encountered error while cancelling query: {}".format(e), err=True, fg="red")
                 else:
                     logger.debug("Did not get a connection id, skip cancelling query")
+                    self.echo("Did not get a connection id, skip cancelling query", err=True, fg="red")
             except NotImplementedError:
                 self.echo("Not Yet Implemented.", fg="yellow")
             except OperationalError as e: