From fff396e3d03405f76e27fa63792dcb1ad384e54e Mon Sep 17 00:00:00 2001 From: "data-plane-api(Azure Pipelines)" Date: Mon, 19 Dec 2022 15:52:51 +0000 Subject: [PATCH] [Bandwidth-Limit] Fix bandwidth delay calculation logic (#24024) * Fix bandwidth delay calculation logic Signed-off-by: Zhiyong Yang Mirrored from https://github.com/envoyproxy/envoy @ 230a1780c62299b0bce0b99f8a76c90ca4207855 --- .../http/bandwidth_limit/v3/bandwidth_limit.proto | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/envoy/extensions/filters/http/bandwidth_limit/v3/bandwidth_limit.proto b/envoy/extensions/filters/http/bandwidth_limit/v3/bandwidth_limit.proto index 3aa332e7..0d9ae747 100644 --- a/envoy/extensions/filters/http/bandwidth_limit/v3/bandwidth_limit.proto +++ b/envoy/extensions/filters/http/bandwidth_limit/v3/bandwidth_limit.proto @@ -72,11 +72,13 @@ message BandwidthLimit { // // .. note:: // - // * If set true, the response trailers ``bandwidth-request-delay-ms`` and ``bandwidth-response-delay-ms`` will be added, prefixed by ``response_trailer_prefix``. - // * bandwidth-request-delay-ms: delay time in milliseconds it took for the request stream transfer. - // * bandwidth-response-delay-ms: delay time in milliseconds it took for the response stream transfer. - // * If :ref:`enable_mode ` is ``DISABLED`` or ``REQUEST``, the trailers will not be set. - // * If both the request and response delay time is 0, the trailers will not be set. + // If set true, the following 4 trailers will be added, prefixed by ``response_trailer_prefix``: + // * bandwidth-request-delay-ms: delay time in milliseconds it took for the request stream transfer including request body transfer time and the time added by the filter. + // * bandwidth-response-delay-ms: delay time in milliseconds it took for the response stream transfer including response body transfer time and the time added by the filter. + // * bandwidth-request-filter-delay-ms: delay time in milliseconds in request stream transfer added by the filter. + // * bandwidth-response-filter-delay-ms: delay time in milliseconds that added by the filter. + // If :ref:`enable_mode ` is ``DISABLED`` or ``REQUEST``, the trailers will not be set. + // If both the request and response delay time is 0, the trailers will not be set. // bool enable_response_trailers = 6;