Skip to content

Commit f5b416b

Browse files
Merge pull request #1 from achmadhadikurnia/main
refactor: enforce API key via header only, remove query parameter check
2 parents e2e6ed1 + bf81e58 commit f5b416b

1 file changed

Lines changed: 4 additions & 6 deletions

File tree

app/Http/Middleware/ApiKeyMiddleware.php

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,7 @@ class ApiKeyMiddleware
1010
/**
1111
* Handle an incoming request.
1212
*
13-
* Validates the API key from either the X-Api-Key header
14-
* or the api_key query parameter.
13+
* Validates the API key from the X-Api-Key header.
1514
*
1615
* @param Request $request
1716
* @param Closure $next
@@ -26,14 +25,13 @@ public function handle(Request $request, Closure $next)
2625
return $next($request);
2726
}
2827

29-
// Check header first, then query parameter
30-
$providedKey = $request->header('X-Api-Key')
31-
?? $request->query('api_key');
28+
// Check header only
29+
$providedKey = $request->header('X-Api-Key');
3230

3331
if (empty($providedKey) || $providedKey !== $configuredKey) {
3432
return response()->json([
3533
'error' => 'Unauthorized',
36-
'message' => 'Invalid or missing API key. Provide via X-Api-Key header or api_key query parameter.',
34+
'message' => 'Invalid or missing API key. Provide via X-Api-Key header.',
3735
], 401);
3836
}
3937

0 commit comments

Comments
 (0)