Skip to content
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.

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

fix: Correct auth login rate limit routes #4698

Merged
merged 4 commits into from
Sep 20, 2024

Conversation

anticorrelator
Copy link
Contributor

resolves #4691

@anticorrelator anticorrelator changed the base branch from main to auth September 20, 2024 15:44
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Sep 20, 2024
@anticorrelator anticorrelator force-pushed the dustin/correct-auth-login-rate-limit-routes branch from 8e0af0a to dd1e9be Compare September 20, 2024 19:43
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Sep 20, 2024
@anticorrelator anticorrelator merged commit a801a8e into auth Sep 20, 2024
17 checks passed
@anticorrelator anticorrelator deleted the dustin/correct-auth-login-rate-limit-routes branch September 20, 2024 20:59
RogerHYang pushed a commit that referenced this pull request Sep 21, 2024
* Use fully qualified auth paths for rate limiter

* Add playwright rate limiter test

* Disable auth rate limits in ITs

* Format w/ prettier
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[auth][rest][bug] rate limiters are not associated with correct routes
2 participants