From c47d3d362e27bc7f03cae2b1e7fbf086f8d2f34b Mon Sep 17 00:00:00 2001
From: David Slusser <david.slusser@sealstorage.io>
Date: Wed, 27 Dec 2023 18:37:29 -0800
Subject: [PATCH 1/4] fixing stale process handling

---
 docs/source/version_history.rst          |  2 ++
 src/djangoaddicts/hostutils/views/gui.py | 23 +++++++++++++++--------
 2 files changed, 17 insertions(+), 8 deletions(-)

diff --git a/docs/source/version_history.rst b/docs/source/version_history.rst
index f5408fd..78a40a5 100644
--- a/docs/source/version_history.rst
+++ b/docs/source/version_history.rst
@@ -15,3 +15,5 @@ Version History
    "0.0.5", "removing ajax view; using only htmx"
    "0.0.6", "updating templates for latest Bootstrap"
    "0.0.7", "adding isort; adding badges to readme"
+   "0.0.8", "fixed modal trigger on host process card"
+   "0.0.9", "fixed stale process handling in host process view"
diff --git a/src/djangoaddicts/hostutils/views/gui.py b/src/djangoaddicts/hostutils/views/gui.py
index 4e1a16d..ef546c4 100644
--- a/src/djangoaddicts/hostutils/views/gui.py
+++ b/src/djangoaddicts/hostutils/views/gui.py
@@ -139,16 +139,23 @@ def get(self, request, *args, **kwargs):
         context["title"] = self.title
         context["now"] = datetime.datetime.now()
         context["subtitle"] = psutil.os.uname()[1]
-        process_list = list(psutil.process_iter())
-        context["process_list"] = process_list
         counts = {
-            "running": len([i for i in process_list if i.status() == "running"]),
-            "sleeping": len([i for i in process_list if i.status() == "sleeping"]),
-            "idle": len([i for i in process_list if i.status() == "idle"]),
-            "stopped": len([i for i in process_list if i.status() == "stopped"]),
-            "zombie": len([i for i in process_list if i.status() == "zombie"]),
-            "dead": len([i for i in process_list if i.status() == "dead"]),
+            "running": 0,
+            "sleeping": 0,
+            "idle": 0,
+            "stopped": 0,
+            "zombie": 0,
+            "dead": 0,
+            "disk-sleep": 0
         }
+        process_list = list(psutil.process_iter())
+        context["process_list"] = process_list
+        for process in process_list:
+            try:
+                counts[process.status()] += 1
+            except psutil.NoSuchProcess:
+                continue
+
         context["counts"] = counts
         filter_form = {}
         filter_form["form"] = HostProcessFilterForm(request.GET or None)

From 4ccc74feaca70e55e6e25c4d1e74881c132b5aab Mon Sep 17 00:00:00 2001
From: David Slusser <david.slusser@sealstorage.io>
Date: Sat, 30 Dec 2023 15:21:03 -0800
Subject: [PATCH 2/4] updating process capture; updating modal button and
 trigger

---
 docs/source/version_history.rst               |  2 +-
 pyproject.toml                                |  2 +-
 .../templates/hostutils/bs5/detail/cpu.html   |  2 +-
 .../templates/hostutils/bs5/detail/disk.html  |  2 +-
 .../hostutils/bs5/detail/network.html         |  2 +-
 src/djangoaddicts/hostutils/views/gui.py      | 26 ++----
 src/djangoaddicts/hostutils/views/htmx.py     | 65 ++++++--------
 tests/unit/test_gui.py                        | 17 ++++
 tests/unit/test_htmx.py                       | 88 +++++++++----------
 9 files changed, 98 insertions(+), 108 deletions(-)

diff --git a/docs/source/version_history.rst b/docs/source/version_history.rst
index 78a40a5..9dc348b 100644
--- a/docs/source/version_history.rst
+++ b/docs/source/version_history.rst
@@ -16,4 +16,4 @@ Version History
    "0.0.6", "updating templates for latest Bootstrap"
    "0.0.7", "adding isort; adding badges to readme"
    "0.0.8", "fixed modal trigger on host process card"
