Merge pull request #54 from Onemind-Services-LLC/OMS-120-Allowed-Host…

…-field-error

OMS-120 fixed allowed host parsing issue

netbox_paas/constants.py

@@ -4,6 +4,7 @@
 from extras.constants import DEFAULT_DASHBOARD
 from utilities.forms import BOOLEAN_WITH_BLANK_CHOICES
 from .nb_settings import *
+from .validators import validate_comma_separated
 
 JELASTIC_API = "https://app.xapp.cloudmydc.com"
 NETBOX_JPS_REPO = "https://raw.githubusercontent.com/Onemind-Services-LLC/netbox-jps/master"
@@ -33,8 +34,8 @@
                     label="Allowed Hosts",
                     help_text="Comma separated list of allowed hosts (FQDN, IP address, or pattern), or '*' for all",
                     placeholder="netbox.example.com, localhost, *",
-                    field=SimpleArrayField,
-                    field_kwargs={'base_field': forms.CharField()},
+                    field=forms.CharField,
+                    field_kwargs={"validators": [validate_comma_separated]},
                 ),
                 Param(
                     key="DB_CONN_MAX_AGE",

netbox_paas/validators.py

@@ -0,0 +1,13 @@
+from django.core.exceptions import ValidationError
+
+__all__ = ('validate_comma_separated',)
+
+
+def validate_comma_separated(value):
+    if not isinstance(value, str):
+        raise ValidationError("This field requires a string of comma-separated values.")
+    values = value.split(',')
+    if any(not item.strip() for item in values):
+        raise ValidationError("Each value should be non-empty and comma-separated without extra spaces.")
+
+    return value