server-side-rendering demo pacakge.json npm install error

[stursby]

npm install in the server-side-rendering demo isn't working. I believe it's the trailing , on line 12. Removing it fixed it for me.

My new package.json

{
  "name": "server-side-rendering",
  "version": "1.0.0",
  "main": "index.js",
  "dependencies": {
    "express": "^4.14.0",
    "handlebars": "^4.0.5",
    "spdy": "^3.4.0",
    "http2": "^3.3.5",
    "mz": "^2.4.0"
  },
  "devDependencies": {}
}

[surma]

🙄 Thanks

[stursby]

No problem! Did you get to the bottom of spdy still serving up HTTP/1 issue? I cloned the repo, booted up the server and also got HTTP/1 (using Node 4).

[surma]

Yeah I also tweeted about it. Node 4 is the issue. It works just fine with
node 5. Get nvm :)

On Thu, Sep 15, 2016, 20:47 charlie hield notifications@github.com wrote:

No problem! Did you get to the bottom of the spdy still serving up HTTP/1
issue? I cloned the repo, booted up the server and also got HTTP/1 :|
(using Node 4).

—
You are receiving this because you modified the open/close state.

Reply to this email directly, view it on GitHub
#50 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOVzWYWJM78J_va7wvqx3rX0O1AEYK3ks5qqaDcgaJpZM4J-P2K
.