Clean number of hours in error log search form

changelog.d/3246.fixed.md

@@ -0,0 +1 @@
+Check hours in radius error log search

python/nav/web/radius/forms.py

@@ -167,6 +167,17 @@ def __init__(self, *args, **kwargs):
             ],
         )
 
+    def clean_time(self):
+        time_type, time = self.cleaned_data["time"]
+        if time_type == "hours":
+            try:
+                datetime.now() - timedelta(hours=int(time))
+            except OverflowError:
+                raise forms.ValidationError(
+                    "They did not have computers %s hours ago" % time
+                )
+        return time
+
 
 class AccountLogSearchForm(forms.Form):
     """Form for searching in the radius account log"""

tests/integration/web/radius_test.py

@@ -11,3 +11,15 @@ def test_top_talkers_should_not_crash_on_big_days(db, client):
 
     assert response.status_code == 200
     assert 'They did not have computers' in smart_str(response.content)
+
+
+def test_error_log_search_should_not_crash_on_big_hours(db, client, admin_username):
+    url = reverse('radius-log_search')
+    url = (
+        url
+        + f'?query_0=username&query_1={admin_username}&log_entry_type=&time_0=hours&time_1=123123123123123&send=Search'
+    )
+    response = client.get(url)
+
+    assert response.status_code == 200
+    assert 'They did not have computers' in smart_str(response.content)