Fix for #26

- Check that we have a token before trying to save a command. Would previously
  403 for users that were not logged in.

Author: rcaloras

bashhub/bashhub.py

@@ -60,6 +60,11 @@ def save(command, path, pid, process_start_time, exit_status):
     if bh_filter and re.findall(bh_filter, command):
         return
 
+    # Check that we have an auth token.
+    if bashhub_globals.BH_AUTH() == "":
+        print("No auth token found. Run 'bashhub setup' to login.")
+        return
+
     command = CommandForm(command, path, exit_status, pid, pid_start_time)
     rest_client.save_command(command)
 

bashhub/version.py

@@ -1 +1 @@
-__version__ =   '1.0.0'
+__version__ =   '1.0.1-dev'

tests/test_bashhub.py

@@ -40,6 +40,15 @@ def print_failed(command):
     result = runner.invoke(bashhub, args)
     assert '' == result.output
 
+    def no_auth_token():
+        return ''
+
+   # Should not try to save a command if we don't have an auth token
+    bashhub_globals.BH_AUTH = no_auth_token
+    result = runner.invoke(bashhub, ['save', 'date', '/tmp', '1', '1000','1'])
+    error_message = "No auth token found. Run 'bashhub setup' to login.\n"
+    assert error_message == result.output
+
 def test_bashhub_version():
     runner = CliRunner()
     result = runner.invoke(bashhub, ['version'])