Skip to content
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

Jump to bottom

http2: add origin frame support #22956

Closed
wants to merge 5 commits into from
Closed

http2: add origin frame support #22956

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
8b8c2a9
http2: add origin frame support
jasnell Sep 17, 2018
1b16710
[Squash] nit
jasnell Sep 19, 2018
05c063a
[Squash] nits
jasnell Sep 19, 2018
bfe03b4
[Squash] nit
jasnell Sep 19, 2018
5311da5
[Squash] nit
jasnell Sep 19, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 2 additions & 2 deletions doc/api/errors.md
View file
Open in desktop
Expand Up @@ -923,7 +923,7 @@ status codes must be an integer between `100` and `199` (inclusive).
<a id="ERR_HTTP2_INVALID_ORIGIN"></a>
### ERR_HTTP2_INVALID_ORIGIN

HTTP/2 ORIGIN frames require a valid origin.
HTTP/2 `ORIGIN` frames require a valid origin.

<a id="ERR_HTTP2_INVALID_PACKED_SETTINGS_LENGTH"></a>
### ERR_HTTP2_INVALID_PACKED_SETTINGS_LENGTH
Expand Down Expand Up @@ -978,7 +978,7 @@ socket attached to an `Http2Session`.
<a id="ERR_HTTP2_ORIGIN_LENGTH"></a>
### ERR_HTTP2_ORIGIN_LENGTH

HTTP/2 ORIGIN frames are limited to a length of 16382 bytes.
HTTP/2 `ORIGIN` frames are limited to a length of 16382 bytes.

<a id="ERR_HTTP2_OUT_OF_STREAMS"></a>
### ERR_HTTP2_OUT_OF_STREAMS
Expand Down
12 changes: 8 additions & 4 deletions doc/api/http2.md
View file
Open in desktop
Expand Up @@ -677,7 +677,8 @@ are passed through as provided by the user or received from the peer.
added: REPLACEME
-->

* `origins` { string } One or more URL Strings passed as separate arguments.
* `origins` { string | URL | Object } One or more URL Strings passed as
separate arguments.

Submits an `ORIGIN` frame (as defined by [RFC 8336][]) to the connected client
to advertise the set of origins for which the server is capable of providing
Expand Down Expand Up @@ -708,13 +709,12 @@ used. The value of the `origin` property *must* be a properly serialized
ASCII origin.

Alternatively, the `origins` option may be used when creating a new HTTP/2
server using either the `http2.createServer()` or `http2.createSecureServer()`
methods:
server using the `http2.createSecureServer()` method:

```js
const http2 = require('http2');
const options = getSecureOptionsSomehow();
options.origins = ['http://example.com', 'http://example.org'];
options.origins = ['http://example.com', 'https://example.org'];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more http)

const server = http2.createSecureServer(options);
server.on('stream', (stream) => {
stream.respond();
Expand Down Expand Up @@ -1990,6 +1990,10 @@ server.listen(80);
<!-- YAML
added: v8.4.0
changes:
- version: REPLACEME
pr-url: https://github.com/nodejs/node/pull/22956
description: Added the `origins` option to automatically send an ORIGIN
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

`ORIGIN`

frame on Http2Session startup.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

`Http2Session`

- version: v8.9.3
pr-url: https://github.com/nodejs/node/pull/17105
description: Added the `maxOutstandingPings` option with a default limit of
Expand Down