proxycfg: reference to entry in map should not panic

This commit is contained in:
Nitya Dhanushkodi 2021-06-17 10:43:33 -07:00
parent 41ec132fce
commit 52043830b4
2 changed files with 8 additions and 10 deletions

3
.changelog/10423.txt Normal file
View File

@ -0,0 +1,3 @@
```release-note:bug
proxycfg: reference to entry in map should not panic.
```

View File

@ -846,8 +846,7 @@ func (s *state) handleUpdateConnectProxy(ctx context.Context, u cache.UpdateEven
// Clean up data from services that were not in the update // Clean up data from services that were not in the update
for sn := range snap.ConnectProxy.WatchedUpstreams { for sn := range snap.ConnectProxy.WatchedUpstreams {
upstream := snap.ConnectProxy.UpstreamConfig[sn] if upstream, ok := snap.ConnectProxy.UpstreamConfig[sn]; ok && upstream.Datacenter != "" && upstream.Datacenter != s.source.Datacenter {
if upstream.Datacenter != "" && upstream.Datacenter != s.source.Datacenter {
continue continue
} }
if _, ok := seenServices[sn]; !ok { if _, ok := seenServices[sn]; !ok {
@ -855,8 +854,7 @@ func (s *state) handleUpdateConnectProxy(ctx context.Context, u cache.UpdateEven
} }
} }
for sn := range snap.ConnectProxy.WatchedUpstreamEndpoints { for sn := range snap.ConnectProxy.WatchedUpstreamEndpoints {
upstream := snap.ConnectProxy.UpstreamConfig[sn] if upstream, ok := snap.ConnectProxy.UpstreamConfig[sn]; ok && upstream.Datacenter != "" && upstream.Datacenter != s.source.Datacenter {
if upstream.Datacenter != "" && upstream.Datacenter != s.source.Datacenter {
continue continue
} }
if _, ok := seenServices[sn]; !ok { if _, ok := seenServices[sn]; !ok {
@ -864,8 +862,7 @@ func (s *state) handleUpdateConnectProxy(ctx context.Context, u cache.UpdateEven
} }
} }
for sn := range snap.ConnectProxy.WatchedGateways { for sn := range snap.ConnectProxy.WatchedGateways {
upstream := snap.ConnectProxy.UpstreamConfig[sn] if upstream, ok := snap.ConnectProxy.UpstreamConfig[sn]; ok && upstream.Datacenter != "" && upstream.Datacenter != s.source.Datacenter {
if upstream.Datacenter != "" && upstream.Datacenter != s.source.Datacenter {
continue continue
} }
if _, ok := seenServices[sn]; !ok { if _, ok := seenServices[sn]; !ok {
@ -873,8 +870,7 @@ func (s *state) handleUpdateConnectProxy(ctx context.Context, u cache.UpdateEven
} }
} }
for sn := range snap.ConnectProxy.WatchedGatewayEndpoints { for sn := range snap.ConnectProxy.WatchedGatewayEndpoints {
upstream := snap.ConnectProxy.UpstreamConfig[sn] if upstream, ok := snap.ConnectProxy.UpstreamConfig[sn]; ok && upstream.Datacenter != "" && upstream.Datacenter != s.source.Datacenter {
if upstream.Datacenter != "" && upstream.Datacenter != s.source.Datacenter {
continue continue
} }
if _, ok := seenServices[sn]; !ok { if _, ok := seenServices[sn]; !ok {
@ -882,8 +878,7 @@ func (s *state) handleUpdateConnectProxy(ctx context.Context, u cache.UpdateEven
} }
} }
for sn, cancelFn := range snap.ConnectProxy.WatchedDiscoveryChains { for sn, cancelFn := range snap.ConnectProxy.WatchedDiscoveryChains {
upstream := snap.ConnectProxy.UpstreamConfig[sn] if upstream, ok := snap.ConnectProxy.UpstreamConfig[sn]; ok && upstream.Datacenter != "" && upstream.Datacenter != s.source.Datacenter {
if upstream.Datacenter != "" && upstream.Datacenter != s.source.Datacenter {
continue continue
} }
if _, ok := seenServices[sn]; !ok { if _, ok := seenServices[sn]; !ok {