refactor(auth): consolidate to single cookie post single-SPA

The dual-cookie machinery (crewli_app_token + crewli_portal_token,
Origin-based resolution) was load-bearing only when the second SPA
existed. apps/portal/ was deleted in WS-3 PR-B1; the resolver code
has been carrying dead branches since then. Collapse to one cookie.

Cookie name retained as crewli_app_token — no session breakage on
deploy. crewli_portal_token is fully purged from the server-side.

CookieBearerToken middleware:
- COOKIE_NAMES array → single COOKIE_NAME constant
- resolveCookieName method (Origin/Referer parsing, host+port
  matching against frontend_app_url/frontend_portal_url) → removed
- Body collapses to: skip if Authorization header present; else
  read crewli_app_token cookie and inject Bearer header

SetAuthCookie trait:
- COOKIE_MAP / resolveCookieName / originMatches → removed
- makeAuthCookie / forgetAuthCookie now take only $token; the
  cookie name is the trait's private constant

Five callers updated to drop the resolveCookieName($request) line
and the cookie-name argument: LoginController (3 sites),
MfaVerifyController (1 site), AuthRefreshController (1 site),
LogoutController (1 site), InvitationController (1 site — caller
list in the prompt missed this one but the same pattern applies).

frontend_portal_url config key retained (per Phase A directive Q1):
EmailChangeController, PasswordResetController, PersonController are
non-auth consumers that build per-app URL maps for outbound emails.
The map structure is now functionally redundant (production resolves
all FRONTEND_* env vars to the same host) but stays structurally
intact. Refactor tracked as TECH-FRONTEND-URL-CONSOLIDATE in the
upcoming docs commit.

HttpOnlyCookieAuthTest:
- Removed 4 dual-cookie tests (login_sets_portal_cookie_for_portal_origin,
  app_cookie_does_not_authenticate_portal_requests,
  portal_cookie_does_not_authenticate_app_requests,
  correct_cookie_authenticates_with_matching_origin)
- Renamed login_sets_app_cookie_for_unknown_origin →
  login_sets_app_cookie_regardless_of_origin; expanded to four
  Origin variants (none, app, unknown, foreign) — pins the new
  origin-agnostic contract
- Removed Origin headers from request calls in remaining tests
  (now meaningless)

Backend test count: 1491 → 1487 (-4 deleted, dual-cookie tests
encoding the obsolete contract). Pint clean. Larastan clean.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
2026-05-06 00:24:01 +02:00
parent 96cb1519de
commit 2e94a107e4
8 changed files with 49 additions and 185 deletions

View File

@@ -4,12 +4,12 @@ declare(strict_types=1);
namespace App\Http\Controllers\Api\V1\Auth; namespace App\Http\Controllers\Api\V1\Auth;
use App\Enums\MfaMethod;
use App\Http\Controllers\Api\V1\Traits\SetAuthCookie; use App\Http\Controllers\Api\V1\Traits\SetAuthCookie;
use App\Http\Controllers\Controller; use App\Http\Controllers\Controller;
use App\Http\Requests\Api\V1\Auth\MfaEmailSendRequest; use App\Http\Requests\Api\V1\Auth\MfaEmailSendRequest;
use App\Http\Requests\Api\V1\Auth\MfaVerifyRequest; use App\Http\Requests\Api\V1\Auth\MfaVerifyRequest;
use App\Http\Resources\Api\V1\MeResource; use App\Http\Resources\Api\V1\MeResource;
use App\Enums\MfaMethod;
use App\Models\User; use App\Models\User;
use App\Services\MfaService; use App\Services\MfaService;
use Illuminate\Http\JsonResponse; use Illuminate\Http\JsonResponse;
@@ -58,19 +58,18 @@ final class MfaVerifyController extends Controller
]); ]);
$token = $user->createToken('auth-token')->plainTextToken; $token = $user->createToken('auth-token')->plainTextToken;
$cookieName = $this->resolveCookieName($request);
return $this->success([ return $this->success([
'user' => new MeResource($user), 'user' => new MeResource($user),
], 'MFA verification successful') ], 'MFA verification successful')
->withCookie($this->makeAuthCookie($cookieName, $token)); ->withCookie($this->makeAuthCookie($token));
} }
public function sendEmailCode(MfaEmailSendRequest $request): JsonResponse public function sendEmailCode(MfaEmailSendRequest $request): JsonResponse
{ {
$sessionToken = $request->validated('mfa_session_token'); $sessionToken = $request->validated('mfa_session_token');
$cacheKey = 'mfa_session:' . $sessionToken; $cacheKey = 'mfa_session:'.$sessionToken;
$session = Cache::get($cacheKey); $session = Cache::get($cacheKey);
if (! $session) { if (! $session) {

View File

@@ -24,7 +24,6 @@ final class AuthRefreshController extends Controller
// Create a new token // Create a new token
$newToken = $user->createToken('auth-token')->plainTextToken; $newToken = $user->createToken('auth-token')->plainTextToken;
$cookieName = $this->resolveCookieName($request);
$user->load(['organisations', 'roles', 'permissions']); $user->load(['organisations', 'roles', 'permissions']);
@@ -34,6 +33,6 @@ final class AuthRefreshController extends Controller
]); ]);
return $this->success(new MeResource($user), 'Token refreshed') return $this->success(new MeResource($user), 'Token refreshed')
->withCookie($this->makeAuthCookie($cookieName, $newToken)); ->withCookie($this->makeAuthCookie($newToken));
} }
} }