-   "0.0.9", "fixed stale process handling in host process view"
+   "0.0.9", "fixed stale process handling in host process view; updated modals"
diff --git a/pyproject.toml b/pyproject.toml
index 058fa80..da81842 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -108,7 +108,7 @@ ignore_missing_imports = true
 
 # https://docs.pytest.org/en/6.2.x/customize.html#pyproject-toml
 [tool.pytest.ini_options]
-addopts = "-s -v -x --strict-markers -m 'not extra' --doctest-modules"
+#addopts = "-s -v -x --strict-markers -m 'not extra' --doctest-modules"
 filterwarnings = [
     "ignore::UserWarning",
 ]
diff --git a/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/cpu.html b/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/cpu.html
index 46284aa..8c9c309 100644
--- a/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/cpu.html
+++ b/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/cpu.html
@@ -168,7 +168,7 @@
                     <ul class="list-group mx-auto justify-content-center" style="max-width: 93%; margin-top: .5%; margin-bottom: .5%">
                         {% for cpu, data in cpu_data.items %}
                         <li class="list-group-item bg-light shadow-sm hvr-grow mb-3">
-                            <a href="#" hx-get="{% url 'hostutils:get_cpu_stats' cpu %}" hx-target="#modal_wrapper">
+                            <a href="#" hx-get="{% url 'hostutils:get_cpu_stats' cpu %}" hx-target="#modal_wrapper" data-bs-toggle="modal" data-bs-target="#modal_wrapper">
                                 <div class="row">
                                     <div class="col-xs-12 col-md-2"><span class="key">CPU:</span><span class="value">{{ cpu }}</span></div>
                                     <div class="col-xs-12 col-md-2"><span class="key">Percent:</span><span class="value">{{ data.percent }}%</span></div>
diff --git a/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/disk.html b/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/disk.html
index 9248943..68706bb 100644
--- a/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/disk.html
+++ b/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/disk.html
@@ -136,7 +136,7 @@
                     <ul class="list-group mx-auto justify-content-center" style="max-width: 93%; margin-top: .5%; margin-bottom: .5%">
                         {% for item in partition_lists %}
                         <li class="list-group-item bg-light shadow-sm hvr-grow mb-3">
-                            <a href="#" hx-get="{% url 'hostutils:get_partition_stats' %}?part={{ item.mountpoint }}" hx-target="#modal_wrapper">
+                            <a href="#" hx-get="{% url 'hostutils:get_partition_stats' %}?part={{ item.mountpoint }}" hx-target="#modal_wrapper" data-bs-toggle="modal" data-bs-target="#modal_wrapper">
                                 <div class="row mb-1">
                                     <div class="col-xs-12 col-md-3"><span class="key">Device:</span><span class="value">{{ item.device }}</span></div>
                                     <div class="col-xs-12 col-md-2"><span class="key">Filesystem:</span><span class="value">{{ item.fstype }}</span></div>
diff --git a/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/network.html b/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/network.html
index 195e0ee..61c7d5a 100644
--- a/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/network.html
+++ b/src/djangoaddicts/hostutils/templates/hostutils/bs5/detail/network.html
@@ -141,7 +141,7 @@
                     <ul class="list-group mx-auto justify-content-center" style="max-width: 93%; margin-top: .5%; margin-bottom: .5%">
                         {% for k,v in interface_list.items %}
                         <li class="list-group-item bg-light shadow-sm hvr-grow mb-3">
-                            <a href="#" hx-get="{% url 'hostutils:get_interface_stats' k %}" hx-target="#modal_wrapper">
+                            <a href="#" hx-get="{% url 'hostutils:get_interface_stats' k %}" hx-target="#modal_wrapper" data-bs-toggle="modal" data-bs-target="#modal_wrapper">
                                 <div class="row">
                                     <div class="key h5">{{ k }}:</div>
                                     {% for item in v %}
