From 4838a8a56fa52d09cb10742ef8616a81eedf8f87 Mon Sep 17 00:00:00 2001 From: Lizan Zhou Date: Wed, 9 May 2018 13:43:28 -0700 Subject: [PATCH 1/2] hardcode authority header for mixer temporarily Signed-off-by: Lizan Zhou --- src/envoy/utils/grpc_transport.h | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/envoy/utils/grpc_transport.h b/src/envoy/utils/grpc_transport.h index bda56b6621f..131db3bd07d 100644 --- a/src/envoy/utils/grpc_transport.h +++ b/src/envoy/utils/grpc_transport.h @@ -45,7 +45,12 @@ class GrpcTransport : public Grpc::TypedAsyncRequestCallbacks, ResponseType* response, Tracing::Span& parent_span, istio::mixerclient::DoneFunc on_done); - void onCreateInitialMetadata(Http::HeaderMap&) override {} + void onCreateInitialMetadata(Http::HeaderMap& metadata) override { + // We generate cluster name contains invalid characters, so override the + // authority header temorarily until it can be specified via CDS. + // See https://github.com/envoyproxy/envoy/issues/3297 for details. + metadata.Host()->value("mixer"); + } void onSuccess(std::unique_ptr&& response, Tracing::Span& span) override; From f4a718ef93a2dc5c8720f8614f06c2506b1079ce Mon Sep 17 00:00:00 2001 From: Lizan Zhou Date: Wed, 9 May 2018 14:01:36 -0700 Subject: [PATCH 2/2] fix compile error Signed-off-by: Lizan Zhou --- src/envoy/utils/grpc_transport.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/envoy/utils/grpc_transport.h b/src/envoy/utils/grpc_transport.h index 131db3bd07d..41bbffe5407 100644 --- a/src/envoy/utils/grpc_transport.h +++ b/src/envoy/utils/grpc_transport.h @@ -49,7 +49,7 @@ class GrpcTransport : public Grpc::TypedAsyncRequestCallbacks, // We generate cluster name contains invalid characters, so override the // authority header temorarily until it can be specified via CDS. // See https://github.com/envoyproxy/envoy/issues/3297 for details. - metadata.Host()->value("mixer"); + metadata.Host()->value("mixer", 5); } void onSuccess(std::unique_ptr&& response,