fix: add error handling to middleware to prevent INVOCATION_FAILED
- Check if Supabase env vars are configured before using them - Wrap Supabase auth call in try/catch to prevent 500 crashes - Graceful fallback: pass through on error instead of crashing https://claude.ai/code/session_01H2aRGDaKgarPvhay2HxN6Y
This commit is contained in:
@@ -2,14 +2,37 @@ import { createServerClient } from "@supabase/ssr";
|
|||||||
import { NextResponse, type NextRequest } from "next/server";
|
import { NextResponse, type NextRequest } from "next/server";
|
||||||
|
|
||||||
export async function middleware(request: NextRequest) {
|
export async function middleware(request: NextRequest) {
|
||||||
|
// Si les env vars Supabase ne sont pas configurees, laisser passer
|
||||||
|
const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL;
|
||||||
|
const supabaseAnonKey = process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY;
|
||||||
|
|
||||||
|
if (
|
||||||
|
!supabaseUrl ||
|
||||||
|
!supabaseAnonKey ||
|
||||||
|
supabaseUrl.includes("your-project") ||
|
||||||
|
supabaseAnonKey === "your-anon-key"
|
||||||
|
) {
|
||||||
|
// Pas de Supabase configure : rediriger les routes protegees vers login
|
||||||
|
const protectedPaths = ["/dashboard", "/formations", "/profil"];
|
||||||
|
const isProtected = protectedPaths.some((p) =>
|
||||||
|
request.nextUrl.pathname.startsWith(p)
|
||||||
|
);
|
||||||
|
|
||||||
|
if (isProtected) {
|
||||||
|
const url = request.nextUrl.clone();
|
||||||
|
url.pathname = "/login";
|
||||||
|
return NextResponse.redirect(url);
|
||||||
|
}
|
||||||
|
|
||||||
|
return NextResponse.next();
|
||||||
|
}
|
||||||
|
|
||||||
|
try {
|
||||||
let supabaseResponse = NextResponse.next({
|
let supabaseResponse = NextResponse.next({
|
||||||
request,
|
request,
|
||||||
});
|
});
|
||||||
|
|
||||||
const supabase = createServerClient(
|
const supabase = createServerClient(supabaseUrl, supabaseAnonKey, {
|
||||||
process.env.NEXT_PUBLIC_SUPABASE_URL!,
|
|
||||||
process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY!,
|
|
||||||
{
|
|
||||||
cookies: {
|
cookies: {
|
||||||
getAll() {
|
getAll() {
|
||||||
return request.cookies.getAll();
|
return request.cookies.getAll();
|
||||||
@@ -26,8 +49,7 @@ export async function middleware(request: NextRequest) {
|
|||||||
);
|
);
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
}
|
});
|
||||||
);
|
|
||||||
|
|
||||||
const {
|
const {
|
||||||
data: { user },
|
data: { user },
|
||||||
@@ -57,6 +79,11 @@ export async function middleware(request: NextRequest) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
return supabaseResponse;
|
return supabaseResponse;
|
||||||
|
} catch (e) {
|
||||||
|
console.error("Middleware error:", e);
|
||||||
|
// En cas d'erreur, laisser passer la requete plutot que crasher
|
||||||
|
return NextResponse.next();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
export const config = {
|
export const config = {
|
||||||
|
|||||||
Reference in New Issue
Block a user