Skip to content

Commit 110dd0d

Browse files
authored
Port security fixes to default login rate limiter (#473)
1 parent 31a3412 commit 110dd0d

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

stubs/FortifyServiceProvider.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
use Illuminate\Http\Request;
1111
use Illuminate\Support\Facades\RateLimiter;
1212
use Illuminate\Support\ServiceProvider;
13+
use Illuminate\Support\Str;
1314
use Laravel\Fortify\Fortify;
1415

1516
class FortifyServiceProvider extends ServiceProvider
@@ -33,9 +34,9 @@ public function boot(): void
3334
Fortify::resetUserPasswordsUsing(ResetUserPassword::class);
3435

3536
RateLimiter::for('login', function (Request $request) {
36-
$email = (string) $request->email;
37+
$throttleKey = Str::transliterate(Str::lower($request->input(Fortify::username())).'|'.$request->ip());
3738

38-
return Limit::perMinute(5)->by($email.$request->ip());
39+
return Limit::perMinute(5)->by($throttleKey);
3940
});
4041

4142
RateLimiter::for('two-factor', function (Request $request) {

0 commit comments

Comments
 (0)