The ratelimit draft is now managed by the IETF

[ioggstream]

This PR

The RateLimit Headers draft has been adopted by the IETF httpapi workgroup.

Please provide feedback on the ongoing standardization work on https://github.com/ietf-wg-httpapi/ratelimit-headers
so that we can achieve the RFC status :)

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[codecov]

Codecov Report

Merging #3246 (452e291) into master (ccff172) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3246      +/-   ##
============================================
- Coverage     74.05%   74.05%   -0.01%     
- Complexity    12999    13003       +4     
============================================
  Files          1133     1133              
  Lines         49197    49197              
  Branches       6259     6259              
============================================
- Hits          36435    36431       -4     
- Misses         9546     9549       +3     
- Partials       3216     3217       +1     

Impacted Files	Coverage Δ	Complexity Δ
...p/armeria/common/throttling/ThrottlingHeaders.java	100.00% <ø> (ø)	1.00 <0.00> (ø)
...om/linecorp/armeria/client/HttpSessionHandler.java	71.59% <0.00%> (-9.47%)	49.00% <0.00%> (-5.00%)
...eria/internal/common/DefaultSplitHttpResponse.java	77.84% <0.00%> (-2.40%)	14.00% <0.00%> (ø%)
...linecorp/armeria/client/HttpRequestSubscriber.java	64.55% <0.00%> (-1.27%)	29.00% <0.00%> (-1.00%)
...p/armeria/internal/common/eureka/InstanceInfo.java	51.75% <0.00%> (-0.88%)	23.00% <0.00%> (-1.00%)
...inecorp/armeria/server/HttpResponseSubscriber.java	79.21% <0.00%> (-0.40%)	58.00% <0.00%> (ø%)
...corp/armeria/common/logging/DefaultRequestLog.java	76.42% <0.00%> (-0.26%)	223.00% <0.00%> (-1.00%)
...armeria/internal/common/CancellationScheduler.java	76.15% <0.00%> (+0.41%)	82.00% <0.00%> (+1.00%)
...java/com/linecorp/armeria/server/DefaultRoute.java	90.74% <0.00%> (+0.92%)	57.00% <0.00%> (+1.00%)
...necorp/armeria/client/HeapBasedEventLoopState.java	93.81% <0.00%> (+1.03%)	32.00% <0.00%> (+1.00%)
... and 6 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ccff172...452e291. Read the comment docs.

[trustin]

Awesome! Thanks for your first contribution, @ioggstream 🙇 Would you mind signing the CLA?

[ikhoon]

Good to know. Thanks! @ioggstream

[ikhoon]

Gentle ping @ioggstream. 😉

[trustin]

Superseded by #3374. Thanks a lot for suggesting this, @ioggstream! I think @max904-github might have some feedback about your IETF draft by the way, since he is the main author of that functionality.

[ioggstream]

Sure @max904-github let me know your feedback!

Moreover we are evaluating Structured-Fields https://www.rfc-editor.org/rfc/rfc8941.html which simplifies http header parsing too...