-
Notifications
You must be signed in to change notification settings - Fork 9
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
fix: catch ClosedByInterruptException and handle it at debug/info #212
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #212 +/- ##
==========================================
- Coverage 86.35% 85.03% -1.32%
==========================================
Files 13 13
Lines 1033 1049 +16
Branches 106 108 +2
==========================================
Hits 892 892
- Misses 87 99 +12
- Partials 54 58 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
Unit Tests Coverage Report
Minimum allowed coverage is Generated by 🐒 cobertura-action against 8b3e8d7 |
Integration Tests Coverage Report
Minimum allowed coverage is Generated by 🐒 cobertura-action against 8b3e8d7 |
Issue #, if available:
Description of changes:
Handle
ClosedByInterruptException
instead of onlyIOException
to avoid unhelpful logs at too high of a level.Why is this change necessary:
How was this change tested:
Any additional information or context required to review the change:
Checklist:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.