delta/node-new.git/src/node_http2.h, branch devsnek-patch-1

http2: allow setting the local window size of a session

2020-11-11T15:14:56+00:00

PR-URL: https://github.com/nodejs/node/pull/35978
Fixes: https://github.com/nodejs/node/issues/31084
Refs: https://github.com/nodejs/node/pull/26962
Reviewed-By: Matteo Collina 
Reviewed-By: Ricky Zhou <0x19951125@gmail.com>

http2: reinject data received before http2 is attached

2020-10-25T10:29:45+00:00

Reinject the data already received from the TLS
socket when the HTTP2 client is attached with a
delay

Fixes: https://github.com/nodejs/node/issues/35475

PR-URL: https://github.com/nodejs/node/pull/35678
Reviewed-By: Anna Henningsen 
Reviewed-By: Matteo Collina 
Reviewed-By: Rich Trott 
Reviewed-By: Alba Mendez 
Reviewed-By: Franziska Hinkelmann 
Reviewed-By: Ricky Zhou <0x19951125@gmail.com>

http2,tls: store WriteWrap using BaseObjectPtr

2020-10-06T11:26:33+00:00

Create weak `WriteWrap` and `ShutdownWrap` objects, and when
referencing them in C++ is necessary, use `BaseObjectPtr<>`
instead of plain pointers to keep these objects from being
garbage-collected.

This solves issues that arise when the underlying `StreamBase`
instance is weak, but the `WriteWrap` or `ShutdownWrap` instances
are not; in that case, they would otherwise potentially stick
around in memory after the stream that they originally belong
to is long gone.

It probably makes sense to use `BaseObjectptr<>` more extensively
in `StreamBase` in the long run as well.

PR-URL: https://github.com/nodejs/node/pull/35488
Reviewed-By: James M Snell 
Reviewed-By: Matteo Collina

http2: add support for sensitive headers

2020-07-15T22:25:47+00:00

Add support for “sensitive”/“never-indexed” HTTP2 headers.

Fixes: https://github.com/nodejs/node/issues/34091

PR-URL: https://github.com/nodejs/node/pull/34145
Reviewed-By: James M Snell 
Reviewed-By: Denys Otrishko

src: extract AllocatedBuffer from env.h

2020-05-30T23:20:00+00:00

Cleanup up env.h by removing things that are not
specific to `Environment`.

PR-URL: https://github.com/nodejs/node/pull/33291
Reviewed-By: Anna Henningsen 
Reviewed-By: David Carlier

http2: refactor and cleanup http2

2020-04-21T20:39:55+00:00

* cleanup constants in http2 binding
  The error constants were just doing some weird things. Cleanup
  and improve maintainability.
* simplify settings to reduce duplicate code
* improve style consistency and correctness
  Use snake_case for getters and setters at c++ level,
  avoid unnecessary use of enums,
  use consistent style for exported vs. internal constants,
  avoid unnecessary memory info reporting,
  use setters/getters for flags_ for improved code readability
* make EmitStatistics function private
* un-nest Http2Settings and Http2Ping
* refactoring and cleanup of Http2Settings and Http2Ping
* avoid ** syntax for readability
  The **session and **stream syntax for getting the underlying
  nghttp2 pointers is not ideal for readability
* use const references for Http2Priority
* remove unnecessary GetStream function
* refactor Http2Scope to use BaseObjectPtr
* move utility function to anonymous namespace
* refactor and simplify Origins
* Use an AllocatedBuffer instead of MaybeStackBuffer
* Use a const reference instead of pointer
* use BaseObjectPtr for Http2Streams map
* move MemoryInfo impl to cc

Signed-off-by: James M Snell 

PR-URL: https://github.com/nodejs/node/pull/32884
Reviewed-By: Matteo Collina 
Reviewed-By: Franziska Hinkelmann

src: add AliasedStruct utility

2020-04-14T00:21:43+00:00

For http2 (and eventually QUIC) we have a struct that is backed
by a v8::BackingStore and exposed to the JavaScript side as an
ArrayBuffer and TypedArray. This is similar to AliasedBuffer
except that it is fronted by a struct on the C++ side.

```c++
struct foo {
  uint32_t ex1;
  uint32_t ex2;
};

AliasedStruct foo_;

foo_->ex1 = 1;
foo_->ex2 = 2;

foo_.GetArrayBuffer();
```

Signed-off-by: James M Snell 

PR-URL: https://github.com/nodejs/node/pull/32778
Reviewed-By: Anna Henningsen 
Reviewed-By: Gerhard Stöbich

src: move HTTP/2 state out of Environment

2020-04-06T00:06:37+00:00

Moves state that is specific to HTTP/2 into the HTTP/2 implementation
as a cleanup.

PR-URL: https://github.com/nodejs/node/pull/32538
Reviewed-By: James M Snell

src: minor http2 refactorings

2020-04-02T16:15:38+00:00

* Simplify Http2Priority struct
* BooleanValue => IsTrue/IsFalse

Signed-off-by: James M Snell 

PR-URL: https://github.com/nodejs/node/pull/32551
Reviewed-By: Anna Henningsen 
Reviewed-By: Rich Trott

src: rename http2 class and suppress compile warnings

2020-04-02T16:15:36+00:00

Suppress compile warnings on Windows, rename class for
consistent styling.

Signed-off-by: James M Snell 

PR-URL: https://github.com/nodejs/node/pull/32551
Reviewed-By: Anna Henningsen 
Reviewed-By: Rich Trott