Skip to content

feat: implement working Elasticsearch gzip compression #2708

Description

@strawgate

Problem

Current compress: true config option does nothing - no Content-Encoding header is set and body is not actually compressed. Commit 3c60f84 disabled it after bug #2625.

Why

Users enabling compression expect data to be compressed, but ES receives uncompressed body with no header.

Requirements

  1. Add gzip compression in send.rs (use existing GzEncoder code)
  2. Add Content-Encoding: gzip header when config.compress == true
  3. Preserve buffer reuse optimization from original design
  4. Test against real Elasticsearch

Files

  • crates/ffwd-output/src/elasticsearch/send.rs - add header
  • crates/ffwd-output/src/elasticsearch/transport.rs - already passes body

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions