search

developmentseed / jekyll-hook

No Longer Maintained | A server that listens for GitHub webhook posts and renders a Jekyll site
BSD 3-Clause "New" or "Revised" License
508 stars 83 forks source link

Fix req.body.payload and double-parsing of JSON #15

Closed johndmulhausen closed 10 years ago

johndmulhausen commented 10 years ago

a) req.body.payload doesn't work anymore (JSON is directly in req.body) b) The payload doesn't need to be JSON parsed - it's being double-parsed here, causing "Unexpected token" errors

parkr commented 10 years ago

Which versions of Node cause this error?

NicolasLM commented 10 years ago

I have the same issue with nodejs v0.10.25 from Ubuntu 14.04 repository. This pull-request fixes the problem.

scisco commented 10 years ago

The issue was resolved here: c5b18e5b5786f03d5d395854cde0241bd27244e0