Make rate limiting fail-open on connection errors

Upstash Redis fetch failures were causing registration (and potentially
other endpoints) to return 500 errors. Rate limiting now catches
connection errors and allows the request through rather than blocking it.

Author: onamfc

src/app/api/auth/register/route.ts

@@ -88,9 +88,8 @@ export async function POST(request: NextRequest) {
     );
   } catch (error) {
     dev.error("Registration error:", error, {tag: "auth"});
-    const message = error instanceof Error ? error.message : String(error);
     return NextResponse.json(
-      { error: "An error occurred during registration", debug: message },
+      { error: "An error occurred during registration" },
       { status: 500 },
     );
   }

src/lib/rate-limit.ts

@@ -95,14 +95,19 @@ export async function checkRateLimit(
     return { success: true };
   }
 
-  const result = await limiter.limit(identifier);
-
-  return {
-    success: result.success,
-    limit: result.limit,
-    remaining: result.remaining,
-    reset: result.reset,
-  };
+  try {
+    const result = await limiter.limit(identifier);
+
+    return {
+      success: result.success,
+      limit: result.limit,
+      remaining: result.remaining,
+      reset: result.reset,
+    };
+  } catch (error) {
+    console.error("Rate limiting error (failing open):", error);
+    return { success: true };
+  }
 }
 
 /**