pipedream icon indicating copy to clipboard operation
pipedream copied to clipboard

[FEATURE] `$.send.http` should throw a `ConfigurationError` if `body` instead of `data` is passed

Open ctrlaltdylan opened this issue 3 years ago • 0 comments

Is your feature request related to a problem? Please describe.

When using $.send.http to send HTTP requests in workflows, it's very easy to forget that data is the correct key for payloads, and not body.

Example:

// this request fails silently, the request is never made.
await $.send.http({
        url: "https://<endpoint>.m.pipedream.net/",
        method: 'POST',
        body: item
      });

Currently if you forget this, the HTTP request isn't sent - but it fails silently.

This leads to a very bad DX. Developers are unsure why their requests aren't working.

Describe the solution you'd like $.send.http should throw a ConfigurationError if body instead of data is passed.

Additional context

Example affecting a user: https://pipedream-users.slack.com/archives/CPTJYRY5A/p1659026266899159?thread_ts=1659023776.746629&cid=CPTJYRY5A

ctrlaltdylan avatar Jul 28 '22 17:07 ctrlaltdylan