feat: add API key auth middleware for /api/* routes (task 3.1)
- Create src/middleware.ts with Next.js middleware - Reads API_KEY env var and checks X-API-Key header on all /api/* routes - Skips auth for /api/health endpoint - Fails open (with warning) when API_KEY is not configured - Returns 401 Unauthorized when key is missing or mismatched - Mark task 3.1 as complete in tasks.md Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
0cd21887e4
commit
577bb2e56e
2 changed files with 34 additions and 1 deletions
|
|
@ -19,7 +19,7 @@
|
|||
|
||||
## 3. Authentication
|
||||
|
||||
- [ ] 3.1 `[sonnet]` Create `src/middleware.ts` with API key auth middleware: read `API_KEY` env var, check `X-API-Key` header on all `/api/*` routes except `/api/health`, return 401 if invalid
|
||||
- [x] 3.1 `[sonnet]` Create `src/middleware.ts` with API key auth middleware: read `API_KEY` env var, check `X-API-Key` header on all `/api/*` routes except `/api/health`, return 401 if invalid
|
||||
- [ ] 3.2 `[sonnet]` Add FastAPI `Depends()` API key dependency in `services/ml/app/main.py`: read `API_KEY` env var, check `X-API-Key` header, exempt `/health` endpoint
|
||||
- [ ] 3.3 `[sonnet]` Update all Next.js proxy routes to forward `X-API-Key` header to ML service
|
||||
- [ ] 3.4 `[haiku]` Add `API_KEY` to `.env.example` with placeholder value and instructions
|
||||
|
|
|
|||
33
src/middleware.ts
Normal file
33
src/middleware.ts
Normal file
|
|
@ -0,0 +1,33 @@
|
|||
import { NextResponse } from "next/server";
|
||||
import type { NextRequest } from "next/server";
|
||||
|
||||
export function middleware(request: NextRequest) {
|
||||
const { pathname } = request.nextUrl;
|
||||
|
||||
// Skip auth check for the health endpoint
|
||||
if (pathname === "/api/health") {
|
||||
return NextResponse.next();
|
||||
}
|
||||
|
||||
const apiKey = process.env.API_KEY;
|
||||
|
||||
// If API_KEY is not configured, skip auth check (fail-open for development)
|
||||
if (!apiKey) {
|
||||
console.warn(
|
||||
"Warning: API_KEY environment variable is not set. API authentication is disabled."
|
||||
);
|
||||
return NextResponse.next();
|
||||
}
|
||||
|
||||
const requestApiKey = request.headers.get("X-API-Key");
|
||||
|
||||
if (!requestApiKey || requestApiKey !== apiKey) {
|
||||
return NextResponse.json({ error: "Unauthorized" }, { status: 401 });
|
||||
}
|
||||
|
||||
return NextResponse.next();
|
||||
}
|
||||
|
||||
export const config = {
|
||||
matcher: ["/api/:path*"],
|
||||
};
|
||||
Loading…
Add table
Add a link
Reference in a new issue