Skip to content

Commit c972ce2

Browse files
rexagodtargos
authored andcommittedJun 2, 2020
http2: comment on usage of Object.create(null)
Refs: #29829 PR-URL: #33183 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent 7817b87 commit c972ce2

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed
 

‎lib/internal/http2/compat.js

+3
Original file line numberDiff line numberDiff line change
@@ -287,6 +287,9 @@ class Http2ServerRequest extends Readable {
287287
closed: false,
288288
didRead: false,
289289
};
290+
// Headers in HTTP/1 are not initialized using Object.create(null) which,
291+
// although preferable, would simply break too much code. Ergo header
292+
// initialization using Object.create(null) in HTTP/2 is intentional.
290293
this[kHeaders] = headers;
291294
this[kRawHeaders] = rawHeaders;
292295
this[kTrailers] = {};

0 commit comments

Comments
 (0)
Please sign in to comment.