status-im/consul
2
0
Fork 0
mirror of https://github.com/status-im/consul.git synced 2025-01-26 05:29:55 +00:00
Code Issues Projects Releases Wiki Activity

Fix peered service protocols using proxy-defaults.

Browse Source
This commit is contained in:
Derek Menteer 2022-10-27 14:27:07 -05:00 committed by Derek Menteer
parent 218eb3e1ea
commit 1483c94531
1 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
agent/configentry/merge_service_config.go
View File

@ -173,11 +173,21 @@ func MergeServiceConfig(defaults *structs.ServiceConfigResponse, service *struct
us.MeshGateway.Mode = remoteCfg.MeshGateway.Mode
}
preMergeProtocol, found := us.Config["protocol"]
// Merge in everything else that is read from the map
if err := mergo.Merge(&us.Config, remoteCfg.Config); err != nil {
return nil, err
}
// Reset the protocol to its pre-merged version for peering upstreams.
if us.DestinationPeer != "" {
if found {
us.Config["protocol"] = preMergeProtocol
} else {
delete(us.Config, "protocol")
}
}
// Delete the mesh gateway key from opaque config since this is the value that was resolved from
// the servers and NOT the final merged value for this upstream.
// Note that we use the "mesh_gateway" key and not other variants like "MeshGateway" because