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.

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

Jump to bottom

Add rate limiting for API calls and update the votes list #2

Merged
merged 3 commits into from
Jan 24, 2025
Merged

Add rate limiting for API calls and update the votes list #2

merged 3 commits into from
Jan 24, 2025

Conversation

mrkara
Copy link
Contributor

@mrkara mrkara commented Jan 22, 2025 •
edited
Loading

Summary

  • Add a basic rate limiting function
  • Make rate limiting adjustable
  • Update the votes list to add CVV003
  • Update the README

Checklist

  • Code is properly formatted
  • All commits are signed
  • Tests included/updated or not needed
  • Documentation (manuals or wiki) has been updated or is not required

mrkara added 3 commits January 22, 2025 21:51
@mrkara
Add basic rate limiting
c085f9e 
Signed-off-by: Muhammet Kara <[email protected]>
@mrkara
Make rate limiting adjustable
832317b 
Signed-off-by: Muhammet Kara <[email protected]>
@mrkara
Add CVV003 to the votes list
98ad836 
Signed-off-by: Muhammet Kara <[email protected]>
@mrkara mrkara merged commit 9b81d59 into main Jan 24, 2025
2 checks passed
@mrkara mrkara deleted the kara/ratelimit branch January 24, 2025 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Adrian-Wrona Adrian-Wrona Adrian-Wrona approved these changes

@mssteuer mssteuer mssteuer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Receiving 429 Client Error: Too Many Requests for url:
3 participants
@mrkara @mssteuer @Adrian-Wrona