Estoy tratando de hacer lo siguiente. Tengo lo sgts endpoints.
/
/secret
/user/login //Post-Get
/user/logout //Post
/user/register //Post-Get
/user/recuperar //Post-Get
Tengo 2 middlewares unlogged y logged
Aqui estan mis rutas:
router.use(logged.unlogged);
router.use("/logout",logged.logged);
router.route("/login").get(login.login_get).post(login.login_post);
router.route("/recuperar").get(recuperar.recuperar_get).post(recuperar.recuperar_post);
router.route("/registrar").get(registrar.registrar_get).post(registrar.registrar_post);
router.route("/logout").get(logout);
Quiero que me dea acceso a esos endpoints cuando no esta logeado , y cuando esta logeado no me permita el acceso a los mismos, por eso pongo esos 2 middlewares.
Estos son mis middlewares
var logged=function(req,res,next){
if(req.session.user_id!=undefined){
next();
}
else{
console.log(req.session);
res.redirect("/");
}
}
var unlogged=function(req,res,next){
if(req.session.user_id!=undefined) res.redirect("/user/login");
else{
next();
}
}
module.exports={
logged:logged,
unlogged:unlogged
}
Una ves que entro a login y pongo mis password y contraseña, encuentra al usuario y deberia redirigirme al endpoint /password el cual solo seactiva si esta logeado la persona.
User.findOne({userName:req.body.userName,password:req.body.password},function(err,us){
if(err)console.log(String(err));
console.log("Usuario encontra\n",us._id);
if(us){
req.session.user_id=us._id;
console.log("Se redirigira a password");
res.redirect("/password");
}
res.redirect("/");
});
Pero me sale el siguiente error
Can't set headers after ther are sent, pero no lo entiendo bien , dice que no puedo cambiar los headers luego de que los envie ,no veo mi error.
Ahora , no sé si la forma en la que estoy haciendo lo que quiero es la correcta , u hay otra forma más ligera para hacer lo que quiero.
De stackoverflow en ingles ,lo que entendi es posiblemente el error estea en el middleware ya que res.redirect modifica las cabeceras ,pero no me queda del todo claro.
EDICION El error se me quito poniendole un return a todos los redirect involucrados , pero no me queda claro porque es necesario el return , además deberia ponerle en todos,otra cuestión es si hay una manera mas fácil de ejecutar lo que trato de hacer.
return
forzas a que se detenga la ejecución del middleware y no siga paralelamente a los mapeos de las rutas.