diff --git a/src/djangoaddicts/hostutils/views/gui.py b/src/djangoaddicts/hostutils/views/gui.py
index ef546c4..1311342 100644
--- a/src/djangoaddicts/hostutils/views/gui.py
+++ b/src/djangoaddicts/hostutils/views/gui.py
@@ -148,24 +148,16 @@ def get(self, request, *args, **kwargs):
             "dead": 0,
             "disk-sleep": 0
         }
-        process_list = list(psutil.process_iter())
-        context["process_list"] = process_list
-        for process in process_list:
+        process_list = []
+        for process in psutil.process_iter():
             try:
                 counts[process.status()] += 1
-            except psutil.NoSuchProcess:
-                continue
-
+                process_list.append({"pid": process.pid, 
+                                     "name": process.name(), 
+                                     "status": process.status(), 
+                                     "started_at": process.create_time()})
+            except (psutil.NoSuchProcess, psutil.AccessDenied):
+                pass
         context["counts"] = counts
-        filter_form = {}
-        filter_form["form"] = HostProcessFilterForm(request.GET or None)
-        filter_form["modal_name"] = "filter_processes"
-        filter_form["modal_size"] = "modal-lg"
-        filter_form["modal_title"] = "Filter Host Processes"
-        filter_form["hx_method"] = "hx-get"
-        filter_form["hx_url"] = "/hostutils/get_host_processes"
-        filter_form["hx_target"] = "id_process_list_container"
-        filter_form["method"] = "GET"
-        filter_form["action"] = "Filter"
-        context["filter_form"] = filter_form
+        context["process_list"] = process_list
         return render(request, self.template_name, context=context)
diff --git a/src/djangoaddicts/hostutils/views/htmx.py b/src/djangoaddicts/hostutils/views/htmx.py
index ac50c0e..ec24f56 100644
--- a/src/djangoaddicts/hostutils/views/htmx.py
+++ b/src/djangoaddicts/hostutils/views/htmx.py
@@ -14,8 +14,7 @@
 class GetHostCpuStats(BuildBootstrapModalView):
     """Get statistics for a given CPU"""
 
-    modal_button_close = None
-    modal_button_submit = "Close"
+    modal_button_submit = None
     modal_size = "modal-lg"
     modal_title = "CPU Details"
 
@@ -38,8 +37,7 @@ def get(self, request, *args, **kwargs):
 class GetHostNetworkStats(BuildBootstrapModalView):
     """Get statistics for a given network interface"""
 
-    modal_button_close = None
-    modal_button_submit = "Close"
+    modal_button_submit = None
     modal_size = "modal-lg"
     modal_title = "Network Interface Details"
 
@@ -57,8 +55,7 @@ def get(self, request, *args, **kwargs):
 class GetHostParitionStats(BuildBootstrapModalView):
     """Get statistics for a given disk partition"""
 
-    modal_button_close = None
-    modal_button_submit = "Close"
+    modal_button_submit = None
     modal_title = "Partition Details"
 
     def get(self, request, *args, **kwargs):
@@ -76,39 +73,31 @@ def get(self, request, *args, **kwargs):
 class GetHostProcesses(View):
     """Get host processes"""
 
-    @staticmethod
-    def get_process_count(process_list: list, status: str) -> int:
-        """get a count of processes for a given status
-
-        Args:
-            process_list (list): list of processes as returned from psutil.process_iter()
-            status (str): name of process status to count
-
-        Returns:
-            int: number of processes of 'status'
-        """
-        count = 0
-        for process in process_list:
-            try:
-                if process.status() == status:
-                    count += 1
-            except psutil.NoSuchProcess:
-                continue
-        return count
-
     def get(self, request):
         """Get host prcesses"""
         context = {}
-        process_list = list(psutil.process_iter())
         filter_form = HostProcessFilterForm(request.GET or None)
