diff --git a/source/common/quic/envoy_quic_stream.h b/source/common/quic/envoy_quic_stream.h index 504b74934df1..bbe4759339ae 100644 --- a/source/common/quic/envoy_quic_stream.h +++ b/source/common/quic/envoy_quic_stream.h @@ -184,7 +184,6 @@ class EnvoyQuicStream : public virtual Http::StreamEncoder, std::unique_ptr metadataMapFromHeaderList(const quic::QuicHeaderList& header_list); - // Returns true if the cumulative limit on METADATA headers has been reached // after adding `bytes`. bool mustRejectMetadata(size_t bytes) { diff --git a/test/integration/multiplexed_integration_test.cc b/test/integration/multiplexed_integration_test.cc index 033431f4a7e1..40f2e997a691 100644 --- a/test/integration/multiplexed_integration_test.cc +++ b/test/integration/multiplexed_integration_test.cc @@ -293,7 +293,6 @@ class Http2MetadataIntegrationTest : public HttpProtocolIntegrationTest { protocol_options.mutable_explicit_http_config() ->mutable_http2_protocol_options() ->set_allow_metadata(true); - } ConfigHelper::setProtocolOptions( *bootstrap.mutable_static_resources()->mutable_clusters(0), protocol_options); @@ -302,7 +301,8 @@ class Http2MetadataIntegrationTest : public HttpProtocolIntegrationTest { [&](envoy::extensions::filters::network::http_connection_manager::v3::HttpConnectionManager& hcm) -> void { hcm.mutable_http2_protocol_options()->set_allow_metadata(true); - hcm.mutable_http3_protocol_options()->set_allow_metadata(true); }); + hcm.mutable_http3_protocol_options()->set_allow_metadata(true); + }); } void testRequestMetadataWithStopAllFilter(); @@ -311,9 +311,7 @@ class Http2MetadataIntegrationTest : public HttpProtocolIntegrationTest { void runHeaderOnlyTest(bool send_request_body, size_t body_size); - Http::CodecClient::Type upstreamProtocol() { - return GetParam().upstream_protocol; - } + Http::CodecClient::Type upstreamProtocol() { return GetParam().upstream_protocol; } protected: // Utility function to prepend filters. Note that the filters @@ -590,13 +588,13 @@ TEST_P(Http2MetadataIntegrationTest, TestResponseMetadata) { EXPECT_EQ(4, response->metadataMapsDecodedCount()); // Upstream responds with headers, 100-continue and data. - response = - codec_client_->makeRequestWithBody(Http::TestRequestHeaderMapImpl{{":method", "GET"}, - {":path", "/dynamo/url"}, - {":scheme", "http"}, - {":authority", "sni.lyft.com"}, - {"expect", "100-contINUE"}}, - 10); + response = codec_client_->makeRequestWithBody( + Http::TestRequestHeaderMapImpl{{":method", "GET"}, + {":path", "/dynamo/url"}, + {":scheme", "http"}, + {":authority", "sni.lyft.com"}, + {"expect", "100-contINUE"}}, + 10); waitForNextUpstreamRequest(); upstream_request_->encode1xxHeaders(Http::TestResponseHeaderMapImpl{{":status", "100"}});