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

[staging] Rate limiter integration in log only mode #934

Merged
merged 3 commits into from
Aug 29, 2022
Merged

[staging] Rate limiter integration in log only mode #934

merged 3 commits into from
Aug 29, 2022

Conversation

crisog
Copy link
Contributor

@crisog crisog commented Aug 29, 2022

PR #932

adshmh and others added 3 commits August 29, 2022 12:22
@adshmh
First integration with rate-limiter service: log warning only
3781bf4 
Signed-off-by: Arash Deshmeh <[email protected]>
@adshmh
Make RATE_LIMITER_URL an optional environment var
7d9ad50 
Signed-off-by: Arash Deshmeh <[email protected]>
@adshmh
Merge pull request #932 from pokt-foundation/rate-limiter-integration…
8450d01 
…-in-log-only-mode

First integration with rate-limiter service: log warning only
@height
Copy link

height bot commented Aug 29, 2022

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@adshmh
Copy link
Contributor

adshmh commented Aug 29, 2022

LGTM

@rem1niscence rem1niscence merged commit 0333e32 into staging Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@crisog @adshmh @rem1niscence