Skip to content

V10.17.0 proposal #29150

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
BethGriggs wants to merge 19 commits into from
Closed

V10.17.0 proposal #29150

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
00f153d
http2: improve http2 code a bit
jasnell Oct 30, 2018
a0a14c8
src: pass along errors from http2 object creation
addaleax Jan 30, 2019
ab0f2ac
deps: update nghttp2 to 1.37.0
gengjiawen Mar 29, 2019
fedfa12
deps: update nghttp2 to 1.38.0
gengjiawen Apr 18, 2019
a397c88
deps: update nghttp2 to 1.39.1
gengjiawen Jun 27, 2019
74507fa
deps: update nghttp2 to 1.39.2
addaleax Aug 14, 2019
76a7ada
http2: improve JS-side debug logging
addaleax Aug 5, 2019
2eb914f
http2: only call into JS when necessary for session events
addaleax Aug 9, 2019
7f11465
http2: do not create ArrayBuffers when no DATA received
addaleax Aug 9, 2019
05dada4
http2: limit number of rejected stream openings
addaleax Aug 10, 2019
477461a
http2: limit number of invalid incoming frames
addaleax Aug 12, 2019
f4242e2
http2: handle 0-length headers better
addaleax Aug 10, 2019
460f896
http2: shrink default `vector::reserve()` allocations
addaleax Aug 10, 2019
17357d3
http2: consider 0-length non-end DATA frames an error
addaleax Aug 10, 2019
0ce699c
http2: stop reading from socket if writes are in progress
addaleax Aug 10, 2019
c152449
http2: pause input processing if sending output
addaleax Aug 11, 2019
0acbe05
http2: allow security revert for Ping/Settings Flood
addaleax Aug 12, 2019
d85e400
test: apply test-http2-max-session-memory-leak from v12.x
addaleax Aug 13, 2019
8994fff
2019-08-15, Version 10.17.0 'Dubnium' (LTS)
BethGriggs Aug 15, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
http2: shrink default vector::reserve() allocations 
Allocating memory upfront comes with overhead, and in particular,
`std::vector` implementations do not necessarily return memory
to the system when one might expect that (e.g. after shrinking the
vector).

Backport-PR-URL: #29123
PR-URL: #29122
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
@addaleax @BethGriggs
addaleax authored and BethGriggs committed Aug 15, 2019
commit 460f896c631f1be52b0ab6c9f30e0b66f601b2a1
4 changes: 2 additions & 2 deletions src/node_http2.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -632,7 +632,7 @@ Http2Session::Http2Session(Environment* env,
// fails.
CHECK_EQ(fn(&session_, callbacks, this, *opts, *allocator_info), 0);

outgoing_storage_.reserve(4096);
outgoing_storage_.reserve(1024);
outgoing_buffers_.reserve(32);

{
Expand Down Expand Up @@ -1947,7 +1947,7 @@ Http2Stream::Http2Stream(Http2Session* session,
if (max_header_pairs_ == 0) {
max_header_pairs_ = DEFAULT_MAX_HEADER_LIST_PAIRS;
}
current_headers_.reserve(max_header_pairs_);
current_headers_.reserve(std::min(max_header_pairs_, 12u));

// Limit the number of header octets
max_header_length_ =
Expand Down