Express4.10 bodyParser req.body undefined - node.js

Question

I'm trying to build the login of a node application trying to access the route / login get: req.body undefined
Error:
TypeError: Cannot read property 'usuario' of undefined
at login (/home/makros/workspace/ntalk/controllers/home.js:8:24)
at Layer.handle [as handle_request] (/home/makros/workspace/ntalk/node_modules/express/lib/router/layer.js:82:5)
at next (/home/makros/workspace/ntalk/node_modules/express/lib/router/route.js:100:13)
at Route.dispatch (/home/makros/workspace/ntalk/node_modules/express/lib/router/route.js:81:3)
at Layer.handle [as handle_request] (/home/makros/workspace/ntalk/node_modules/express/lib/router/layer.js:82:5)
at /home/makros/workspace/ntalk/node_modules/express/lib/router/index.js:235:24
at Function.proto.process_params (/home/makros/workspace/ntalk/node_modules/express/lib/router/index.js:313:12)
at /home/makros/workspace/ntalk/node_modules/express/lib/router/index.js:229:12
at Function.match_layer (/home/makros/workspace/ntalk/node_modules/express/lib/router/index.js:296:3)
at next (/home/makros/workspace/ntalk/node_modules/express/lib/router/index.js:190:10)
app.js
var express = require('express'),
load = require('express-load'),
cookieParser = require('cookie-parser'),
session = require('express-session'),
bodyParser = require('body-parser'),
app = express();
load('models')
.then('controllers')
.then('routes')
.into(app);
app.set('views', __dirname + '/views');
app.set('view engine', 'ejs');
app.use(cookieParser('secret'));
app.use(session({
secret: 'secret',
resave: true,
saveUninitialized: true
}));
app.use(bodyParser.urlencoded({ extended: false }));
app.use(bodyParser.json());
app.use(express.static(__dirname + '/public'));
app.listen(3000,function(){
console.log('Started!');
});
package.json
{
"name": "ntalk",
"version": "1.0.0",
"private": true,
"scripts": {
"start": "node ./bin/www"
},
"dependencies": {
"body-parser": "^1.9.0",
"cookie-parser": "^1.3.3",
"ejs": "~0.8.5",
"express": ">= 0.0.0",
"express-load": "^1.1.14",
"express-session": "^1.8.2"
}
}
controllers/home.js
module.exports = function(app) {
return {
index: function(req, res) {
res.render('home/index');
},
login: function(req, res){
console.log(req.params);
var email = req.body.usuario.email,
nome = req.body.usuario.nome;
if(email && nome) {
var usuario = req.body.usuario;
usuario['contatos'] = [];
req.session.usuario = usuario;
res.redirect('/contatos');
} else {
res.redirect('/');
}
},
logout: function(req, res){
req.session.destroy();
res.redirect('/');
}
};
};
routes/home.js
module.exports = function(app) {
var home = app.controllers.home;
app.get('/', home.index);
app.get ('/entrar', home.login);
app.get('/sair', home.logout);
};
views/home/index.ejs
<% include ../header %>
<header>
<h1>Ntalk</h1>
<h4>Bem-vindo!</h4>
</header>
<section>
<form action="/entrar" method="post">
<input type="text" name="usuario[nome]" placeholder="Seu nome">
<br>
<input type="text" name="usuario[email]" placeholder="Seu e-mail">
<br>
<button type="submit">Entrar</button>
</form>
</section>
<% include ../footer %>
Please help me!

Answer 1

You're adding your route handlers to app before your body parsing middleware. All routes/middleware/etc. added to app are executed in order in Express 4.
So move this:
load('models')
.then('controllers')
.then('routes')
.into(app);
after this line:
app.use(express.static(__dirname + '/public'));
On an unrelated note, you typically should place your express.static() usage near the top of your route/middleware list for efficiency. This prevents automatic session creation and unnecessary cookie parsing for static asset requests for example.

Answer 2

try move
app.use(bodyParser.urlencoded({ extended: false }));
app.use(bodyParser.json());
before
load('models')
.then('controllers')
.then('routes')
.into(app);

Answer 3

In fact the problem was in my implementation of express-load, it injects dependencies but does not configure the app, so I decided not to use it:
app.js
var express = require('express'),
path = require('path'),
cookieParser = require('cookie-parser'),
session = require('express-session'),
bodyParser = require('body-parser');
var routes = require('./routes/home'),
contacts = require('./routes/contacts');
var app = express();
app.set('views', path.join(__dirname, 'views'));
app.set('view engine', 'ejs');
app.use(bodyParser.json());
app.use(bodyParser.urlencoded({ extended: false }));
app.use(cookieParser());
app.use(session({
secret: 'secret',
resave: true,
saveUninitialized: true
}));
app.use(express.static(path.join(__dirname, 'public')));
app.use('/', routes);
app.use('/contatos', contacts);
app.listen(3000,function(){
console.log('Started!');
});
routes/home.js
var router = require('express').Router();
var home = require('../controllers/home');
router.get('/', home.index);
router.post('/entrar', home.login);
router.get('/sair', home.logout);
module.exports = router;
controllers/home.js
module.exports = {
index: function(req, res) {
/*do somethings*/
},
login: function(req, res){
/*do somethings*/
},
logout: function(req, res){
/*do somethings*/
}
};
There must be a better way to implement using express-load with express4, but not yet found.