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

In memory rate limiter. #767

Merged
merged 4 commits into from
Dec 14, 2022
Merged

In memory rate limiter. #767

merged 4 commits into from
Dec 14, 2022

Conversation

anjor
Copy link
Contributor

@anjor anjor commented Dec 13, 2022

No description provided.

@anjor
Copy link
Contributor Author

anjor commented Dec 13, 2022

Don't merge yet. I would like to make the rate limit configurable.

@anjor anjor requested a review from en0ma December 13, 2022 12:56
@alvin-reyes alvin-reyes marked this pull request as draft December 13, 2022 14:43
@anjor anjor marked this pull request as ready for review December 13, 2022 17:54
en0ma
en0ma previously approved these changes Dec 14, 2022
Copy link
Contributor

@en0ma en0ma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@en0ma
Copy link
Contributor

en0ma commented Dec 14, 2022

@anjor please add it to the shuttles too

@anjor
Copy link
Contributor Author

anjor commented Dec 14, 2022

@en0ma done. Mind taking a look again?

Copy link
Contributor

@en0ma en0ma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@anjor anjor merged commit 4ffebc9 into dev Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants