Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Connector ignore invalid transformation configuration #4003

Closed
sehz opened this issue May 11, 2024 · 1 comment · Fixed by #4004
Closed

Bug: Connector ignore invalid transformation configuration #4003

sehz opened this issue May 11, 2024 · 1 comment · Fixed by #4004
Assignees
Labels
bug Something isn't working features/connector

Comments

@sehz
Copy link
Contributor

sehz commented May 11, 2024

Steps to reproduce. If any transforms section contains invalid configuration like:

# config-example.yaml
apiVersion: 0.1.0
meta:
  version: 0.3.3
  name: cat-facts
  type: http-source
  topic: cat-facts
  create-topic: true
http:
  endpoint: "https://catfact.ninja/fact"
  interval: 10s
transforms:
  - uses: infinyon/jolt@0.1.0
    with:
      -   spec:
             - operation: default
               spec:
                source: "http-connector"
             - operation: remove
                spec:
                 length: ""

connector silently drops transform section which cause user to think transform is not working when in fact transform is not even applied.

@sehz sehz added bug Something isn't working features/connector labels May 11, 2024
@morenol
Copy link
Contributor

morenol commented May 11, 2024

We could use something like what was suggested here:

serde-rs/json#447

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working features/connector
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants