search

slime-io / slime

An intelligent ServiceMesh manager based on Istio
https://slime-io.github.io/
Other
424 stars 78 forks source link

Replace DownstreamRemoteAddress with DownstreamDirectRemoteAddress #376

Closed Patrick0308 closed 1 year ago

Patrick0308 commented 1 year ago

DownstreamDirectRemoteAddress always the physical peer, but DownstreamRemote Address is not when request has XFF headers. See document on https://www.envoyproxy.io/docs/envoy/latest/configuration/observability/access_log/usage

Fix #367

MouceL commented 1 year ago

the DownstreamDirectRemoteAddress was introduced into go-control-plane in v0.9.3 and after version

Patrick0308 commented 1 year ago

The version of slime's go-control-plane is v0.9.9-0.20210217033140-668b12f5399d . It's ok.