search

logstash-plugins / logstash-integration-rabbitmq

Logstash Integration Plugin for RabbitMQ, including Logstash Input and Output Plugins
Apache License 2.0
5 stars 25 forks source link

Undeclared exchange recovery #21

Closed yaauie closed 4 years ago

yaauie commented 4 years ago

Revival of https://github.com/logstash-plugins/logstash-input-rabbitmq/pull/118

Resolves: https://github.com/logstash-plugins/logstash-integration-rabbitmq/issues/10 Resolves: https://github.com/logstash-plugins/logstash-integration-rabbitmq/issues/11

colinsurprenant commented 4 years ago

LGTM minus minor comment in the reset! method.

yaauie commented 4 years ago

I've added a simple rescue -> log-and-squash clause for exceptions in Inputs::RabbitMQ#reset!, rebased, and have a green build.