Skip to content
This repository has been archived by the owner on Feb 27, 2024. It is now read-only.

Mention RateLimit Draft #1

Open
ioggstream opened this issue Jun 3, 2020 · 2 comments
Open

Mention RateLimit Draft #1

ioggstream opened this issue Jun 3, 2020 · 2 comments

Comments

@ioggstream
Copy link

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
Copy link
Owner

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
Copy link
Author

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!

leonardolang pushed a commit to leonardolang/actix-ratelimit that referenced this issue Nov 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants