chore: fix deployment eval error and add middleware safety checks
This commit is contained in:
@@ -1,6 +1,12 @@
|
||||
import { createClient } from '@supabase/supabase-js';
|
||||
|
||||
const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL!;
|
||||
const supabaseAnonKey = process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY!;
|
||||
const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL;
|
||||
const supabaseAnonKey = process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY;
|
||||
|
||||
export const supabase = createClient(supabaseUrl, supabaseAnonKey);
|
||||
if (!supabaseUrl || !supabaseAnonKey) {
|
||||
console.error('Migration Error: Supabase credentials are not set.');
|
||||
}
|
||||
|
||||
export const supabase = (supabaseUrl && supabaseAnonKey)
|
||||
? createClient(supabaseUrl, supabaseAnonKey)
|
||||
: null as any; // Fallback or handle null in services
|
||||
|
||||
@@ -4,10 +4,20 @@ import type { NextRequest } from 'next/server';
|
||||
|
||||
export async function middleware(req: NextRequest) {
|
||||
const res = NextResponse.next();
|
||||
const supabase = createMiddlewareClient({ req, res });
|
||||
|
||||
// Refresh session if expired - required for Server Components/Actions
|
||||
await supabase.auth.getSession();
|
||||
// Safety check for environment variables in production
|
||||
if (!process.env.NEXT_PUBLIC_SUPABASE_URL || !process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY) {
|
||||
console.warn('Deployment: NEXT_PUBLIC_SUPABASE environment variables are missing.');
|
||||
return res;
|
||||
}
|
||||
|
||||
try {
|
||||
const supabase = createMiddlewareClient({ req, res });
|
||||
// Refresh session if expired - required for Server Components/Actions
|
||||
await supabase.auth.getSession();
|
||||
} catch (err) {
|
||||
console.error('Middleware Error:', err);
|
||||
}
|
||||
|
||||
return res;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user