-
Notifications
You must be signed in to change notification settings - Fork 3
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.
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
Update deps and instructions for hapi v.16 scrub #7
Changes from all commits
e725b63
a953e0e
90f9e67
bb70ac9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,2 @@ | ||
node_modules | ||
package-lock.json |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,5 +3,6 @@ language: node_js | |
node_js: | ||
- "4" | ||
- "6" | ||
- "8.7.0" | ||
|
||
after_script: "npm run coveralls" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
## API | ||
### The plugin | ||
You should register Underdog in any plugin that would like to take advantage of its features; it does not take any options. Underdog specifies the `once` [plugin attribute](http://hapijs.com/api#plugins), which means hapi will ensure it is not registered multiple times to the same connection. | ||
|
||
Note, Underdog utilizes an `onPreResponse` extension to finalize server-push, so if you modify the response in an `onPreResponse` extension and would like that response to push additional resources, you must specify `{ before: 'underdog' }` as an option when calling [`server.ext()`](http://hapijs.com/api#serverextevent-method-options). | ||
|
||
### `reply.push([response], path, [headers])` | ||
- `response` - the hapi [response](http://hapijs.com/api#response-object) for which you'd like to push additional resources. Should not be an error response. When this argument is omitted Underdog will attempt to use the request's currently set response, which will fail with an error if there is no such response. | ||
- `path` - the path to another resource living on the same connection as the current request that you'd like to server-push. If the path is relative (does not begin with `/`) then it is adjusted based upon the relevant [realm's](http://hapijs.com/api#serverrealm) route prefix. | ||
- `headers` - any headers that you would like to add to the push-request for this resource. By default a user-agent header is automatically added to match the user-agent of the originating request. | ||
|
||
Note that when push-requests are resolved credentials from the originating request will be used, and that routes marked as `isInternal` are accessible. | ||
|
||
## Extras | ||
- The HTTP/2 spec ([here](http://httpwg.org/specs/rfc7540.html)) | ||
- The `http2` node module ([here](https://github.com/molnarg/node-http2)) | ||
- The `spdy` (also implementing HTTP/2) node module ([here](https://github.com/indutny/node-spdy)) | ||
- For debugging HTTP/2 in Chrome, see `chrome://net-internals/#http2` |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -486,7 +486,7 @@ describe('Underdog', () => { | |
srv.stop(done); | ||
}); | ||
|
||
request.on('push', () => next(new Error('Should not make it here'))); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Returning an error wont do anything! You'd have to at least throw it. I think that There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ay, gotcha. thanks for pointing that out. slopslopslop. fixing now! |
||
request.on('push', () => done(new Error('Should not make it here'))); | ||
}); | ||
}); | ||
|
||
|
@@ -531,7 +531,7 @@ describe('Underdog', () => { | |
srv.stop(done); | ||
}); | ||
|
||
request.on('push', () => next(new Error('Should not make it here'))); | ||
request.on('push', () => done(new Error('Should not make it here'))); | ||
}); | ||
}); | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this version exactly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ohhh totally missed your commit message, sorry! Wow!! I would definitely have expected that to be a semver major!
nodejs/node#15685
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh man, yea. took me a bit to peek through their changelog to sort that out. 🎺