TerminalWitchcraft / actix-ratelimit

Rate limiter framework for Actix web
MIT License
127 stars 25 forks source link

Mention RateLimit Draft #1

Open ioggstream opened 4 years ago

ioggstream commented 4 years ago

I wish

The project to mention the ratelimit-headers draft: https://www.ietf.org/id/draft-polli-ratelimit-headers-03.html

Notes

It seems we're trying to standardize the behavior you have implemented :) Please take a look at https://github.com/ioggstream/draft-polli-ratelimit-headers

If you confirm that, we can cite actix-ratelimit in the spec as an implementor. In that case just drop a comment here:

TerminalWitchcraft commented 4 years ago

Hi,

Seems like to abide by the convention mentioned, I need to change the header names from x-ratelimit-remaining to Ratelimit-remaining and so on. If this is the case, I can do the change in the next release while also mentioning about the same in this project's README. Is this what you expect?

ioggstream commented 4 years ago

For now it would just be great to mention the draft in the README, so that I can:

Once we go RFC we can change the names :P

As an implementor, your feedback on the actual contents of the draft are very welcome!