search

logstash-plugins / logstash-input-syslog

Apache License 2.0
37 stars 38 forks source link

return instead of killing entire process #33

Closed nneul closed 4 years ago

nneul commented 8 years ago

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

jordansissel commented 8 years ago

Let's figure out the cause of these crashes and fix that. It could helpme if you focus only on one plugin for this until we have a solution. Otherwise I get lost in the noise seeing so many PRs for the same thing across plugins.

On Sat, Nov 19, 2016 at 1:40 PM Nathan Neulinger notifications@github.com wrote:

Thanks for contributing to Logstash! If you haven't already signed our

CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

You can view, comment on, or merge this pull request online at:

https://github.com/logstash-plugins/logstash-input-syslog/pull/33 Commit Summary

  • return instead of killing entire process

File Changes

Patch Links:

- https://github.com/logstash-plugins/logstash-input-syslog/pull/33.patch

https://github.com/logstash-plugins/logstash-input-syslog/pull/33.diff

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/logstash-plugins/logstash-input-syslog/pull/33, or mute the thread https://github.com/notifications/unsubscribe-auth/AAIC6sg35R25k2NM9gC_4y5Pr-X6cJoDks5q_2zVgaJpZM4K3X_i .

nneul commented 8 years ago

ok, commenting on other PR with clarification.