consul/envoyextensions/extensioncommon
Michael Zalimeni ad03a5d0f2
Avoid panic applying TProxy Envoy extensions (#17537)
When UpstreamEnvoyExtender was introduced, some code was left duplicated
between it and BasicEnvoyExtender. One path in that code panics when a
TProxy listener patch is attempted due to no upstream data in
RuntimeConfig matching the local service (which would only happen in
rare cases).

Instead, we can remove the special handling of upstream VIPs from
BasicEnvoyExtender entirely, greatly simplifying the listener filter
patch code and avoiding the panic. UpstreamEnvoyExtender, which needs
this code to function, is modified to ensure a panic does not occur.

This also fixes a second regression in which the Lua extension was not
applied to TProxy outbound listeners.
2023-06-01 13:04:39 -04:00
..
basic_envoy_extender.go Avoid panic applying TProxy Envoy extensions (#17537) 2023-06-01 13:04:39 -04:00
basic_extension_adapter.go Add Upstream Service Targeting to Property Override Extension (#17517) 2023-05-30 14:53:42 -04:00
envoy_extender.go
envoy_extender_test.go ENT->OSS merge for Consolidate `ListEnvoyExtender` into `BasicEnvoyExtender` (#17491) 2023-05-26 11:10:31 -07:00
resources.go Add Upstream Service Targeting to Property Override Extension (#17517) 2023-05-30 14:53:42 -04:00
resources_test.go ENT->OSS merge for Consolidate `ListEnvoyExtender` into `BasicEnvoyExtender` (#17491) 2023-05-26 11:10:31 -07:00
runtime_config.go Add Upstream Service Targeting to Property Override Extension (#17517) 2023-05-30 14:53:42 -04:00
runtime_config_test.go Add Upstream Service Targeting to Property Override Extension (#17517) 2023-05-30 14:53:42 -04:00
upstream_envoy_extender.go Avoid panic applying TProxy Envoy extensions (#17537) 2023-06-01 13:04:39 -04:00