fix: admin/internet: unwanted error displayed in HTTPS box when no ce…

…rtificate is provided
HameemDakheel · Dec 10, 2023 · 8b3c71a · 8b3c71a
1 parent 3ee57c3
commit 8b3c71a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/admin/src/InternetPage.ts b/admin/src/InternetPage.ts
@@ -102,7 +102,7 @@ export default function InternetPage() {
             isCertError(error) && h(Alert, { severity: 'warning' }, error),
             prefix("Error: ", error)
                 || disabled && h(LinkBtn, { onClick: notEnabled }, "Not enabled"),
-            cert.element || with_(cert.data, c => h(Box, {},
+            cert.element || with_(cert.data, c => c.none ? "No certificate configured" : h(Box, {},
                 h(CardMembership, { fontSize: 'small', sx: { mr: 1, verticalAlign: 'middle' } }), "Current certificate",
                 h('ul', {},
                     h('li', {}, "Domain: ", c.altNames?.join(' + ') ||'-'),

diff --git a/src/api.net.ts b/src/api.net.ts
@@ -79,7 +79,7 @@ const apis: ApiHandlers = {
     },
 
     get_cert() {
-        return getCertObject() || new ApiError(HTTP_NOT_FOUND)
+        return getCertObject() || { none: true }
     }
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -79,7 +79,7 @@ const apis: ApiHandlers = { @@
         },
         get_cert() {
-            return getCertObject() || new ApiError(HTTP_NOT_FOUND)
+            return getCertObject() || { none: true }
         }
     }
@@ Expand Down @@