envoyproxy / envoyproxy/envoy

local ratelimit: support `apply_on_stream_done` for local rate limit

Open
#37,790 1 comment 0 reactions 1 assignee Claimed by @wbpcode View on GitHub
area/ratelimit enhancement no stalebot
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 20h
Merged PRs (30d)
437

Description

*Title*: *local ratelimit: support `apply_on_stream_done` for local rate limit*

*Description*:

`apply_on_stream_done` was introduced to support post-commit of usage. It make sense to limit the resource usage that could only get from the response.

But it's not supported now for local rate limit.

[optional *Relevant Links*:]
>Any extra documentation required to understand the issue.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.