♻️ Adjust the polling interval to 30s for valid certs

src/hooks/useCertificate.ts

@@ -28,6 +28,8 @@ type UseCertificateReturn = {
 };
 
 const POLL_JITTER_MS = durationToMs(1, "seconds");
+/** A valid cert isn't going anywhere soon, so check on it far less often. */
+const VALID_POLL_INTERVAL_MS = durationToMs(30, "seconds");
 
 export function useCertificate({
     orgId,
@@ -35,7 +37,7 @@ export function useCertificate({
     fullDomain,
     initialCertValue,
     polling = false,
-    pollingInterval = 5000
+    pollingInterval = durationToMs(5, "seconds")
 }: UseCertificateProps): UseCertificateReturn {
     const api = createApiClient(useEnvContext());
     const queryClient = useQueryClient();
@@ -49,19 +51,19 @@ export function useCertificate({
     const { data, isLoading, isError, refetch } = useQuery({
         ...certQuery,
         enabled: Boolean(orgId && domainId && fullDomain),
-        // Add a small random offset to each tick. Without it, every row in a
+        refetchInterval: (query) => {
-        // table would poll at the exact same instant, hitting the server in
+            if (!polling) return false;
-        // bursts instead of spreading the requests out.
+            const interval =
-        refetchInterval: polling
+                query.state.data?.status === "valid"
-            ? () =>
+                    ? VALID_POLL_INTERVAL_MS
-                  Math.max(
+                    : pollingInterval;
-                      1000,
+            // Add a small random offset to each tick. Without it, every row in
-                      Math.round(
+            // a table would poll at the exact same instant, hitting the server
-                          pollingInterval +
+            // in bursts instead of spreading the requests out.
-                              (Math.random() * 2 - 1) * POLL_JITTER_MS
+            // the requests will be jittered by less or more than 1s
-                      )
+            const jitter = (Math.random() * 2 - 1) * POLL_JITTER_MS;
-                  )
+            return Math.max(1000, interval + jitter);
-            : false,
+        },
         // An explicit `null` is a real answer ("this domain has no certificate")
         // and is seeded like any other, so a server-prefetched row doesn't
         // refetch on mount. Only an omitted prop leaves the cache empty.