You've already forked isop-mirror
fix: prevent non-admin users from managing API keys
This commit is contained in:
24
backend/app/Http/Middleware/AdministratorOnly.php
Normal file
24
backend/app/Http/Middleware/AdministratorOnly.php
Normal file
@@ -0,0 +1,24 @@
|
||||
<?php
|
||||
|
||||
namespace App\Http\Middleware;
|
||||
|
||||
use Closure;
|
||||
use Illuminate\Http\Request;
|
||||
use Symfony\Component\HttpFoundation\Response;
|
||||
|
||||
class AdministratorOnly
|
||||
{
|
||||
/**
|
||||
* Handle an incoming request.
|
||||
*
|
||||
* @param \Closure(\Illuminate\Http\Request): (\Symfony\Component\HttpFoundation\Response) $next
|
||||
*/
|
||||
public function handle(Request $request, Closure $next): Response
|
||||
{
|
||||
if ($request->user()->role !== 'ADMIN') {
|
||||
return response(status: 403);
|
||||
}
|
||||
|
||||
return $next($request);
|
||||
}
|
||||
}
|
||||
@@ -6,6 +6,7 @@ use App\Http\Controllers\ExternalApiController;
|
||||
use App\Http\Controllers\InternshipController;
|
||||
use App\Http\Controllers\StudentDataController;
|
||||
use App\Http\Controllers\InternshipStatusDataController;
|
||||
use App\Http\Middleware\AdministratorOnly;
|
||||
use App\Models\Company;
|
||||
use App\Models\StudentData;
|
||||
use Illuminate\Http\Request;
|
||||
@@ -66,8 +67,8 @@ Route::prefix('/companies')->middleware("auth:sanctum")->group(function () {
|
||||
Route::delete("/{id}", [CompanyController::class, 'delete']);
|
||||
});
|
||||
|
||||
Route::prefix('/external')->middleware("auth:sanctum")->group(function () {
|
||||
Route::prefix('/keys')->group(function () {
|
||||
Route::prefix('/external')->group(function () {
|
||||
Route::prefix('/keys')->middleware(['auth:sanctum', AdministratorOnly::class])->group(function () {
|
||||
Route::get("/", [ExternalApiController::class, 'all_keys'])->name("api.external.keys.create");
|
||||
Route::put("/", [ExternalApiController::class, 'create_key'])->name("api.external.keys.list");
|
||||
Route::delete("/{id}", [ExternalApiController::class, 'destroy_key'])->name("api.external.keys.delete");
|
||||
|
||||
Reference in New Issue
Block a user