small fixes

Author: rshewitt

app/templates/view_validators.html

@@ -53,20 +53,18 @@ <h2>Validation Results</h2>
         {% if not data.record_errors %}
             <p>No validation errors found</p>
         {% else %}
-        <div class="usa-table-container" tabindex="0">
-            <div class="error-list">
-                {% for record_error in data.record_errors %}
-                <div class="error-block">
-                    <p style="word-break: break-all"><strong>Identifier:</strong> {{ record_error[0] ~ " (dataset position)" if record_error[0] is number else record_error[0] }}</p>
-                    <p>
-                        <strong>Error Message:</strong> 
-                        <ul>
-                            <li>{{ record_error[1] }}</li>
-                        </ul>
-                    </p>
-                </div>
-                {% endfor %}
+        <div class="error-list">
+            {% for record_error in data.record_errors %}
+            <div class="error-block">
+                <p style="word-break: break-all"><strong>Identifier:</strong> {{ record_error[0] ~ " (dataset position)" if record_error[0] is number else record_error[0] }}</p>
+                <p>
+                    <strong>Error Message:</strong> 
+                    <ul>
+                        <li>{{ record_error[1] }}</li>
+                    </ul>
+                </p>
             </div>
+            {% endfor %}
         </div>
         {% endif %}
     {% endif %}
@@ -76,7 +74,7 @@ <h2>Validation Results</h2>
 
 {% block scripts %}
 
-<script>
+<script nonce="{{ csp_nonce() }}">
 function toggleInputFields() {
     const method = document.getElementById("fetch_method").value;
 

app/util.py

@@ -99,7 +99,7 @@ def is_public_ip(hostname: str) -> bool:
 
 def fetch_json_from_url(url: str) -> dict:
 
-    MAX_CONTENT_LENGTH = 10 * 1024 * 1024  # 10MB limit
+    max_content_length = 10 * 1024 * 1024  # 10MB limit
 
     parsed = urlparse(url)
 
@@ -123,8 +123,8 @@ def fetch_json_from_url(url: str) -> dict:
     if "application/json" not in content_type:
         raise ValueError("URL did not return JSON.")
 
-    content = response.raw.read(MAX_CONTENT_LENGTH + 1)
-    if len(content) > MAX_CONTENT_LENGTH:
+    content = response.raw.read(max_content_length + 1)
+    if len(content) > max_content_length:
         raise ValueError("JSON payload too large.")
 
     return response.json()

tests/playwright/test_validator.py

@@ -37,7 +37,7 @@ def test_ui_validate_by_url(self, upage):
         upage.locator("input[type=submit]").click()
 
         # error table should be visible and with 5 validation errors
-        expect(upage.locator(".usa-table-container")).to_be_visible()
+        expect(upage.locator(".error-list")).to_be_visible()
         expect(upage.locator(".error-block")).to_have_count(5)
 
     def test_ui_validate_by_json(self, upage, dcatus_long_description_json):
@@ -53,7 +53,7 @@ def test_ui_validate_by_json(self, upage, dcatus_long_description_json):
         upage.locator("input[type=submit]").click()
 
         # error table should be visible and with 5 validation errors
-        expect(upage.locator(".usa-table-container")).to_be_visible()
+        expect(upage.locator(".error-list")).to_be_visible()
         expect(upage.locator(".error-block")).to_have_count(1)
 
     def test_api_validate_by_url(self, upage, validator_api_url):