clue · GitHub

Merged

Merged

Conversation

// Content-Length value is not an integer or not a single integer
$this->emit('error', new \Exception('The value of `Content-Length` is not valid'));
return;
}
}
}

if ($request->hasHeader('Content-Length')) {
// stream must emit an 'end' here, because empty data won't be emitted
$stream->emit('end');
$conn->removeListener('data', array($stream, 'handleData'));
}

@legionth

if ($request->hasHeader('Content-Length')) {
// remove 'Content-Length' header accordig to: https://tools.ietf.org/html/rfc7230#section-3.3.3
$headers = $request->getHeaders();
unset($headers['Content-Length']);
$request->getQueryParams(),
$request->getProtocolVersion(),
$headers
);
}

$stream = new LengthLimitedStream($conn, $contentLength);
}
if (!$this->closed) {
$this->emit('end');
$this->close();
}
$request->on('end', $endEvent);
$request->on('close', $closeEvent);
$request->on('error', $errorEvent);
});

@legionth

@legionth

clue

clue approved these changes Feb 21, 2017

jsor

jsor approved these changes Feb 21, 2017

WyriHaximus

@WyriHaximus

Merged

@clue clue mentioned this pull request

Mar 7, 2017

Closed

Read the original on github.com ↗