search

hapijs / good

hapi process monitoring
Other
525 stars 160 forks source link

Stream pipeline error #544

Closed gagle closed 7 years ago

gagle commented 7 years ago

What about this?

https://github.com/hapijs/good/blob/master/lib/monitor.js#L129

If the pumpified stream emits an error it silently prints to the console the error. Are there better alternatives like implementing a backoff algorithm and try to recreate the pipline?

arb commented 7 years ago

That sounds like adding a ton of complexity for something that won't happen very often. That error handler should only pop if there is a like a node core stream and pipe error.

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.