search

cloudamqp / lavinmq

Lightweight and fast AMQP (0-9-1) server
https://lavinmq.com
Apache License 2.0
582 stars 33 forks source link

Dont log handled exception as error with backtrace #776

Closed spuun closed 1 month ago

spuun commented 2 months ago

WHAT is this pull request doing?

This will change log level and remove backtrace when we log errors in the shovel. Since the errors are handled and the shovel should reconnect warn should be enough.

This PR needs cloudamqp/amqp-client.cr#47 to get rid of all scary logging.

HOW can this pull request be tested?

Run a shovel and close the connection from mgmt ui.

spuun commented 1 month ago

Note that this won't take fully effect until we've released a new version of amqp-client.cr.