-        context["counts"] = {
-            "running": self.get_process_count(process_list, "running"),
-            "sleeping": self.get_process_count(process_list, "sleeping"),
-            "idle": self.get_process_count(process_list, "idle"),
-            "stopped": self.get_process_count(process_list, "stopped"),
-            "zombie": self.get_process_count(process_list, "zombie"),
-            "dead": self.get_process_count(process_list, "dead"),
+        counts = {
+            "running": 0,
+            "sleeping": 0,
+            "idle": 0,
+            "stopped": 0,
+            "zombie": 0,
+            "dead": 0,
+            "disk-sleep": 0
         }
+        
+        process_list = []
+        for process in psutil.process_iter():
+            try:
+                counts[process.status()] += 1
+                process_list.append({"pid": process.pid, 
+                                     "name": process.name(), 
+                                     "status": process.status(), 
+                                     "started_at": process.create_time()})
+            except (psutil.NoSuchProcess, psutil.AccessDenied):
+                continue
+        context["counts"] = counts
 
         if request.GET.dict().get("clear", None):
             context["clear_filter"] = False
@@ -120,20 +109,20 @@ def get(self, request):
                 if filter_form.cleaned_data.get("status", None):
                     filtered_process_list = []
                     for i in process_list:
-                        if i.status() in filter_form.cleaned_data["status"]:
+                        if i["status"] in filter_form.cleaned_data["status"]:
                             filtered_process_list.append(i)
                     process_list = filtered_process_list
 
                 if filter_form.cleaned_data.get("created_at__gte", None):
                     filtered_process_list = []
                     for i in process_list:
-                        if i.create_time() > filter_form.cleaned_data["created_at__gte"].timestamp():
+                        if i["started_at"] > filter_form.cleaned_data["created_at__gte"].timestamp():
                             filtered_process_list.append(i)
                     process_list = filtered_process_list
                 if filter_form.cleaned_data.get("created_at__lte", None):
                     filtered_process_list = []
                     for i in process_list:
-                        if i.create_time() < filter_form.cleaned_data["created_at__lte"].timestamp():
+                        if i["started_at"] < filter_form.cleaned_data["created_at__lte"].timestamp():
                             filtered_process_list.append(i)
                     process_list = filtered_process_list
 
@@ -154,7 +143,7 @@ def get(self, request):
         context["filter_form"] = filter_form
         return render(request, template_name="hostutils/bs5/snippets/host_process_card_swap.htm", context=context)
 
-
+        
 class GetHostProcessStats(BuildBootstrapModalView):
     """Get statistics for a given process"""
 
diff --git a/tests/unit/test_gui.py b/tests/unit/test_gui.py
index 81640be..7103097 100644
--- a/tests/unit/test_gui.py
+++ b/tests/unit/test_gui.py
@@ -1,5 +1,9 @@
 from django.test import TestCase
 from django.shortcuts import reverse
+from unittest.mock import patch
+import psutil
+import subprocess
+import itertools
 
 
 class ShowHostCpuTests(TestCase):
@@ -66,3 +70,16 @@ def test_get(self):
         response = self.client.get(url)
         self.assertEqual(response.status_code, 200)
         self.assertTemplateUsed(response, "hostutils/bs5/detail/processes.html")
+
+    def test_get_with_invalid(self):
+        """verify page is redered if psutil.NoSuchProcess is raised"""
+        url = reverse("hostutils:host_process")
+        process_list = psutil.process_iter()
+        with patch("psutil.process_iter") as mocked_process_list:
+            p = subprocess.Popen("ls", stdout=subprocess.PIPE)
+            mp = iter((psutil.Process(p.pid), psutil.Process(p.pid)))
+            p.communicate()
+            mocked_process_list.return_value = itertools.chain(mp, process_list)
+            response = self.client.get(url)
+            self.assertEqual(response.status_code, 200)
+            self.assertTemplateUsed(response, "hostutils/bs5/detail/processes.html")
diff --git a/tests/unit/test_htmx.py b/tests/unit/test_htmx.py
index f0d1621..14a725c 100644
--- a/tests/unit/test_htmx.py
+++ b/tests/unit/test_htmx.py
@@ -11,6 +11,7 @@
 
 class GetHostProcessessTests(TestCase):
     """test GetHostProcesses ajax view"""
+
     def setUp(self):
         super(GetHostProcessessTests, self).setUp()
         self.url = reverse("hostutils:get_host_processes")
@@ -45,70 +46,59 @@ def test_get_with_status_running(self):
             self.client.get(self.url, data={"status": "running"}, **self.headers)
 
     def test_get_with_invalid(self):
-        with self.assertRaises(psutil.NoSuchProcess):
+        process_list = psutil.process_iter()
+        with patch("psutil.process_iter") as mocked_process_list:
+            p = subprocess.Popen("ls", stdout=subprocess.PIPE)
+            mp = iter((psutil.Process(p.pid), psutil.Process(p.pid)))
+            p.communicate()
+            mocked_process_list.return_value = itertools.chain(mp, process_list)
+            self.client.get(self.url, **self.headers)
+
+    def test_get_with_status_invalid(self):
+        with patch("psutil.Process.status") as status:
+            status.return_value = "running"
             process_list = psutil.process_iter()
-            with patch('psutil.process_iter') as mocked_process_list:
-                p = subprocess.Popen('ls', stdout=subprocess.PIPE)
+            with patch("psutil.process_iter") as mocked_process_list:
+                p = subprocess.Popen("ls", stdout=subprocess.PIPE)
                 mp = iter((psutil.Process(p.pid), psutil.Process(p.pid)))
                 p.communicate()
-                mocked_process_list.return_value = itertools.chain(
-                    mp, 
-                    process_list
-                )
-                self.client.get(self.url, **self.headers)
-
-    def test_get_with_status_invalid(self):
-        with self.assertRaises(psutil.NoSuchProcess):
-            with patch("psutil.Process.status") as status:
-                status.return_value = "running"
-                process_list = psutil.process_iter()
-                with patch('psutil.process_iter') as mocked_process_list:
-                    p = subprocess.Popen('ls', stdout=subprocess.PIPE)
-                    mp = iter((psutil.Process(p.pid), psutil.Process(p.pid)))
-                    p.communicate()
-                    mocked_process_list.return_value = itertools.chain(
-                        mp, 
-                        process_list
-                    )
-                    self.client.get(self.url, data={"status": "running"}, **self.headers)
+                mocked_process_list.return_value = itertools.chain(mp, process_list)
+                self.client.get(self.url, data={"status": "running"}, **self.headers)
 
     def test_get_with_gte_invalid(self):
-        with self.assertRaises(psutil.NoSuchProcess):
-            with patch("psutil.Process.status") as status:
-                status.return_value = "running"
-                process_list = psutil.process_iter()
-                with patch('psutil.process_iter') as mocked_process_list:
-                    p = subprocess.Popen('ls', stdout=subprocess.PIPE)
-                    mp = iter((psutil.Process(p.pid), psutil.Process(p.pid)))
-                    p.communicate()
-                    mocked_process_list.return_value = itertools.chain(
-                        mp, 
-                        process_list
-                    )
-                    self.client.get(self.url, data={"status": "running", 
-                                                    "created_at__gte": self.now - timezone.timedelta(days=1)}, 
-                                    **self.headers)
+        with patch("psutil.Process.status") as status:
+            status.return_value = "running"
+            process_list = psutil.process_iter()
+            with patch("psutil.process_iter") as mocked_process_list:
+                p = subprocess.Popen("ls", stdout=subprocess.PIPE)
+                mp = iter((psutil.Process(p.pid), psutil.Process(p.pid)))
+                p.communicate()
+                mocked_process_list.return_value = itertools.chain(mp, process_list)
+                self.client.get(
+                    self.url,
+                    data={"status": "running", "created_at__gte": self.now - timezone.timedelta(days=1)},
+                    **self.headers,
+                )
 
     def test_get_with_lte_invalid(self):
-        # with self.assertRaises(psutil.NoSuchProcess):
         with patch("psutil.Process.status") as status:
             status.return_value = "running"
             process_list = psutil.process_iter()
-            with patch('psutil.process_iter') as mocked_process_list:
-                p = subprocess.Popen('ls', stdout=subprocess.PIPE)
+            with patch("psutil.process_iter") as mocked_process_list:
+                p = subprocess.Popen("ls", stdout=subprocess.PIPE)
                 mp = iter((psutil.Process(p.pid), psutil.Process(p.pid)))
                 p.communicate()
-                mocked_process_list.return_value = itertools.chain(
-                    mp, 
-                    process_list
+                mocked_process_list.return_value = itertools.chain(mp, process_list)
+                self.client.get(
+                    self.url,
+                    data={"status": "running", "created_at__lte": self.now - timezone.timedelta(days=1)},
+                    **self.headers,
                 )
-                self.client.get(self.url, data={"status": "running", 
-                                                "created_at__lte": self.now - timezone.timedelta(days=1)}, 
-                                **self.headers)
 
 
 class GetHostCpuStatsTests(TestCase):
     """test GetHostCpuStats htmx view"""
+
     def setUp(self):
         super(GetHostCpuStatsTests, self).setUp()
         self.url = reverse("hostutils:get_cpu_stats", kwargs={"cpu": 1})
@@ -129,9 +119,9 @@ def test_get_with_invalid_request(self):
         self.assertEqual(response.status_code, 400)
 
 
-
 class GetHostNetworkStatsTests(TestCase):
     """test GetHostNetworkStats htmx view"""
+
     def setUp(self):
         super(GetHostNetworkStatsTests, self).setUp()
         self.url = reverse("hostutils:get_interface_stats", kwargs={"interface": "lo"})
@@ -154,6 +144,7 @@ def test_get_with_invalid_request(self):
 
 class GetHostParitionStatsTests(TestCase):
     """test GetHostParitionStats htmx view"""
+
     def setUp(self):
         super(GetHostParitionStatsTests, self).setUp()
         self.url = reverse("hostutils:get_partition_stats") + "?part=/"
@@ -176,6 +167,7 @@ def test_get_with_invalid_request(self):
 
 class GetHostProcessStatsTests(TestCase):
     """test GetHostProcessStats htmx view"""
+
     def setUp(self):
         super(GetHostProcessStatsTests, self).setUp()
         self.url = reverse("hostutils:get_process_stats", kwargs={"pid": 1})

From 8a7706d950bf43436f1cc733cc61ea3d2dafd06e Mon Sep 17 00:00:00 2001
From: David Slusser <david.slusser@sealstorage.io>
Date: Sat, 30 Dec 2023 15:33:00 -0800
Subject: [PATCH 3/4] code quaility

---
 .github/workflows/safety.yaml             |  2 +-
 src/djangoaddicts/hostutils/views/gui.py  | 23 ++++++++------------
 src/djangoaddicts/hostutils/views/htmx.py | 26 ++++++++++-------------
 3 files changed, 21 insertions(+), 30 deletions(-)

diff --git a/.github/workflows/safety.yaml b/.github/workflows/safety.yaml
index 0a47d7b..d1001ca 100644
--- a/.github/workflows/safety.yaml
+++ b/.github/workflows/safety.yaml
@@ -5,7 +5,7 @@ jobs:
     runs-on: ubuntu-latest
     name: "safety"
     steps:
-      - uses: davidslusser/actions_python_safety@v1.0.0
+      - uses: davidslusser/actions_python_safety@separate_pip
         with:
           options: "--debug"
           pip_install_command: "pip install -e .[dev]"
diff --git a/src/djangoaddicts/hostutils/views/gui.py b/src/djangoaddicts/hostutils/views/gui.py
index 1311342..3520d86 100644
--- a/src/djangoaddicts/hostutils/views/gui.py
+++ b/src/djangoaddicts/hostutils/views/gui.py
@@ -6,7 +6,6 @@
 from django.views.generic import View
 
 # import forms
-from djangoaddicts.hostutils.forms import HostProcessFilterForm
 
 
 class ShowHost(View):
@@ -139,23 +138,19 @@ def get(self, request, *args, **kwargs):
         context["title"] = self.title
         context["now"] = datetime.datetime.now()
         context["subtitle"] = psutil.os.uname()[1]
-        counts = {
-            "running": 0,
-            "sleeping": 0,
-            "idle": 0,
-            "stopped": 0,
-            "zombie": 0,
-            "dead": 0,
-            "disk-sleep": 0
-        }
+        counts = {"running": 0, "sleeping": 0, "idle": 0, "stopped": 0, "zombie": 0, "dead": 0, "disk-sleep": 0}
         process_list = []
         for process in psutil.process_iter():
             try:
                 counts[process.status()] += 1
-                process_list.append({"pid": process.pid, 
-                                     "name": process.name(), 
-                                     "status": process.status(), 
-                                     "started_at": process.create_time()})
+                process_list.append(
+                    {
+                        "pid": process.pid,
+                        "name": process.name(),
+                        "status": process.status(),
+                        "started_at": process.create_time(),
+                    }
+                )
             except (psutil.NoSuchProcess, psutil.AccessDenied):
                 pass
         context["counts"] = counts
diff --git a/src/djangoaddicts/hostutils/views/htmx.py b/src/djangoaddicts/hostutils/views/htmx.py
index ec24f56..69120f9 100644
--- a/src/djangoaddicts/hostutils/views/htmx.py
+++ b/src/djangoaddicts/hostutils/views/htmx.py
@@ -77,24 +77,20 @@ def get(self, request):
         """Get host prcesses"""
         context = {}
         filter_form = HostProcessFilterForm(request.GET or None)
-        counts = {
-            "running": 0,
-            "sleeping": 0,
-            "idle": 0,
-            "stopped": 0,
-            "zombie": 0,
-            "dead": 0,
-            "disk-sleep": 0
-        }
-        
+        counts = {"running": 0, "sleeping": 0, "idle": 0, "stopped": 0, "zombie": 0, "dead": 0, "disk-sleep": 0}
+
         process_list = []
         for process in psutil.process_iter():
             try:
                 counts[process.status()] += 1
-                process_list.append({"pid": process.pid, 
-                                     "name": process.name(), 
-                                     "status": process.status(), 
-                                     "started_at": process.create_time()})
+                process_list.append(
+                    {
+                        "pid": process.pid,
+                        "name": process.name(),
+                        "status": process.status(),
+                        "started_at": process.create_time(),
+                    }
+                )
             except (psutil.NoSuchProcess, psutil.AccessDenied):
                 continue
         context["counts"] = counts
@@ -143,7 +139,7 @@ def get(self, request):
         context["filter_form"] = filter_form
         return render(request, template_name="hostutils/bs5/snippets/host_process_card_swap.htm", context=context)
 
-        
+
 class GetHostProcessStats(BuildBootstrapModalView):
     """Get statistics for a given process"""
 

From aba30d18343a36a1927fcddce6a117766b060af2 Mon Sep 17 00:00:00 2001
From: David Slusser <david.slusser@sealstorage.io>
Date: Sat, 30 Dec 2023 16:17:49 -0800
Subject: [PATCH 4/4] restoring safety

---
 .github/workflows/safety.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/safety.yaml b/.github/workflows/safety.yaml
index d1001ca..0a47d7b 100644
--- a/.github/workflows/safety.yaml
+++ b/.github/workflows/safety.yaml
@@ -5,7 +5,7 @@ jobs:
     runs-on: ubuntu-latest
     name: "safety"
     steps:
-      - uses: davidslusser/actions_python_safety@separate_pip
+      - uses: davidslusser/actions_python_safety@v1.0.0
         with:
           options: "--debug"
           pip_install_command: "pip install -e .[dev]"