Fix error at bootstrapping when RabbitMQ server is down

ArieGato / serilog-sinks-rabbitmq

Serilog Sink for RabbitMq

Apache License 2.0

53 stars 51 forks source link

Fix error at bootstrapping when RabbitMQ server is down #77

Closed huan086 closed 5 years ago

huan086 commented 5 years ago

Fixes #55. Initialize RabbitMQ connection only on the first call to RabbitMQClient.Publish. This follows the same pattern used in Microsoft.Extensions.Caching.Redis.

huan086 commented 5 years ago

@tsvetelintsonev can you review?

tsvetelintsonev commented 5 years ago

@huan086 I can review it tomorrow :)

madslyng commented 5 years ago

If naming is the only objection to this PR. Let's OK it, and we can have a talk about naming convention.