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
Propagate and honor http.ErrAbortHandler #82588
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: liggitt The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
f1bebcd
to
c0e66fc
Compare
Am fine with the direction. |
c0e66fc
to
b3640ee
Compare
b3640ee
to
40e4366
Compare
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.
One minor suggestion then lgtm.
40e4366
to
f174cf8
Compare
updated godocs, added tests |
/priority critical-urgent |
/lgtm |
f174cf8
to
4341529
Compare
New changes are detected. LGTM label has been removed. |
fixed spelling error ( |
…8-upstream-release-1.16 Automated cherry pick of #82588: Propagate and honor http.ErrAbortHandler
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Alternative to #82586
Fixes #82591
Prevents excessive log spam when proxied connections are closed by the backend.
The http stdlib uses a propagated panic as a sentinel value to indicate the panic is expected and no stack trace should be logged. This PR makes our panic-logging handlers honor that documented sentinel value.
Does this PR introduce a user-facing change?:
/sig api-machinery
/cc @deads2k @sttts