fixes example with necessary bodyParser

[jpbochi]

I was following the example in the README that has this middleware.

server.use(function (req, res, next) {
  if (req.method === 'POST') {
    req.body.createdAt = Date.now()
  }
  // Continue to JSON Server router
  next()
})

The rewrite has no effect on the behaviour of the server. One thing I noticed is that req.body was always empty by that time. I noticed that it needed to be parsed before the body editing middleware was set.

This problem might have started to happen after some version of express. I'm using 4.13.4, and the code submitted here works fine for me.

[typicode]

Good catch, thank you 👍
I'll see if it's possible to fix it directly in json-server.

[jpbochi]

I guess that one solution would be to include body-parser as part of jsonServer.defaults().

[typicode]

Yes. The thing I don't know is if there would be some side effects if it's used at the same time in defaults() and router().

[jpbochi]

from what I tested, there was no problem. I'm not sure if express detected that I was repeating a middleware, or if body-parser repeated middleware detected that the body was already parsed and didn't do anything.

It's hard to tell exactly what subtle side-effects could this cause to people that customized json-server. Probably nothing, but you never know what crazy shit people are doing with your open source library. Congrats, by the way. Solid work here.

[typicode]

There seems to be some issue for people using http-proxy-middleware #363

So in v0.9.0, which was just released, body-parser is available using jsonServer.bodyParser and the README has been updated:
https://github.com/typicode/json-server#custom-routes-example