Skip to content

Added RateLimiter and beta signup apis#7

Merged
jay-dee7 merged 1 commit intomasterfrom
rate-limiter
Jul 12, 2021
Merged

Added RateLimiter and beta signup apis#7
jay-dee7 merged 1 commit intomasterfrom
rate-limiter

Conversation

@guacamole
Copy link
Member

  • Added basic RateLimiter to help against attacks, RateLimiter package yet to implement
  • Added Beta register API to accumulate Emails for Beta

@guacamole guacamole requested a review from jay-dee7 July 12, 2021 15:47
@guacamole guacamole self-assigned this Jul 12, 2021
@@ -0,0 +1,3 @@
mod-fix:
Copy link
Member

Choose a reason for hiding this comment

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

@guacamole this is a bad fix, which'll work for now. We'll have to make sure we have a better and idiomatic way to do this

Copy link
Member Author

Choose a reason for hiding this comment

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

we can track this via an open github issue

@jay-dee7
Copy link
Member

@guacamole also please rebase from master

@jay-dee7 jay-dee7 merged commit 6d5b253 into master Jul 12, 2021
@jay-dee7 jay-dee7 deleted the rate-limiter branch September 5, 2021 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants