-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Avoid context canceled errors #1789
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1789 +/- ##
==========================================
+ Coverage 89.60% 89.63% +0.02%
==========================================
Files 32 32
Lines 2656 2663 +7
==========================================
+ Hits 2380 2387 +7
Misses 178 178
Partials 98 98
Continue to review full report at Codecov.
|
591dc97
to
f5707c8
Compare
f5707c8
to
0d908d9
Compare
b13f4a7
to
216ee0e
Compare
216ee0e
to
d8c05b4
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.
Looks good. Thanks for the updates.
Approved from my side. @aldas please merge, if there are no more comments from your side.
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
d8c05b4
to
d3e0a0b
Compare
Return 499 Client Closed Request when the client has closed the request before the server could send a response Signed-off-by: Seena Fallah <seenafallah@gmail.com>
d3e0a0b
to
01fbe2a
Compare
Return 499 Client Closed Request when the client has closed the request before the server could send a response
Signed-off-by: Seena Fallah seenafallah@gmail.com