fix: avoid filtering by prefix unless necessary

There's no need to filter by the / prefix.

fix: avoid filtering by prefix unless necessary
There's no need to filter by the / prefix.
ec27f419 · Steven Allen · b72ac7e0 · ec27f419
Commit ec27f419 authored Feb 10, 2020 by Steven Allen
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 4 deletions

query/query_impl.go query/query_impl.go +2 -4

No files found.
--- a/query/query_impl.go
+++ b/query/query_impl.go
@@ -129,12 +129,10 @@ func NaiveQueryApply(q Query, qr Results) Results {
 			}
 			prefix = path.Clean(prefix)
 		}
-		// If the prefix isn't "/", end it in a "/" so we only find keys
-		// _under_ the prefix.
+		// If the prefix is empty, ignore it.
 		if prefix != "/" {
-			prefix += "/"
+			qr = NaiveFilter(qr, FilterKeyPrefix{prefix + "/"})
 		}
-		qr = NaiveFilter(qr, FilterKeyPrefix{prefix})
 	}
 	for _, f := range q.Filters {
 		qr = NaiveFilter(qr, f)