Skip to content

[RateLimiter] default lock_factory to auto #20858

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 7, 2025
Merged

[RateLimiter] default lock_factory to auto #20858

merged 1 commit into from
Apr 7, 2025

Conversation

kbond
Copy link
Member

@kbond kbond commented Apr 4, 2025

Ref: symfony/symfony#60099
Closes #20854.

This is a small DX improvement so don't think it needs too much explanation.

@carsonbot carsonbot added this to the 7.3 milestone Apr 4, 2025
@javiereguiluz javiereguiluz merged commit ca34f50 into symfony:7.3 Apr 7, 2025
2 of 3 checks passed
@javiereguiluz
Copy link
Member

Thanks Kevin! You're right that this is minor, but still we added a versionadded directive to mention the behavior change. Thanks!

@kbond kbond deleted the rate-limiter-lock-config branch April 7, 2025 07:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
RateLimiter Status: Reviewed
Projects
None yet
Milestone
7.3
Development

Successfully merging this pull request may close these issues.

[FrameworkBundle][RateLimiter] default lock_factory to auto
3 participants
@kbond @javiereguiluz @carsonbot