http2: fix res.setHeader to update linkedlist APIs

PR-URL: nodejs#75 Reviewed-By: James M Snell <[email protected]>
jasnell · Jun 22, 2017 · e7db81d · e7db81d
1 parent 26021ca
commit e7db81d
Show file tree

Hide file tree

Showing 2 changed files with 53 additions and 3 deletions.
diff --git a/lib/internal/http2/compat.js b/lib/internal/http2/compat.js
@@ -36,7 +36,8 @@ function setHeader(list, name, value) {
 function llistToHeaders(list, count) {
   var ret = {};
   while (!linkedList.isEmpty(list)) {
-    var item = linkedList.shift(list);
+    var item = list._idlePrev;
+    linkedList.remove(item);
     var key = item[0];
 
     if (ret[key]) {
@@ -279,8 +280,18 @@ class Http2ServerResponse extends Stream {
     };
     this[kStream] = stream;
     stream[kResponse] = this;
-    this[kHeaders] = linkedList.create();
-    this[kTrailers] = linkedList.create();
+    var headerList = {
+      _idleNext: null,
+      _idlePrev: null,
+    };
+    linkedList.init(headerList);
+    this[kHeaders] = headerList;
+    var trailersList = {
+      _idleNext: null,
+      _idlePrev: null,
+    };
+    linkedList.init(trailersList);
+    this[kHeaders] = trailersList;
     this.writable = true;
     stream.on('drain', onStreamResponseDrain);
     stream.on('error', onStreamResponseError);

diff --git a/test/parallel/test-http2-server-set-header.js b/test/parallel/test-http2-server-set-header.js
@@ -0,0 +1,39 @@
+'use strict';
+
+const common = require('../common');
+const assert = require('assert');
+const http2 = require('http2');
+const path = require('path');
+const tls = require('tls');
+const net = require('net');
+const fs = require('fs');
+const body =
+  '<html><head></head><body><h1>this is some data</h2></body></html>';
+
+const server = http2.createServer((req, res) => {
+  res.setHeader('foobar', 'baz');
+  res.setHeader('X-POWERED-BY', 'node-test');
+  res.end(body);
+});
+
+server.listen(0, common.mustCall(() => {
+  const client = http2.connect(`http://localhost:${server.address().port}`);
+  const headers = { ':path': '/' };
+  const req = client.request(headers);
+  req.setEncoding('utf8');
+  req.on('response', common.mustCall(function(headers) {
+    assert.strictEqual(headers['foobar'], 'baz');
+    assert.strictEqual(headers['x-powered-by'], 'node-test');
+  }));
+
+  let data = '';
+  req.on('data', (d) => data += d);
+  req.on('end', () => {
+    assert.strictEqual(body, data);
+    server.close();
+    client.destroy();
+  });
+  req.end();
+}));
+
+server.on('error', common.mustNotCall());