web interface: fix erasing the db when connected

Author: AlyaGomaa

config/slips.yaml

@@ -81,12 +81,13 @@ parameters:
    # zeek breaks the connection into smaller connections
    tcp_inactivity_timeout : 60
 
-   # Should we delete the previously stored data in the DB when we start??
+   # Should we delete the previously stored data in the DB when we start?
    # By default False. Meaning we don't DELETE the DB by default.
    deletePrevdb : True
+
    # You can remember the data in all the previous runs of the DB if you put False.
-   # Redis will remember as long as the redis server is not down. The persistance is
-   # on the memory, not disk.
+   # Redis will remember as long as the redis server is not down. The persistence is
+   # in memory, not disk.
    # deletePrevdb : False
 
    # Set the label for all the flows that are being read.

slips_files/common/parsers/config_parser.py

@@ -514,7 +514,7 @@ def mac_db_update_period(self):
             update_period = 1209600  # 2 weeks
         return update_period
 
-    def deletePrevdb(self):
+    def delete_prev_db(self):
         return self.read_configuration("parameters", "deletePrevdb", True)
 
     def rotation_period(self):

slips_files/core/database/database_manager.py

@@ -38,9 +38,9 @@ def __init__(
             self.logger, redis_port, start_redis_server, **kwargs
         )
 
-        # in some rare cases we don't wanna start sqlite,
-        # like when using -S
-        # we just want to connect to redis to get the PIDs
+        # in some rare cases we don't wanna create the sqlite db from scratch,
+        # like when using -S to stop the daemon, we just wanna connect to
+        # the existing one
         self.sqlite = None
         if start_sqlite:
             self.sqlite = self.create_sqlite_db(output_dir)

slips_files/core/database/redis_db/database.py

@@ -184,7 +184,7 @@ def _set_redis_options(cls):
     @classmethod
     def _read_configuration(cls):
         conf = ConfigParser()
-        cls.deletePrevdb: bool = conf.deletePrevdb()
+        cls.deletePrevdb: bool = conf.delete_prev_db()
         cls.disabled_detections: List[str] = conf.disabled_detections()
         cls.width = conf.get_tw_width_as_float()
         cls.client_ips: List[str] = conf.client_ips()

slips_files/core/database/sqlite_db/database.py

@@ -15,7 +15,10 @@
 
 
 class SQLiteDB:
-    """Stores all the flows slips reads and handles labeling them"""
+    """
+    Stores all the flows slips reads and handles labeling them
+    Creates a new db and connects to it if there's none in the given output_dir
+    """
 
     name = "SQLiteDB"
     # used to lock each call to commit()

webinterface/database/database.py

@@ -52,7 +52,8 @@ def get_db_manager_obj(self, port: int = False) -> Optional[DBManager]:
                 output_dir,
                 port,
                 start_redis_server=False,
-                start_sqlite=False,
+                start_sqlite=True,
+                flush_db=False,
             )
         except RuntimeError:
             return