Skip to content

If there is an error in transform -> pass error to callback#46

Open
ZJONSSON wants to merge 2 commits intoironSource:masterfrom
ZJONSSON:stream-emit-error
Open

If there is an error in transform -> pass error to callback#46
ZJONSSON wants to merge 2 commits intoironSource:masterfrom
ZJONSSON:stream-emit-error

Conversation

@ZJONSSON
Copy link
Contributor

@ZJONSSON ZJONSSON commented Feb 2, 2018

closes #45

The error will in turn be emitted as an error event

@ZJONSSON ZJONSSON changed the title If there is an error in transform, pass error to callback If there is an error in transform -> pass error to callback Feb 2, 2018
The error will in turn be emitted as an `error` event
…llback are captured subsequently and included in the callback.
@ed-sparkes
Copy link

Any chance this can be merged?

@emranuddin75
Copy link

Its a useful change to merge in, and would be very much appreciated to capture the rejection rather than silently dying.
Let me now when that would likely to be merged so that i can harden the code.

@samsullivan
Copy link

samsullivan commented Sep 1, 2020

Ideally errors would be Error objects, not a string; but, I agree, anything merged would be a welcomed improvement.

@cnd4
Copy link

cnd4 commented Aug 24, 2021

Hello, I am facing an issue with the unhandled rejection, is it possible to merge this PR?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UnhandledPromiseRejectionWarning on error

5 participants