-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature Failover sink #142
Conversation
- Add failure sink to log events while RabbitMQ is unavailable - Removed AmqpUri support - Refactored the Configuration extensions - Add console app samples projects - replaced hostname config with hostnames - upgrade codecov task to v4
Add readme file as package readme
@sungam3r could you take a look at this PR? |
Add more tests for Configuration Extensions
@@ -1,3 +1,7 @@ | |||
# https://docs.codecov.com/docs/codecov-yaml | |||
|
|||
ignore: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it should be done at coverlet level via exclude filters
src/Serilog.Sinks.RabbitMQ/Sinks/RabbitMQ/EmitEventFailureHandling.cs
Outdated
Show resolved
Hide resolved
src/Serilog.Sinks.RabbitMQ/Sinks/RabbitMQ/EmitEventFailureHandling.cs
Outdated
Show resolved
Hide resolved
src/Serilog.Sinks.RabbitMQ/Sinks/RabbitMQ/RabbitMQConnectionFactory.cs
Outdated
Show resolved
Hide resolved
src/Serilog.Sinks.RabbitMQ/Sinks/RabbitMQ/RabbitMQSinkConfiguration.cs
Outdated
Show resolved
Hide resolved
I looked briefly all the changes. |
Co-authored-by: Ivan Maximov <sungam3r@yandex.ru>
…ling.cs Co-authored-by: Ivan Maximov <sungam3r@yandex.ru>
…ling.cs Co-authored-by: Ivan Maximov <sungam3r@yandex.ru>
…ation.cs Co-authored-by: Ivan Maximov <sungam3r@yandex.ru>
… tests Removed HostNames count in RabbitMQConnectionFactory. There's always 1 host name
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #142 +/- ##
===========================================
+ Coverage 81.86% 96.53% +14.66%
===========================================
Files 9 8 -1
Lines 353 346 -7
Branches 64 41 -23
===========================================
+ Hits 289 334 +45
+ Misses 34 5 -29
+ Partials 30 7 -23 ☔ View full report in Codecov by Sentry. |
Fixes #133