http: coerce content-length to number

PR-URL: #57458 Fixes: #57456 Reviewed-By: Tim Perry <pimterry@gmail.com> Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

Author: marco-ippolito

lib/_http_outgoing.js

@@ -606,7 +606,7 @@ function matchHeader(self, state, field, value) {
  break;
  case 'content-length':
  state.contLen = true;
- self._contentLength = value;
+ self._contentLength = +value;
  self._removedContLen = false;
  break;
  case 'date':

test/parallel/test-http-content-length-mismatch.js

@@ -78,3 +78,23 @@ function shouldThrowOnFewerBytes() {
 shouldThrowOnMoreBytes();
 shouldNotThrow();
 shouldThrowOnFewerBytes();
+
+
+{
+ const server = http.createServer(common.mustCall((req, res) => {
+ res.strictContentLength = true;
+ // Pass content-length as string
+ res.setHeader('content-length', '5');
+ res.end('12345');
+ }));
+
+
+ server.listen(0, common.mustCall(() => {
+ http.get({ port: server.address().port }, common.mustCall((res) => {
+ res.resume().on('end', common.mustCall(() => {
+ assert.strictEqual(res.statusCode, 200);
+ server.close();
+ }));
+ }));
+ }));
+}