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

Refactor to make it usable for grpc-gateway and other use cases (not simple http server) #59

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Refactor to make it usable for grpc-gateway and other use cases (not simple http server) #59

wants to merge 5 commits into from

Commits on Jan 10, 2018

  1. LimitByRequest: return true instead of http error, false instead of nil

    @ilius
    ilius committed Jan 10, 2018
    Configuration menu
    Copy the full SHA
    305dac0 View commit details
    Browse the repository at this point in the history

  2. don't set response headers in LimitByRequest func, do it in LimitHand… 

    …ler func
    @ilius
    ilius committed Jan 10, 2018
    Configuration menu
    Copy the full SHA
    c2fd627 View commit details
    Browse the repository at this point in the history

  3. update comment

    @ilius
    ilius committed Jan 10, 2018
    Configuration menu
    Copy the full SHA
    ce8107d View commit details
    Browse the repository at this point in the history

Commits on Jan 16, 2018

  1. small simplification in LimitByKeys

    @ilius
    ilius committed Jan 16, 2018
    Configuration menu
    Copy the full SHA
    667acfd View commit details
    Browse the repository at this point in the history

  2. wrap rate.Limiter in a new struct called LowLevelLimiter, add LastBlo… 

    …ck in it, return prevBlockTime in LimitByRequest
    @ilius
    ilius committed Jan 16, 2018
    Configuration menu
    Copy the full SHA
    37e8f46 View commit details
    Browse the repository at this point in the history