🐛 (catalogd) `serveJSON` lines instead of `http.serverContent` for no-params by anik120 · Pull Request #1725 · operator-framework/operator-controller
joelanford
changed the title
(catalogd)
🐛 (catalogd) serveJSON lines instead of http.serverContent for no-paramsserveJSON lines instead of http.serverContent for no-params
…arams The metas endpoint was using `serverJSONLines` for serving queries that are parameterized, which copies content from a reader to the response writer under the hood. As a result no-parameterized query responses don't have range request support. The endpoint was however using `http.ServeContent` for cases when no parameters were provided, which does have range request support. This PR switches the `http.ServeContent` out for `serveJSONLines` to make sure metas endpoint is consistent in it's lack of support for range requests. Signed-off-by: Anik Bhattacharjee <anbhatta@redhat.com>
Merged
via the queue into
operator-framework:main
with commit f6b1130
20 of 21 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters