8000 [RateLimiter][Security] Improve performance of login/request rate limiter by Seldaek · Pull Request #46110 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[RateLimiter][Security] Improve performance of login/request rate limiter #46110

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
fabpot merged 5 commits into from
Jul 24, 2022
Merged

[RateLimiter][Security] Improve performance of login/request rate limiter #46110

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
a029188
Allow consuming 0 tokens to peek at the token storage
Seldaek Apr 19, 2022
9a2ec0c
Add PeekableRequestRateLimiterInterface and fix the LoginThrottlingLi…
Seldaek Apr 19, 2022
c3415a1
Make sure the retryAfter date is always set on RateLimit even when pe…
Seldaek Apr 19, 2022
679fa91
Add missing return type
Seldaek May 5, 2022
5c360db
[RateLimiter] Fix peeking in token bucket/fixed window and add tests
wouterj Jul 24, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Allow consuming 0 tokens to peek at the token storage
  • Loading branch information
@Seldaek @wouterj
Seldaek authored and wouterj committed Jul 24, 2022
commit a029188317314bac4cb8be6a1f74e4f400ae58f3
5 changes: 4 additions & 1 deletion src/Symfony/Component/RateLimiter/Policy/FixedWindowLimiter.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,10 @@ public function reserve(int $tokens = 1, float $maxTime = null): Reservation

$reservation = new Reservation($now + $waitDuration, new RateLimit($window->getAvailableTokens($now), \DateTimeImmutable::createFromFormat('U', floor($now + $waitDuration)), false, $this->limit));
}
$this->storage->save($window);

if (0 < $tokens) {
$this->storage->save($window);
}
} finally {
$this->lock->release();
}
Expand Down
5 changes: 4 additions & 1 deletion src/Symfony/Component/RateLimiter/Policy/SlidingWindowLimiter.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,10 @@ public function consume(int $tokens = 1): RateLimit
}

$window->add($tokens);
$this->storage->save($window);

if (0 < $tokens) {
$this->storage->save($window);
}

return new RateLimit($this->getAvailableTokens($window->getHitCount()), $window->getRetryAfter(), true, $this->limit);
} finally {
Expand Down
4 changes: 3 additions & 1 deletion src/Symfony/Component/RateLimiter/Policy/TokenBucketLimiter.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,9 @@ public function reserve(int $tokens = 1, float $maxTime = null): Reservation
$reservation = new Reservation($now + $waitDuration, new RateLimit(0, \DateTimeImmutable::createFromFormat('U', floor($now + $waitDuration)), false, $this->maxBurst));
}

$this->storage->save($bucket);
if (0 < $tokens) {
$this->storage->save($bucket);
}
} finally {
$this->lock->release();
}
Expand Down
0