View File

@@ -18,6 +18,7 @@ use Illuminate\Support\Facades\Gate;
final class InvitationController extends Controller final class InvitationController extends Controller
{ {
use SetAuthCookie; use SetAuthCookie;
public function __construct( public function __construct(
private readonly InvitationService $invitationService, private readonly InvitationService $invitationService,
) {} ) {}
@@ -65,7 +66,6 @@ final class InvitationController extends Controller
); );
$sanctumToken = $user->createToken('auth-token')->plainTextToken; $sanctumToken = $user->createToken('auth-token')->plainTextToken;
$cookieName = $this->resolveCookieName($request);
return $this->success([ return $this->success([
'user' => [ 'user' => [
@@ -76,7 +76,7 @@ final class InvitationController extends Controller
'email' => $user->email, 'email' => $user->email,
], ],
], 'Uitnodiging geaccepteerd') ], 'Uitnodiging geaccepteerd')
->withCookie($this->makeAuthCookie($cookieName, $sanctumToken)); ->withCookie($this->makeAuthCookie($sanctumToken));
} }
public function revoke(Organisation $organisation, UserInvitation $invitation): JsonResponse public function revoke(Organisation $organisation, UserInvitation $invitation): JsonResponse

View File

@@ -65,13 +65,11 @@ final class LoginController extends Controller
// Return MFA challenge — NO auth token, NO auth cookie. // Return MFA challenge — NO auth token, NO auth cookie.
// Expire the auth cookie to invalidate any stale browser session. // Expire the auth cookie to invalidate any stale browser session.
$cookieName = $this->resolveCookieName($request);
return response()->json([ return response()->json([
'success' => true, 'success' => true,
'mfa_required' => true, 'mfa_required' => true,
...$mfaSession, ...$mfaSession,
])->withCookie($this->forgetAuthCookie($cookieName)); ])->withCookie($this->forgetAuthCookie());
} }
// MFA required by policy but not yet set up — issue token with flag // MFA required by policy but not yet set up — issue token with flag
@@ -80,11 +78,10 @@ final class LoginController extends Controller
$data = $response->getData(true); $data = $response->getData(true);
$data['mfa_setup_required'] = true; $data['mfa_setup_required'] = true;
$cookieName = $this->resolveCookieName($request);
$token = $user->createToken('auth-token')->plainTextToken; $token = $user->createToken('auth-token')->plainTextToken;
return response()->json($data) return response()->json($data)
->withCookie($this->makeAuthCookie($cookieName, $token)); ->withCookie($this->makeAuthCookie($token));
} }
// No MFA — issue token as normal // No MFA — issue token as normal
@@ -101,11 +98,10 @@ final class LoginController extends Controller
]); ]);
$token = $user->createToken('auth-token')->plainTextToken; $token = $user->createToken('auth-token')->plainTextToken;
$cookieName = $this->resolveCookieName($request);
return $this->success([ return $this->success([
'user' => new MeResource($user), 'user' => new MeResource($user),
], 'Login successful') ], 'Login successful')
->withCookie($this->makeAuthCookie($cookieName, $token)); ->withCookie($this->makeAuthCookie($token));
} }
} }

