Merge pull request #1362 from benphelps/feature/ping-4xx

Accept http 401 / 403 for ping

src/components/services/ping.jsx

@@ -25,7 +25,7 @@ export default function Ping({ service }) {
 
   const statusText = `${service.ping}: HTTP status ${data.status}`;
   
-  if (data && data.status !== 200) {
+  if (data.status > 403) {
     return (
       <div className="w-auto px-1.5 py-0.5 text-center bg-theme-500/10 dark:bg-theme-900/50 rounded-b-[3px] overflow-hidden" title={statusText}>
         <div className="text-[8px] font-bold text-rose-500/80">{data.status}</div>
@@ -33,12 +33,10 @@ export default function Ping({ service }) {
     );
   }
   
-  if (data && data.status === 200) {
+  return (
-    return (
+    <div className="w-auto px-1.5 py-0.5 text-center bg-theme-500/10 dark:bg-theme-900/50 rounded-b-[3px] overflow-hidden" title={statusText}>
-      <div className="w-auto px-1.5 py-0.5 text-center bg-theme-500/10 dark:bg-theme-900/50 rounded-b-[3px] overflow-hidden" title={statusText}>
+      <div className="text-[8px] font-bold text-emerald-500/80">{t("common.ms", { value: data.latency, style: "unit", unit: "millisecond", unitDisplay: "narrow", maximumFractionDigits: 0 })}</div>
-        <div className="text-[8px] font-bold text-emerald-500/80">{t("common.ms", { value: data.latency, style: "unit", unit: "millisecond", unitDisplay: "narrow", maximumFractionDigits: 0 })}</div>
+    </div>
-      </div>
+  );
-    );
-  }
 
 }

src/pages/api/ping.js

@@ -21,7 +21,7 @@ export default async function handler(req, res) {
     });
     let endTime = performance.now();
     
-    if (status >= 400) {
+    if (status > 403) {
       // try one more time as a GET in case HEAD is rejected for whatever reason
       startTime = performance.now();
       [status] = await httpProxy(pingURL);