Skip to content

Commit 991b61c

Browse files
committed
[Squash] Update test todo comment
1 parent 27cebcc commit 991b61c

File tree

1 file changed

+4
-9
lines changed

1 file changed

+4
-9
lines changed

test/parallel/test-http2-too-many-streams.js

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -53,15 +53,10 @@ server.listen(0, common.mustCall(() => {
5353
{
5454
const req = client.request();
5555
req.resume();
56-
// TODO(jasnell): Still investigating precisely why, but on Windows,
57-
// the error is not emitted because the underlying
58-
// mechanism ensures that only one request is sent
59-
// at a time per the maxConcurrentStreams setting.
60-
// This likely has to do with the way the response
61-
// is being handled on the server side. This is safe
62-
// to ignore on Windows because of the assert.strictEqual
63-
// check in the on('stream') handler which ensures that
64-
// only one request is being handled at any given time.
56+
// TODO(jasnell): On Windows, the error is not emitted because the
57+
// underlying mechanism ensures that only one request is sent at a time
58+
// per the maxConcurrentStreams setting. Need to investigate why the
59+
// same is not happening on posix systems.
6560
if (!common.isWindows) {
6661
req.on('error', common.expectsError({
6762
code: 'ERR_HTTP2_STREAM_ERROR',

0 commit comments

Comments
 (0)