View File

@@ -17,9 +17,7 @@ final class LogoutController extends Controller
{ {
$request->user()->currentAccessToken()->delete(); $request->user()->currentAccessToken()->delete();
$cookieName = $this->resolveCookieName($request);
return $this->success(null, 'Logged out successfully') return $this->success(null, 'Logged out successfully')
->withCookie($this->forgetAuthCookie($cookieName)); ->withCookie($this->forgetAuthCookie());
} }
} }

View File

@@ -4,42 +4,18 @@ declare(strict_types=1);
namespace App\Http\Controllers\Api\V1\Traits; namespace App\Http\Controllers\Api\V1\Traits;
use Illuminate\Http\Request;
use Symfony\Component\HttpFoundation\Cookie; use Symfony\Component\HttpFoundation\Cookie;
trait SetAuthCookie trait SetAuthCookie
{ {
private const COOKIE_MAP = [ private const COOKIE_NAME = 'crewli_app_token';
'app' => 'crewli_app_token',
'portal' => 'crewli_portal_token',
];
private const COOKIE_TTL_MINUTES = 60 * 24 * 7; // 7 days private const COOKIE_TTL_MINUTES = 60 * 24 * 7; // 7 days
protected function resolveCookieName(Request $request): string protected function makeAuthCookie(string $token): Cookie
{
$origin = $request->headers->get('Origin')
?? $request->headers->get('Referer')
?? '';
$appUrl = config('app.frontend_app_url', 'http://localhost:5174');
$portalUrl = config('app.frontend_portal_url', 'http://localhost:5175');
if ($this->originMatches($origin, $appUrl)) {
return self::COOKIE_MAP['app'];
}
if ($this->originMatches($origin, $portalUrl)) {
return self::COOKIE_MAP['portal'];
}
return self::COOKIE_MAP['app'];
}
protected function makeAuthCookie(string $cookieName, string $token): Cookie
{ {
return new Cookie( return new Cookie(
name: $cookieName, name: self::COOKIE_NAME,
value: $token, value: $token,
expire: now()->addMinutes(self::COOKIE_TTL_MINUTES), expire: now()->addMinutes(self::COOKIE_TTL_MINUTES),
path: '/', path: '/',
@@ -50,10 +26,10 @@ trait SetAuthCookie
); );
} }
protected function forgetAuthCookie(string $cookieName): Cookie protected function forgetAuthCookie(): Cookie
{ {
return new Cookie( return new Cookie(
name: $cookieName, name: self::COOKIE_NAME,
value: '', value: '',
expire: now()->subMinute(), expire: now()->subMinute(),
path: '/', path: '/',
@@ -63,19 +39,4 @@ trait SetAuthCookie
sameSite: 'Strict', sameSite: 'Strict',
); );
} }
private function originMatches(string $origin, string $configuredUrl): bool
{
if ($origin === '' || $configuredUrl === '') {
return false;
}
// Parse to compare host+port, ignoring trailing slashes and paths
$originHost = parse_url($origin, PHP_URL_HOST);
$originPort = parse_url($origin, PHP_URL_PORT);
$configHost = parse_url($configuredUrl, PHP_URL_HOST);
$configPort = parse_url($configuredUrl, PHP_URL_PORT);
return $originHost === $configHost && $originPort === $configPort;
}
} }

View File

