Verbose command didn't propery add/remove the ?error_trace http param

crate · Jul 22, 2017 · b5093c0 · b5093c0
1 parent 681ed9a
commit b5093c0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/CHANGES.txt b/CHANGES.txt
@@ -5,6 +5,9 @@ Changes for crash
 Unreleased
 ==========
 
+ - Fixed: ``\verbose`` command didn't not switch on verbosity if ``crash`` was
+   launched without the ```--verbose` flag.
+
 2017/07/11 0.21.3
 =================
 

diff --git a/src/crate/crash/commands.py b/src/crate/crash/commands.py
@@ -109,6 +109,11 @@ class ToggleVerboseCommand(Command):
     @noargs_command
     def __call__(self, cmd, *args, **kwargs):
         cmd.error_trace = not cmd.error_trace
+        if cmd.error_trace:
+            cmd.connection.client.path += '?error_trace=1'
+        else:
+            cmd.connection.client.path.replace('?error_trace=1', '')
+
         return 'Verbose {0}'.format(
             cmd.error_trace and 'ON' or 'OFF'
         )