[Bug] bodyParser configuration does not work

[jiayisheji]

I'm submitting a...

[ ] Regression 
[x] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

I use NestApplicationOptions to turn on the bodyParser and cannot get the body

Expected behavior

I can get the body

Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

import { NestFactory } from '@nestjs/core';
import { AppModule } from './app.module';

async function bootstrap() {
  const app = await NestFactory.create(AppModule, {
    bodyParser: true,
  });
  app.use((req, res, next) => {
    console.log('req.body', req.body);
    next();
  });
  await app.listen(3000);
}
bootstrap();

Print the result:

req.body undefined

import { NestFactory } from '@nestjs/core';
import { urlencoded, json } from 'body-parser';
import { AppModule } from './app.module';

async function bootstrap() {
  const app = await NestFactory.create(AppModule);
  app.use(urlencoded({ extended: true }));
  app.use(json());
  app.use((req, res, next) => {
    console.log('req.body', req.body);
    next();
  });
  await app.listen(3000);
}
bootstrap();

Print the result:

req.body {}

Environment

[System Information]
OS Version     : Windows 10
NodeJS Version : v8.11.1
NPM Version    : 5.6.0
[Nest Information]
microservices version : 5.3.2
websockets version    : 5.3.1
mongoose version      : 5.2.1
passport version      : 5.0.1
testing version       : 5.3.1
common version        : 5.3.2
core version          : 5.3.3

Others:

[whtiehack]

https://docs.nestjs.com/controllers @Body

[jiayisheji]

@whtiehack We think differently

[kamilmysliwiec]

Hi @jiayisheji,
You need to call init().

const app = await NestFactory.create(AppModule, {
    bodyParser: true,
});
await app.init();
app.use((req, res, next) => {
    console.log('req.body', req.body);
    next();
});
await app.listen(3000);

[kamilmysliwiec]

By the way, bodyParser is enabled by default, you don't have to explicitly pass true.

[jiayisheji]

@kamilmysliwiec Thank you

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.