#2510: cleaning up logs and add logging handler clearing to prevent duplicate logs

src/primaite/simulator/network/hardware/base.py

@@ -1360,7 +1360,6 @@ class Node(SimComponent):
         self.software_manager.install(application)
         application_instance = self.software_manager.software.get(str(application.__name__))
         self.applications[application_instance.uuid] = application_instance
-        self.sys_log.info(f"Installed application {application_instance.name}")
         _LOGGER.debug(f"Added application {application_instance.name} to node {self.hostname}")
         self._application_request_manager.add_request(
             application_instance.name, RequestType(func=application_instance._request_manager)

src/primaite/simulator/system/applications/application.py

@@ -131,7 +131,6 @@ class Application(IOSoftware):
         """Install Application."""
         super().install()
         if self.operating_state == ApplicationOperatingState.CLOSED:
-            self.sys_log.info(f"Installing Application {self.name}")
             self.operating_state = ApplicationOperatingState.INSTALLING
 
     def receive(self, payload: Any, session_id: str, **kwargs) -> bool:

src/primaite/simulator/system/core/software_manager.py

@@ -97,7 +97,6 @@ class SoftwareManager:
         software.software_manager = self
         self.software[software.name] = software
         self.port_protocol_mapping[(software.port, software.protocol)] = software
-        self.sys_log.info(f"Installed {software.name}")
         if isinstance(software, Application):
             software.operating_state = ApplicationOperatingState.CLOSED
 

src/primaite/simulator/system/core/sys_log.py

@@ -52,6 +52,7 @@ class SysLog:
         file_handler.setFormatter(logging.Formatter(log_format))
 
         self.logger = logging.getLogger(f"{self.hostname}_sys_log")
+        self.logger.handlers.clear()  # clear handlers
         self.logger.setLevel(logging.DEBUG)
         self.logger.addHandler(file_handler)
 

src/primaite/simulator/system/services/ntp/ntp_client.py

@@ -115,7 +115,6 @@ class NTPClient(Service):
         :param timestep: The current timestep number. (Amount of time since simulation episode began)
         :type timestep: int
         """
-        self.sys_log.info(f"{self.name} apply_timestep")
         super().apply_timestep(timestep)
         if self.operating_state == ServiceOperatingState.RUNNING:
             # request time from server

tests/assets/configs/basic_switched_network.yaml

@@ -75,6 +75,7 @@ simulation:
       default_gateway: 192.168.10.1
       dns_server: 192.168.1.10
       applications:
+      - type: RansomwareScript
       - type: WebBrowser
         options:
           target_url: http://arcd.com/users/