Skip to content

Commit

Permalink
Ensure monitoring/_bulk route is fully aggregated
Browse files Browse the repository at this point in the history
This route still exists in the code even if it is two major version old.
We should still support it properly with our streaming changes.
  • Loading branch information
Tim-Brooks committed Sep 20, 2024
1 parent e257d93 commit 407a48b
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -374,7 +374,9 @@ protected HttpMessage createMessage(String[] initialLine) throws Exception {
// combines the HTTP message pieces into a single full HTTP request (with headers and body)
final HttpObjectAggregator aggregator = new Netty4HttpAggregator(
handlingSettings.maxContentLength(),
httpPreRequest -> enabled.get() == false || (httpPreRequest.rawPath().endsWith("/_bulk") == false)
httpPreRequest -> enabled.get() == false
|| ((httpPreRequest.rawPath().endsWith("/_bulk") == false)
|| httpPreRequest.rawPath().startsWith("/_xpack/monitoring/_bulk"))
);
aggregator.setMaxCumulationBufferComponents(transport.maxCompositeBufferComponents);
ch.pipeline()
Expand Down

0 comments on commit 407a48b

Please sign in to comment.