@@ -10,74 +10,21 @@ use Symfony\Component\HttpFoundation\Response;
final class CookieBearerToken final class CookieBearerToken
{ {
private const COOKIE_NAMES = [ private const COOKIE_NAME = 'crewli_app_token';
'crewli_app_token',
'crewli_portal_token',
];
public function handle(Request $request, Closure $next): Response public function handle(Request $request, Closure $next): Response
{ {
// Skip if an Authorization header is already present // Skip if an Authorization header is already present (e.g. portal-token
// Bearer flow for artists/suppliers, or server-to-server callers).
if ($request->hasHeader('Authorization')) { if ($request->hasHeader('Authorization')) {
return $next($request); return $next($request);
} }
// Resolve the cookie name for the requesting app via Origin header. $token = $request->cookie(self::COOKIE_NAME);
// This prevents cross-app cookie leakage on localhost where the if ($token) {
// browser sends all cookies regardless of port. $request->headers->set('Authorization', 'Bearer '.$token);
$cookieName = $this->resolveCookieName($request);
if ($cookieName) {
$token = $request->cookie($cookieName);
if ($token) {
$request->headers->set('Authorization', 'Bearer ' . $token);
}
} }
return $next($request); return $next($request);
} }
private function resolveCookieName(Request $request): ?string
{
$origin = $request->headers->get('Origin')
?? $request->headers->get('Referer')
?? '';
if ($origin === '') {
// No Origin — fall back to first available cookie (e.g. server-to-server)
foreach (self::COOKIE_NAMES as $name) {
if ($request->cookie($name)) {
return $name;
}
}
return null;
}
$originHost = parse_url($origin, PHP_URL_HOST);
$originPort = parse_url($origin, PHP_URL_PORT);
$map = [
'app' => [config('app.frontend_app_url', 'http://localhost:5174'), 'crewli_app_token'],
'portal' => [config('app.frontend_portal_url', 'http://localhost:5175'), 'crewli_portal_token'],
];
foreach ($map as [$configuredUrl, $cookieName]) {
$configHost = parse_url($configuredUrl, PHP_URL_HOST);
$configPort = parse_url($configuredUrl, PHP_URL_PORT);
if ($originHost === $configHost && $originPort === $configPort) {
return $cookieName;
}
}
// Origin didn't match any configured frontend — fall back to first available
foreach (self::COOKIE_NAMES as $name) {
if ($request->cookie($name)) {
return $name;
}
}
return null;
}
} }

View File

