HTTP2 server options must be an object #19

New Issue

Ghost · 2019-11-26T16:55:52Z

Ghost commented

2019-11-26 16:55:52 +00:00

In example, the glx.http2Server method appears to be documented with the incorrect signature.

var http2Server = glx.http2Server(function(req, res) {})

is wrong. It should probably be

var http2Server = glx.http2Server({ ...tlsOptions }, function (req, res) {})

internal/http2/util.js:513
    throw new ERR_INVALID_ARG_TYPE(name, types || 'Object', value);
    ^

TypeError [ERR_INVALID_ARG_TYPE]: The "options" argument must be of type Object. Received type function

What TLS options are supposed to be provided to the HTTP2 server?

[example](https://git.rootprojects.org/root/greenlock-express.js/src/branch/master/examples/http2/server.js#L20) In example, the `glx.http2Server` method appears to be documented with the incorrect signature. ```javascript var http2Server = glx.http2Server(function(req, res) {}) ``` is wrong. It should probably be ```javascript var http2Server = glx.http2Server({ ...tlsOptions }, function (req, res) {}) ``` ``` internal/http2/util.js:513 throw new ERR_INVALID_ARG_TYPE(name, types || 'Object', value); ^ TypeError [ERR_INVALID_ARG_TYPE]: The "options" argument must be of type Object. Received type function ``` What TLS options are supposed to be provided to the HTTP2 server?

coolaj86 commented

2019-11-26 17:01:52 +00:00

An empty object {} or null should do just fine.

Would you mind making a PR for the docs?

An empty object `{}` or `null` should do just fine. Would you mind making a PR for the docs?

Ghost commented

2020-08-05 15:28:32 +00:00

@solderjs I think you fixed this one.

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: root/greenlock-express.js#19