Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

Allow the sandbox decoder to drop messages #1398

Open
trink opened this issue Mar 6, 2015 · 4 comments
Open

Allow the sandbox decoder to drop messages #1398

trink opened this issue Mar 6, 2015 · 4 comments

Comments

@trink
Copy link
Contributor

trink commented Mar 6, 2015

a return of -2 like the encoder

@mattbostock
Copy link
Contributor

What happens currently; could a failure to decode a message cause the plugin channel to fill up?

@trink
Copy link
Contributor Author

trink commented Mar 12, 2015

No, currently you have to return success or failure.

However, in cases like de-duping you really don't want to count/log that as a decode failure just to drop the message and you probably don't want to pass it through to the router with a 'dupe' flag only to be ignored down stream.

@kipras
Copy link

kipras commented Sep 8, 2015

Any progress on this? This would be really helpful, because dropping messages after decoding is necessary sometimes.

@pasha-kuznetsov
Copy link

+1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants