Skip to content

Conversation

jsvd
Copy link
Member

@jsvd jsvd commented Jul 21, 2025

Adds two new headers to each bulk request:

  • "X-Elastic-Event-Count": number of actions / documents in that bulk request
  • "X-Elastic-Uncompressed-Request-Length": size in bytes of the request body before compression

X-Elastic-Uncompressed-Request-Length is equal to Content-Length when compression is disabled.

aligns with elastic/beats#45146

also renews test certificates that were out-of-date.

@jsvd jsvd requested a review from robbavey July 22, 2025 16:00
Copy link
Contributor

@robbavey robbavey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good to me - I added a comment on Travis test scopes, which can be resolved here, or in a separate PR

@jsvd jsvd force-pushed the add_uncompressed_headers branch from 0f3f770 to 5e9c9e6 Compare July 29, 2025 09:45
@jsvd jsvd force-pushed the add_uncompressed_headers branch from 5e9c9e6 to 3fafb3f Compare July 29, 2025 14:11
@jsvd jsvd merged commit 8eac7ce into logstash-plugins:main Jul 29, 2025
2 of 3 checks passed
@jsvd jsvd deleted the add_uncompressed_headers branch July 29, 2025 14:12
jsvd added a commit to jsvd/logstash-output-elasticsearch that referenced this pull request Jul 30, 2025
…ts (logstash-plugins#1217) Adds two new headers to each bulk request: * "X-Elastic-Event-Count": number of actions / documents in that bulk request * "X-Elastic-Uncompressed-Request-Length": size in bytes of the request body before compression X-Elastic-Uncompressed-Request-Length is equal to Content-Length when compression is disabled.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants