Fix timeZone errors, refine config fetching

Signed-off-by: Michael Edgar <medgar@redhat.com>
Assisted-by: Claude Code (Sonnet 4.5) <noreply@anthropic.com>
Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: MikeEdgar

ui/app/[locale]/NextIntlProvider.tsx

@@ -1,6 +1,6 @@
 "use client";
 import { AbstractIntlMessages, NextIntlClientProvider } from "next-intl";
-import { ReactNode } from "react";
+import { ReactNode, useState, useEffect } from "react";
 
 type Props = {
   messages: AbstractIntlMessages;
@@ -12,7 +12,14 @@ export default function NextIntlProvider({
   locale,
   children,
 }: Props) {
-  const timeZone = Intl.DateTimeFormat().resolvedOptions().timeZone;
+  // Use UTC as default for SSR to avoid hydration mismatch
+  // Client will update to actual timezone after mount
+  const [timeZone, setTimeZone] = useState<string>("UTC");
+
+  useEffect(() => {
+    setTimeZone(Intl.DateTimeFormat().resolvedOptions().timeZone);
+  }, []);
+
   return (
     <NextIntlClientProvider
       locale={locale}

ui/middleware.ts

@@ -28,14 +28,14 @@ const authMiddleware = withAuth(
   },
 ) as any;
 
-const publicPathnameRegex = RegExp(
+const publicPathnameRegex = new RegExp(
   `^(/(${locales.join("|")}))?(${publicPages
     .flatMap((p) => (p === "/" ? ["", "/"] : p))
     .join("|")})/?$`,
   "i",
 );
 
-const protectedPathnameRegex = RegExp(
+const protectedPathnameRegex = new RegExp(
   `^(/(${locales.join("|")}))?(${protectedPages
     .flatMap((p) => (p === "/" ? ["", "/"] : p))
     .join("|")})/?$`,
@@ -47,11 +47,19 @@ export default async function middleware(req: NextRequest) {
    * Next.js middleware doesn't support reading files, so here we make a (cached)
    * call to the /config endpoint within the same application :(
    */
-  let oidcEnabled = await fetch(`http://127.0.0.1:${process.env.PORT}/config`, {
+  const configUrl = `http://127.0.0.1:${process.env.PORT || '3000'}/config`;
+  log.info({ configUrl }, "Fetching OIDC configuration");
+
+  let oidcEnabled = await fetch(configUrl, {
     cache: "force-cache",
+    signal: AbortSignal.timeout(1000), // 1 second timeout to prevent hanging
   })
     .then((cfg) => cfg.json())
-    .then((cfg) => cfg["oidc"]);
+    .then((cfg) => cfg["oidc"])
+    .catch((err) => {
+      log.warn({ err, configUrl }, "Failed to fetch OIDC config, defaulting to false");
+      return false;
+    });
 
   const searchParams = req.nextUrl.searchParams;
   const requestPath = req.nextUrl.pathname;

ui/package.json

@@ -4,7 +4,7 @@
   "license": "Apache-2.0",
   "private": true,
   "scripts": {
-    "dev": "NODE_OPTIONS='--inspect --require next-logger' next dev | pino-pretty",
+    "dev": "NODE_OPTIONS='--require next-logger' next dev | pino-pretty",
     "build": "next build",
     "start": "NODE_OPTIONS='--require next-logger' next start",
     "lint": "next lint",