New issue
Advanced search Search tips

Issue 844045 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: May 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 844031



Sign in to add a comment

Embedded Test Server doesn't handle send errors

Project Member Reported by mef@chromium.org, May 17 2018

Issue description

The test server in //net/test/embedded_test_server doesn't check the result of send operation expecting that connection will be closed anyway in |done| callback.

This is not true for exabyte response, which posts new |send| callbacks instead.
 

Comment 1 by mef@chromium.org, May 17 2018

Blocking: 844031

Comment 2 by mef@chromium.org, May 21 2018

Status: WontFix (was: Started)
The problem that I was seeing was fixed by https://chromium-review.googlesource.com/c/chromium/src/+/1064395

I've got https://chromium-review.googlesource.com/c/chromium/src/+/1064496 in the works, but it doesn't fix the problem at hand AND breaks too many tests by changing callback signature to accept rv code.

Sign in to add a comment