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 whitelist ip list for excluding by rate limit #610

Open
aminlatifi opened this issue Sep 4, 2021 · 2 comments
Open

Add whitelist ip list for excluding by rate limit #610

aminlatifi opened this issue Sep 4, 2021 · 2 comments
Assignees
@mohammadranjbarz
Labels
enhancement

Comments

@aminlatifi
Copy link
Member

aminlatifi commented Sep 4, 2021 •
edited by mohammadranjbarz
Loading

Exclude bridge monitor from rate limits
It includes two subtask

  1. Exclude in nginx rate limit
  2. Exclude in feathers rate limit. For this one I suggest setting header in nginx and feathers dismiss rate limit if that header exists
@mohammadranjbarz mohammadranjbarz changed the title Exclude bridge monitor from rate limits Add whitelist ip for excluding by rate limit Sep 12, 2021
@mohammadranjbarz mohammadranjbarz changed the title Add whitelist ip for excluding by rate limit Add whitelist ip list for excluding by rate limit Sep 12, 2021
mohammadranjbarz added a commit that referenced this issue Sep 12, 2021
@mohammadranjbarz
Add whitelist ip for rate limit
8b970b3 
related to #610
This was referenced Sep 12, 2021
Merged
Open
@aminlatifi
Copy link
Member Author

aminlatifi commented Sep 13, 2021 •
edited
Loading

@mohammadranjbarz is nginx part implemented?

@mohammadranjbarz
Copy link
Collaborator

@mohammadranjbarz is nginx part implemented?

@aminlatifi I added new issue for Nginx Giveth/giveth-dapp#2539

@mohammadranjbarz mohammadranjbarz mentioned this issue Sep 18, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mohammadranjbarz mohammadranjbarz

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aminlatifi @mohammadranjbarz