From e9800501f0d3cda54e69a63b0951195f05bb75e7 Mon Sep 17 00:00:00 2001 From: James Peach Date: Tue, 18 May 2021 16:57:15 +1000 Subject: [PATCH] quic: reduce socket option header exposure Move the inclusion of `socket_option_impl.h` from the header to the source to reduce its exposure. Signed-off-by: James Peach --- source/common/quic/active_quic_listener.cc | 5 +++-- source/common/quic/active_quic_listener.h | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/source/common/quic/active_quic_listener.cc b/source/common/quic/active_quic_listener.cc index 2421b70e168b4..ff1f8d2c64624 100644 --- a/source/common/quic/active_quic_listener.cc +++ b/source/common/quic/active_quic_listener.cc @@ -8,15 +8,16 @@ #include -#include "common/runtime/runtime_features.h" #include "common/http/utility.h" +#include "common/network/socket_option_impl.h" #include "common/quic/envoy_quic_alarm_factory.h" #include "common/quic/envoy_quic_connection_helper.h" #include "common/quic/envoy_quic_dispatcher.h" +#include "common/quic/envoy_quic_packet_writer.h" #include "common/quic/envoy_quic_proof_source.h" #include "common/quic/envoy_quic_utils.h" -#include "common/quic/envoy_quic_packet_writer.h" #include "common/quic/envoy_quic_utils.h" +#include "common/runtime/runtime_features.h" namespace Envoy { namespace Quic { diff --git a/source/common/quic/active_quic_listener.h b/source/common/quic/active_quic_listener.h index a923afe09acbe..fb3f1b78cf4c6 100644 --- a/source/common/quic/active_quic_listener.h +++ b/source/common/quic/active_quic_listener.h @@ -3,9 +3,9 @@ #include "envoy/config/listener/v3/quic_config.pb.h" #include "envoy/network/connection_handler.h" #include "envoy/network/listener.h" +#include "envoy/network/socket.h" #include "envoy/runtime/runtime.h" -#include "common/network/socket_option_impl.h" #include "common/protobuf/utility.h" #include "common/quic/envoy_quic_dispatcher.h" #include "common/runtime/runtime_protos.h"