A group of minor fixes2

nikolaposa / rate-limit

🚔 General purpose rate limiter implementation.

MIT License

269 stars 47 forks source link

A group of minor fixes2 #10

Closed poursal closed 5 years ago

poursal commented 5 years ago

Fixes the following:

Can update the limit externally (i.e. increase the limit for authenticated users)
Fix the reset-at wave problems by going to microseconds
Fix marginal bug in HIT, where by increment an expired key it creates a key that never expires
Be able to set the trusted LB that are allowed to send the FORWARD headers
Be able to switch between IP and authenticated user on the Identity Resolver

A bug fix that I introduced and did not include

nikolaposa commented 5 years ago

@poursal I would appreciate if you could split your changes across multiple branches/PRs instead of having a single PR master directly from the master branch of your fork. For instance: fix/reset-at-wave, fix/never-expiring-key, feature/user-identity-resolver, etc. That way it would be much more easier for me to review them.