Logflare / logflare / 33563286823-1

83%

Build Type

push

github

Committed by web-flow

Commit Message

feat: add ndjson payload format option to webhook backend (#3893)

* feat: add ndjson payload format option to webhook backend

The webhook backend gains a `format` config option. `json` (default)
keeps the current JSON array payload. `ndjson` sends newline-delimited
JSON, one event body per line, with the `application/x-ndjson` content
type. A user-configured `content-type` header takes precedence.

The option is implemented through `transform_config/1`, the existing
mechanism for payload shape and derived headers. The shared Pipeline is
unchanged, and `test_connection/1` sends a format-correct probe.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* fix: address PR review on the webhook ndjson format

- Normalize the stored header keys in `transform_config/1` before the
  content-type is added. A config that holds "Content-Type" no longer ships a
  duplicate header.
- Pattern match on the ndjson format instead of an `if` in `transform_config/1`.
- Reuse `NdjsonFormatter.encode/1` in place of a second implementation.
- Extract `format_payload/2`, shared by `test_connection/1` and
  `Pipeline.handle_batch/4`.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UeRcBGtdgk6tUX5gJohbTB

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

Coverage Stats

15 of 15 new or added lines in 2 files covered. (100.0%)

4 existing lines in 3 files now uncovered.

14657 of 17835 relevant lines covered (82.18%)

2167.42 hits per line