@@ -7,7 +7,6 @@ namespace Tests\Feature\Security;
use App\Models\User; use App\Models\User;
use Database\Seeders\RoleSeeder; use Database\Seeders\RoleSeeder;
use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Foundation\Testing\RefreshDatabase;
use Laravel\Sanctum\Sanctum;
use Tests\TestCase; use Tests\TestCase;
final class HttpOnlyCookieAuthTest extends TestCase final class HttpOnlyCookieAuthTest extends TestCase
@@ -45,7 +44,7 @@ final class HttpOnlyCookieAuthTest extends TestCase
$response = $this->postJson('/api/v1/auth/login', [ $response = $this->postJson('/api/v1/auth/login', [
'email' => $user->email, 'email' => $user->email,
'password' => 'password', 'password' => 'password',
], ['Origin' => 'http://localhost:5174']); ]);
$response->assertOk(); $response->assertOk();
$response->assertCookie('crewli_app_token'); $response->assertCookie('crewli_app_token');
@@ -58,7 +57,7 @@ final class HttpOnlyCookieAuthTest extends TestCase
$response = $this->postJson('/api/v1/auth/login', [ $response = $this->postJson('/api/v1/auth/login', [
'email' => $user->email, 'email' => $user->email,
'password' => 'password', 'password' => 'password',
], ['Origin' => 'http://localhost:5174']); ]);
$cookie = $this->findCookie($response, 'crewli_app_token'); $cookie = $this->findCookie($response, 'crewli_app_token');
$this->assertNotNull($cookie, 'Cookie crewli_app_token not found'); $this->assertNotNull($cookie, 'Cookie crewli_app_token not found');
@@ -72,37 +71,41 @@ final class HttpOnlyCookieAuthTest extends TestCase
$response = $this->postJson('/api/v1/auth/login', [ $response = $this->postJson('/api/v1/auth/login', [
'email' => $user->email, 'email' => $user->email,
'password' => 'password', 'password' => 'password',
], ['Origin' => 'http://localhost:5174']); ]);
$cookie = $this->findCookie($response, 'crewli_app_token'); $cookie = $this->findCookie($response, 'crewli_app_token');
$this->assertNotNull($cookie); $this->assertNotNull($cookie);
$this->assertEquals('strict', strtolower($cookie->getSameSite())); $this->assertEquals('strict', strtolower($cookie->getSameSite()));
} }
public function test_login_sets_app_cookie_for_unknown_origin(): void public function test_login_sets_app_cookie_regardless_of_origin(): void
{ {
$user = User::factory()->create(); // Post-WS-3 PR-B2b: there is no per-app cookie resolution. Whatever
// Origin (or no Origin) the request carries, the auth cookie issued
// is always crewli_app_token. The request body alone determines auth.
$cases = [
'no Origin header' => [],
'app Origin' => ['Origin' => 'http://localhost:5174'],
'unknown Origin' => ['Origin' => 'http://localhost:9999'],
'foreign Origin' => ['Origin' => 'https://elsewhere.example.com'],
];
$response = $this->postJson('/api/v1/auth/login', [ foreach ($cases as $label => $headers) {
'email' => $user->email, $user = User::factory()->create();
'password' => 'password',
], ['Origin' => 'http://localhost:9999']);
$response->assertOk(); $response = $this->postJson('/api/v1/auth/login', [
$response->assertCookie('crewli_app_token'); 'email' => $user->email,
} 'password' => 'password',
], $headers);
public function test_login_sets_portal_cookie_for_portal_origin(): void $response->assertOk();
{
$user = User::factory()->create();
$response = $this->postJson('/api/v1/auth/login', [ $cookie = $this->findCookie($response, 'crewli_app_token');
'email' => $user->email, $this->assertNotNull(
'password' => 'password', $cookie,
], ['Origin' => 'http://localhost:5175']); "crewli_app_token must be set for case: {$label}",
);
$response->assertOk(); }
$response->assertCookie('crewli_portal_token');
} }
// --- Middleware Tests --- // --- Middleware Tests ---
@@ -142,7 +145,7 @@ final class HttpOnlyCookieAuthTest extends TestCase
$token = $user->createToken('auth-token')->plainTextToken; $token = $user->createToken('auth-token')->plainTextToken;
$response = $this->withUnencryptedCookie('crewli_app_token', $token) $response = $this->withUnencryptedCookie('crewli_app_token', $token)
->postJson('/api/v1/auth/logout', [], ['Origin' => 'http://localhost:5174']); ->postJson('/api/v1/auth/logout');
$response->assertOk(); $response->assertOk();
@@ -161,7 +164,7 @@ final class HttpOnlyCookieAuthTest extends TestCase
$token = $accessToken->plainTextToken; $token = $accessToken->plainTextToken;
$response = $this->withUnencryptedCookie('crewli_app_token', $token) $response = $this->withUnencryptedCookie('crewli_app_token', $token)
->postJson('/api/v1/auth/refresh', [], ['Origin' => 'http://localhost:5174']); ->postJson('/api/v1/auth/refresh');
$response->assertOk(); $response->assertOk();
$response->assertCookie('crewli_app_token'); $response->assertCookie('crewli_app_token');
@@ -192,45 +195,6 @@ final class HttpOnlyCookieAuthTest extends TestCase
$response->assertUnauthorized(); $response->assertUnauthorized();
} }
// --- Cross-App Isolation Tests ---
public function test_app_cookie_does_not_authenticate_portal_requests(): void
{
$user = User::factory()->create();
$token = $user->createToken('auth-token')->plainTextToken;
// App cookie is set, but request comes from portal origin —
// middleware should only read crewli_portal_token, not crewli_app_token
$response = $this->withUnencryptedCookie('crewli_app_token', $token)
->getJson('/api/v1/auth/me', ['Origin' => 'http://localhost:5175']);
$response->assertUnauthorized();
}
public function test_portal_cookie_does_not_authenticate_app_requests(): void
{
$user = User::factory()->create();
$token = $user->createToken('auth-token')->plainTextToken;
$response = $this->withUnencryptedCookie('crewli_portal_token', $token)
->getJson('/api/v1/auth/me', ['Origin' => 'http://localhost:5174']);
$response->assertUnauthorized();
}
public function test_correct_cookie_authenticates_with_matching_origin(): void
{
$user = User::factory()->create();
$token = $user->createToken('auth-token')->plainTextToken;
// Portal cookie + portal origin = authenticated
$response = $this->withUnencryptedCookie('crewli_portal_token', $token)
->getJson('/api/v1/auth/me', ['Origin' => 'http://localhost:5175']);
$response->assertOk();
$response->assertJsonPath('data.id', $user->id);
}
// --- Helper --- // --- Helper ---
private function findCookie($response, string $name): ?\Symfony\Component\HttpFoundation\Cookie private function findCookie($response, string $name): ?\Symfony\Component\HttpFoundation\Cookie