suricatasc: factorize command parsing

scripts/suricatasc/src/suricatasc.py

@@ -151,27 +151,16 @@ class SuricataSC:
     def close(self):
         self.socket.close()
 
-    def interactive(self):
+    def parse_command(self, command):
-        print "Command list: " + ", ".join(self.cmd_list)
-        try:
-            readline.set_completer(SuricataCompleter(self.cmd_list))
-            readline.set_completer_delims(";")
-            readline.parse_and_bind('tab: complete')
-            while True:
-                command = raw_input(">>> ").strip()
         arguments = None
         if command.split(' ', 2)[0] in self.cmd_list:
-                    if command == "quit":
-                        break;
             if "pcap-file " in command:
                 try:
                     [cmd, filename, output] = command.split(' ', 2)
                 except:
-                            print "Error: arguments to command '%s' is missing" % (command)
+                    raise SuricataCommandException("Arguments to command '%s' is missing" % (command))
-                            continue
                 if cmd != "pcap-file":
-                            print "Error: invalid command '%s'" % (command)
+                    raise SuricataCommandException("Invalid command '%s'" % (command))
-                            continue
                 else:
                     arguments = {}
                     arguments["filename"] = filename
@@ -180,11 +169,9 @@ class SuricataSC:
                 try:
                     [cmd, iface] = command.split(' ', 1)
                 except:
-                            print "Error: unable to split command '%s'" % (command)
+                    raise SuricataCommandException("Unable to split command '%s'" % (command))
-                            continue
                 if cmd != "iface-stat":
-                            print "Error: invalid command '%s'" % (command)
+                    raise SuricataCommandException("Invalid command '%s'" % (command))
-                            continue
                 else:
                     arguments = {}
                     arguments["iface"] = iface
@@ -192,20 +179,33 @@ class SuricataSC:
                 try:
                     [cmd, variable] = command.split(' ', 1)
                 except:
-                            print "Error: unable to split command '%s'" % (command)
+                    raise SuricataCommandException("Unable to split command '%s'" % (command))
-                            continue
                 if cmd != "conf-get":
-                            print "Error: invalid command '%s'" % (command)
+                    raise SuricataCommandException("Invalid command '%s'" % (command))
-                            continue
                 else:
                     arguments = {}
                     arguments["variable"] = variable
             else:
                 cmd = command
         else:
-                    print "Error: unknown command '%s'" % (command)
+            raise SuricataCommandException("Unknown command '%s'" % (command))
-                    continue
+        return (cmd, arguments)
 
+    def interactive(self):
+        print "Command list: " + ", ".join(self.cmd_list)
+        try:
+            readline.set_completer(SuricataCompleter(self.cmd_list))
+            readline.set_completer_delims(";")
+            readline.parse_and_bind('tab: complete')
+            while True:
+                command = raw_input(">>> ").strip()
+                if command == "quit":
+                    break;
+                try:
+                    (cmd, arguments) = self.parse_command(command)
+                except SuricataCommandException, err:
+                    print err
+                    continue
                 cmdret = self.send_command(cmd, arguments)
                 #decode json message
                 if cmdret["return"] == "NOK":