-
Notifications
You must be signed in to change notification settings - Fork 38
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
[SDCISA-13736, SDCISA-10974] Fix bad habits in HttpResourceStorage. #539
[SDCISA-13736, SDCISA-10974] Fix bad habits in HttpResourceStorage. #539
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #539 +/- ##
=============================================
- Coverage 47.70% 47.68% -0.02%
- Complexity 1784 1785 +1
=============================================
Files 225 225
Lines 11704 11713 +9
Branches 1227 1225 -2
=============================================
+ Hits 5583 5585 +2
- Misses 5634 5640 +6
- Partials 487 488 +1 ☔ View full report in Codecov by Sentry. |
log.warn("Failed request to {}: {}", path, asyncResult.cause()); | ||
log.warn("Failed request to {}", path, new Exception("stacktrace", asyncResult.cause())); | ||
log.error/*TODO*/("TODO why do we NOT report this error to bodyHandler?!? Is this" | ||
+ " maybe the reason for all our mysterious timeouts everywhere? (ERR_jGQCALsBAgBJeA)"); |
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.
We should not add logs for personal bug hunting to the codebase. Just build your own release locally and add the logs there.
log.warn("Failed request to {}: {}", uri, asyncResult.cause()); | ||
log.warn("Failed request to {}", uri, new Exception("stacktrace", asyncResult.cause())); | ||
log.error/*TODO*/("TODO why do we not report this error to our caller?!? Is this maybe" | ||
+ " the explanation for all our unexplainable timeouts or lost messages we" |
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.
We should not add logs for personal bug hunting to the codebase. Just build your own release locally and add the logs there
No description provided.