#415 Missing body for POST request in middleware

[JHotterbeekx]

As described in issue 415, the body of post requests was no longer available in middlewares, since version 0.9.0.

The issue was caused by the removal of the 'body-parser' from the server/defaults.js. After re-adding this parser there, the body became available again in the middleware. I've also added an extra test, to verify that the post body is indeed available in the middleware, to prevent future changes to break this again,

[crazy4groovy]

nitpick: lowcase "s" in "should"?

[JHotterbeekx]

I've updated the line, and also updated the code to the new standards. Is it okay to get accepted now? I'd love to have this in the next release.

[JHotterbeekx]

Any chance to accept this pull request so we can move on from version 0.8.23?

[typicode]

Hi

Sorry for the delay, and thank you for the PR, AFAIR I think it was removed on purpose, because it was conflicting in some cases when the project was used as a module in another Express app (can't remember the issue).

If the idea is to make it available to middlewares loaded by the CLI, would it be possible to make it an option of jsonServer.defaults, like { bodyParser: true }, and enable it only in the CLI?

[JHotterbeekx]

I've just updated the code that I think you meant for it to be, could you take a look at it?

[typicode]

Thank you!

[andys8]

@typicode Do you know when these changes will be released?

[typicode]

@andys8 Thanks for the reminder, today :)

[typicode]

Released 👍

[andys8]

Awesome, @typicode !

[crazy4groovy]

So to be clear, I'm using it in this way:

const middlewares = jsonServer.defaults({ bodyParser: true }); server.use(middlewares);