status-im/consul
2
0
Fork 0
mirror of https://github.com/status-im/consul.git synced 2025-01-24 12:40:17 +00:00
Code Issues Projects Releases Wiki Activity

Fix subtle loop bug and add test

Browse Source
This commit is contained in:
Paul Banks 2021-09-22 16:05:11 +01:00
parent 07f81991df
commit 70bc89b7f4
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
agent/xds/routes.go
View File

@ -270,8 +270,11 @@ func findIngressServiceMatchingUpstream(l structs.IngressListener, u structs.Ups
return &s return &s
} }
if s.Name == structs.WildcardSpecifier && if s.Name == structs.WildcardSpecifier &&
s.NamespaceOrDefault() == wantSID.NamespaceOrDefault() { s.NamespaceOrDefault() == wantSID.NamespaceOrDefault() &&
foundSameNSWildcard = &s s.PartitionOrDefault() == wantSID.PartitionOrDefault() {
// Make a copy so we don't take a reference to the loop variable
found := s
foundSameNSWildcard = &found
} }
} }
// Didn't find an exact match. Return the wildcard from same service if we // Didn't find an exact match. Return the wildcard from same service if we