search

metal-stack / metal-lib

Common go packages used across the metal-stack microservices
MIT License
2 stars 1 forks source link

Bug in for loop for meilisearch List Function #152

Closed Honigeintopf closed 1 month ago

Honigeintopf commented 1 month ago

I've noticed that in the Search method, the search queries are overwriting the indexuid. This happens because the same SearchRequest instance reqProto is reused for each index, causing all queries to end up with the IndexUID of the last index, which eventually leads to metalctl audit lsto never list anything than the last index.

https://github.com/metal-stack/metal-lib/blob/045c5070575095e2c892268bc93821327c9aab27/auditing/meilisearch.go#L195-L202

Honigeintopf commented 1 month ago

Fixed it already, but can't push in this repo yet.