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

Add ratelimit headers. #6111

Closed
wants to merge 1 commit into from

Conversation

ioggstream
Copy link

@ioggstream ioggstream commented Dec 30, 2019

What does this PR do?

Stub implementation of https://tools.ietf.org/html/draft-polli-ratelimit-headers-03

Motivation

Avoid clients receiving 429 abruptly

Fixes #6113

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

Other implementors: Kong, 3scale ,
Envoy, Express Gateway PR.

@ldez ldez added this to To review in v2 via automation Jan 1, 2020
@ldez ldez changed the title WIP: add ratelimit headers. Add ratelimit headers. Jan 7, 2020
@dtomcej
Copy link
Contributor

dtomcej commented May 26, 2020

Of note: The draft was updated and renewed in March 2020: https://tools.ietf.org/html/draft-polli-ratelimit-headers-02

@ioggstream
Copy link
Author

ioggstream commented May 26, 2020

Of note: The draft was updated and renewed in March 2020: https://tools.ietf.org/html/draft-polli-ratelimit-headers-02

Yep! I linked 3scale and kong merge commits too.

I need some help to finish this implementation @dtomcej can you provide some guidance?

@ioggstream
Copy link
Author

Hi @dtomcej , the ratelimit draft recently landed on Envoy too envoyproxy/envoy#12410 Do you have any feedback for implementing it into traefik and reorganize this PR?

@kevtainer
Copy link
Contributor

kevtainer commented Aug 10, 2020

@ioggstream Thank you for your interest in Traefik and for the draft PR.

Given the RFC you've mentioned is still in draft state, the team here would prefer to wait until the RFC is accepted, alternatively if there is a large consensus of Traefik users who would like this feature implemented before the acceptance of the RFC draft we would revisit the topic.

In the meantime, it is possible to create custom plugins using Traefik Pilot in the 2.3RC - you can read more about how to build your own.

@kevtainer
Copy link
Contributor

@ioggstream The team has decided to close this PR, in the future we'll consider the PR once the specification has been recommended or enough users have requested this functionality exist on the official rate limit middleware. Feel free to keep the issue open in the meantime and let us know when the RFC has come out of the draft state.

Thank you again for the contribution, and we're looking forward to the custom plugin demonstrating this capability.

@SantoDE SantoDE closed this Aug 18, 2020
v2 automation moved this from To review to Done Aug 18, 2020
@ioggstream
Copy link
Author

@notsureifkevin thanks for your time :) I will take a look at the custom plugin features, but it will take some time.

In case you were going to add ratelimit headers Ratelimit middleware, I just suggest reading the I-D https://tools.ietf.org/html/draft-polli-ratelimit-headers-02 first :D

Have a nice day, R.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
v2
Done
Development

Successfully merging this pull request may close these issues.

Support RateLimit headers Internet Draft
6 participants