generationLimiterKey($email); $maxGenerations = (int) config('core.auth.otp.generation_limit', 3); if (RateLimiter::tooManyAttempts($limiterKey, $maxGenerations)) { throw new OtpThrottledException(RateLimiter::availableIn($limiterKey)); } RateLimiter::hit($limiterKey, (int) config('core.auth.otp.generation_decay_minutes', 10) * 60); $model = config('auth.providers.users.model'); $user = $model::where('email', $email)->first(); $code = str_pad((string) random_int(0, 999999), self::CODE_LENGTH, '0', STR_PAD_LEFT); if ($user) { $user->otp_code = $code; $user->otp_expires_at = now()->addMinutes(self::EXPIRY_MINUTES); $user->otp_attempts = 0; $user->save(); } else { // No row yet — deliberately not created here. See this // class's own docblock for why: creating one on every // generateAndSend() call let anyone mint real User/Customer // rows for an email nobody proved they owned. Cache::put($this->pendingKey($email), [ 'code' => $code, 'expires_at' => now()->addMinutes(self::EXPIRY_MINUTES)->timestamp, 'attempts' => 0, ], now()->addMinutes(self::EXPIRY_MINUTES)); } Mail::to($email)->send(new UserOtpMail($user->name ?? $email, $code)); return true; } /** * A wrong code counts against core.auth.otp.max_attempts and, once * reached, invalidates the code entirely — the shopper must request * a fresh one via generateAndSend() (itself throttled independently * — see this class's own docblock) rather than being able to keep * guessing against a still-live code for the rest of its 10-minute * expiry window. Applies identically to the cache-backed (no User row * yet) and DB-backed (existing User row) paths. */ public function validate(string $email, string $code, ?Request $request = null): ?Authenticatable { $model = config('auth.providers.users.model'); $existing = $model::where('email', $email)->exists(); $result = $existing ? $this->validateExisting($model, $email, $code) : $this->validatePending($model, $email, $code); if (! $result) { return null; } RateLimiter::clear($this->generationLimiterKey($email)); Auth::login($result); $this->sessions->record($result, $request); Event::dispatch(new UserAuthenticated($result)); return $result; } /** * lockForUpdate() + a transaction make the read-check-increment-save * atomic across concurrent requests for the same user — without it, * two guesses fired in parallel can each read the same pre-increment * otp_attempts value and both save past max_attempts, letting an * attacker exceed the lockout by parallelizing requests instead of * sending them serially. */ private function validateExisting(string $model, string $email, string $code): ?Authenticatable { return DB::transaction(function () use ($model, $email, $code) { $user = $model::where('email', $email)->lockForUpdate()->first(); if (! $user || ! $user->otp_expires_at || now()->isAfter($user->otp_expires_at)) { return null; } if (! hash_equals((string) $user->otp_code, $code)) { $user->otp_attempts++; if ($user->otp_attempts >= (int) config('core.auth.otp.max_attempts', 5)) { $user->otp_code = null; $user->otp_expires_at = null; $user->otp_attempts = 0; } $user->save(); return null; } $user->otp_code = null; $user->otp_expires_at = null; $user->otp_attempts = 0; $user->save(); return $user; }); } /** * No User row exists yet, so there's nothing to lockForUpdate() — * Cache::lock() is the equivalent guard against two parallel guesses * against the same pending signup both reading the same pre-increment * attempts count. The User (and, via UserCreated, its paired Customer) * is only ever created here, once the code has actually been proven * correct — never from generateAndSend(). */ private function validatePending(string $model, string $email, string $code): ?Authenticatable { $key = $this->pendingKey($email); return Cache::lock("{$key}:lock", 10)->block(5, function () use ($model, $email, $code, $key) { $pending = Cache::get($key); if (! $pending || now()->timestamp > $pending['expires_at']) { return null; } if (! hash_equals((string) $pending['code'], $code)) { $pending['attempts']++; if ($pending['attempts'] >= (int) config('core.auth.otp.max_attempts', 5)) { Cache::forget($key); } else { Cache::put($key, $pending, now()->addMinutes(self::EXPIRY_MINUTES)); } return null; } Cache::forget($key); $user = $model::firstOrCreate(['email' => $email]); // wasRecentlyCreated is Eloquent's own "did firstOrCreate() // just INSERT, or did it find an existing row" flag. Always // true here in practice (validatePending() only runs when no // row existed moments ago), but checked anyway rather than // assumed, in case of an extremely unlikely race with a // signup completed through some other path in between. if ($user->wasRecentlyCreated) { Event::dispatch(new UserCreated($user)); } return $user; }); } private function generationLimiterKey(string $email): string { return 'otp-generate:'.strtolower($email); } private function pendingKey(string $email): string { return 'otp-pending:'.strtolower($email); } }