thrift: add access logging support (#21155)

Note: support for logging headers (via %REQ()%) will happen in a follow-up.

Risk Level: medium
Testing: unit tests
Doc Changes: included
Release Notes: updated
Fixes #17898.

Signed-off-by: Raul Gutierrez Segales <rgs@pinterest.com>

Mirrored from https://github.com/envoyproxy/envoy @ 01a0ff52ed3a1e55b14c3bbff4daede6c61fd5fb
pull/626/head
data-plane-api(Azure Pipelines) 3 years ago
parent c840a730e5
commit ca7d99e73f
  1. 1
      envoy/extensions/filters/network/thrift_proxy/v3/BUILD
  2. 7
      envoy/extensions/filters/network/thrift_proxy/v3/thrift_proxy.proto

@ -7,6 +7,7 @@ licenses(["notice"]) # Apache 2
api_proto_package(
deps = [
"//envoy/annotations:pkg",
"//envoy/config/accesslog/v3:pkg",
"//envoy/config/core/v3:pkg",
"//envoy/config/route/v3:pkg",
"@com_github_cncf_udpa//udpa/annotations:pkg",

@ -2,6 +2,7 @@ syntax = "proto3";
package envoy.extensions.filters.network.thrift_proxy.v3;
import "envoy/config/accesslog/v3/accesslog.proto";
import "envoy/config/core/v3/config_source.proto";
import "envoy/extensions/filters/network/thrift_proxy/v3/route.proto";
@ -73,7 +74,7 @@ message Trds {
string route_config_name = 2;
}
// [#next-free-field: 9]
// [#next-free-field: 10]
message ThriftProxy {
option (udpa.annotations.versioning).previous_message_type =
"envoy.config.filter.network.thrift_proxy.v2alpha1.ThriftProxy";
@ -112,6 +113,10 @@ message ThriftProxy {
// Optional maximum requests for a single downstream connection. If not specified, there is no limit.
google.protobuf.UInt32Value max_requests_per_connection = 7;
// Configuration for :ref:`access logs <arch_overview_access_logs>`
// emitted by Thrift proxy.
repeated config.accesslog.v3.AccessLog access_log = 9;
}
// ThriftFilter configures a Thrift filter.

Loading…
Cancel
Save