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

feature: new module resty.limit.rate #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

timebug
Copy link

@timebug timebug commented Nov 5, 2017

This PR have been discussed at #26.

This module provides APIs to help the OpenResty/ngx_lua user programmers limit request rate using the "token bucket" method.

Small differences with existing modules, this module use resty.lock to make sure shdict operations across multiple nginx worker process is atomic, but default disable.

@timebug
Copy link
Author

timebug commented Oct 25, 2018

Given that there are some features that apply to our own business environment, we maintain a repository independently, see https://github.com/upyun/lua-resty-limit-rate

@agentzh Thanks to the review code, I got a lot of valuable advice, I will continue to improve in the future if necessary :-)

@timebug timebug closed this Oct 25, 2018
@wangdiqi
Copy link

Why hasn't merged this PR?

@timebug
Copy link
Author

timebug commented Jan 21, 2021

@deetchtoys Hello, I reopened this PR, over the years, this module has been used in some company production environments for a long time, if @agentzh @doujiang24 see fit to merge into limit-traffic, I will continue to improve this.

@timebug timebug reopened this Jan 21, 2021
monkeyDluffy6017 pushed a commit to monkeyDluffy6017/lua-resty-limit-traffic that referenced this pull request Apr 19, 2023
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.

2 participants