fix(Security): Minor fixes from scans

deNBI · Jan 9, 2024 · 1d7fb8c · 1d7fb8c
1 parent b1e7dd0
commit 1d7fb8c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/simple_vm_client/VirtualMachineHandler.py b/simple_vm_client/VirtualMachineHandler.py
@@ -141,13 +141,13 @@ def get_server(self, openstack_id: str) -> VM:
         return server
 
     def get_servers(self) -> list[VM]:
-        servers = openstack_servers = self.openstack_connector.get_servers()
+        servers = self.openstack_connector.get_servers()
         servers_full = []
 
         for server in servers:
             servers_full.append(self.forc_connector.get_playbook_status(server=server))
         serv = thrift_converter.os_to_thrift_servers(openstack_servers=servers)
-        return servers_full
+        return serv
 
     def get_servers_by_ids(self, server_ids: list[str]) -> list[VM]:
         return thrift_converter.os_to_thrift_servers(

diff --git a/simple_vm_client/util/thrift_converter.py b/simple_vm_client/util/thrift_converter.py
@@ -67,8 +67,8 @@ def os_to_thrift_volume(openstack_volume: OpenStack_Volume) -> Volume:
         try:
             device = openstack_volume.attachments[0]["device"]
             server_id = openstack_volume.attachments[0]["server_id"]
-        except Exception:
-            pass
+        except Exception as e:
+            logger.log(f"Exception on os_to_thrift_volume:\n{e}")
     volume = Volume(
         status=openstack_volume.status,
         id=openstack_volume.id,