KorAP / Kalamar

:octopus: Mojolicious-based Frontend for KorAP
BSD 2-Clause "Simplified" License
7 stars 2 forks source link

Fix totalResults handling when Kustvakt returns cache results #219

Closed Akron closed 1 month ago

Akron commented 2 months ago

Kustvakt 0.71.1 introduced a totalResult cache, that rewrites the KoralQuery in the way that is not handled by Kalamar correctly. It introduces a cutoff but returns a totalResult. Currently Kalamar ignores the totalResults for cutoff queries. It's debatable if Kustvakt should make the cached results indistinguishable from non-cached results and don't change the cutoff.

This shoud probably also be discussed with other client devs.

Akron commented 1 month ago

This was a misinterpretation of what Kustvakt does, so I'll remove this. Actually it returns totalResults even when a cutoff is send.