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 to API endpoints #79

Open
dpopp07 opened this issue Jan 18, 2018 · 2 comments
Open

Add rate limiting to API endpoints #79

dpopp07 opened this issue Jan 18, 2018 · 2 comments
Assignees
@dpopp07
Labels
api-rest

Comments

@dpopp07
Copy link
Collaborator

dpopp07 commented Jan 18, 2018

It is good practice to rate-limit API's and it's pretty easy. I'm going to add it as a part of my mission to build out the existing endpoints as much as I can before adding new ones.
@dpopp07 dpopp07 self-assigned this Jan 18, 2018
@dpopp07
Copy link
Collaborator Author

dpopp07 commented Jan 18, 2018

How GitHub does this

@dpopp07
Copy link
Collaborator Author

dpopp07 commented Feb 6, 2018

Have a functioning rate limiter in place. It could use a bit more of a robust implementation and needs to be tested. Since we aren't even deployed yet, this is too low of a priority to take any more time. Iceboxing for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dpopp07 dpopp07

Labels
api-rest
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dpopp07