luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

Whoops. Return an empty set in the event that there are multiple matches.

This commit is contained in:
Sean Chittenden 2017-02-01 15:18:00 -08:00
parent 324215c842
commit b241cfc7fd
No known key found for this signature in database
GPG Key ID: 4EBC9DC16C2E5E16
1 changed files with 25 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
consul/state/catalog.go
View File

@ -694,26 +694,34 @@ func (s *StateStore) NodeServices(ws memdb.WatchSet, nodeNameOrID string) (uint6
return 0, nil, fmt.Errorf("node lookup failed: %s", err)
}
if n == nil {
if len(nodeNameOrID) >= minUUIDLookupLen {
if n != nil {
ws.Add(watchCh)
} else {
if len(nodeNameOrID) < minUUIDLookupLen {
ws.Add(watchCh)
return 0, nil, nil
}
// Attempt to lookup the node by it's node ID
var idWatchCh <-chan struct{}
idWatchCh, n, err = tx.FirstWatch("nodes", "uuid_prefix", nodeNameOrID)
iter, err := tx.Get("nodes", "uuid_prefix", nodeNameOrID)
if err != nil {
return 0, nil, fmt.Errorf("node ID lookup failed: %s", err)
}
n = iter.Next()
if n == nil {
ws.Add(watchCh)
return 0, nil, nil
} else {
ws.Add(idWatchCh)
}
} else {
idWatchCh := iter.WatchCh()
if iter.Next() != nil {
// Watch on the channel that did a node name lookup if we don't find
// anything when searching by Node ID.
ws.Add(watchCh)
return 0, nil, nil
}
} else {
ws.Add(watchCh)
ws.Add(idWatchCh)
}
node := n.(*structs.Node)