ConduitIO / conduit

Conduit streams data between data stores. Kafka Connect replacement. No JVM required.
https://conduit.io
Apache License 2.0
363 stars 42 forks source link

Webhook processor fixes #1597

Closed lovromazgon closed 1 month ago

lovromazgon commented 1 month ago

Description

Followup to https://github.com/ConduitIO/conduit/pull/1552. The default from the deprecated field now conflicts with any headers.content-type option. This PR removes the default from the field and manually sets the default. It also updates an example to show how headers can be configured.

Additionally, it changes the behavior so a response status <500 is not considered a failing request.

Quick checks:

lovromazgon commented 1 month ago

@raulb @hariso please have a look at the last commit, it allows the usage of Go templates in request.body.