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

More aggressive rate limiting #333

Merged
merged 2 commits into from
Sep 11, 2023
Merged

Conversation

dantownsend
Copy link
Member

@dantownsend dantownsend commented Sep 11, 2023

By default, we're going to rate limit more aggressively - the user can always override this behaviour if they choose via the rate_limit_provider argument of create_admin.

This is until we can update all our JS dependencies
@dantownsend dantownsend added the enhancement New feature or request label Sep 11, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #333 (184f955) into master (0636086) will not change coverage.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##           master     #333   +/-   ##
=======================================
  Coverage   93.31%   93.31%           
=======================================
  Files           5        5           
  Lines         359      359           
=======================================
  Hits          335      335           
  Misses         24       24           
Files Changed Coverage Δ
piccolo_admin/endpoints.py 92.61% <ø> (ø)

@dantownsend dantownsend merged commit 455e615 into master Sep 11, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants