0

I tried to implement it somehow but the problem is but It couldn't allow access to existing users, that is on the login system?

*The signup system works perfect.

Thanks in advance.

This is what my signup.jade look like:

head

title

    | Side Project | Signup

body

 form(action='/signup', method='post')

  div

    label Username :

    input(type='text', name='username')

    br

  div

    label Password :

    input(type='password', name='password')

  div

    input(type='submit', value='Submit')

| #{note}

This is what my login.jade look like:

    form(action='/login', method='post')

      div

        label Username:

        input(type='text', name='username')

        br

      div

        label Password :

        input(type='password', name='password')

      div

        input(type='submit', value='Submit')

    | #{note}

This is what my app.js file look like :

   var express = require('express');

   var path = require('path');

   var favicon = require('static-favicon');

   var logger = require('morgan');

   var cookieParser = require('cookie-parser');

   var bodyParser = require('body-parser');

   var routes = require('./routes/index');

   var users = require('./routes/users');

   var app = express();

   //monogDB

   var UserDetails = require('./models/data');


   //Passport.js setup

   var passport = require('passport');

   var LocalStrategy = require('passport-local').Strategy;

   app.use(passport.initialize());

   app.use(passport.session());

   //  view engine setup

   app.set('views', path.join(__dirname, 'views'));

   app.set('view engine', 'jade');


   app.use(favicon());

   app.use(logger('dev'));

   app.use(bodyParser.json());

   app.use(bodyParser.urlencoded());

   app.use(cookieParser());

   app.use(express.static(path.join(__dirname, 'public')));


   app.use('/', routes);

   app.use('/users', users);

   /// catch 404 and forward to error handler

   app.use(function(req, res, next) {

    var err = new Error('Not Found');

    err.status = 404;

    next(err);

   });



   /// error handlers

   // development error handler

   // will print stacktrace

   if (app.get('env') === 'development') {

       app.use(function(err, req, res, next) {

           res.status(err.status || 500);

           res.render('error', {

                message: err.message,

                error: err

           });

       });

   }

   // production error handler

   // no stacktraces leaked to user

   app.use(function(err, req, res, next) {

       res.status(err.status || 500);

       res.render('error', {

           message: err.message,

           error: {}

       });

   });

   //Passport.js

   //Login

   passport.serializeUser(function(user, done) {

     done(null, user);

   });

   passport.use(new LocalStrategy(function(username, password, done) {

     process.nextTick(function() {

       UserDetails.findOne({

         'username': username, 

       }, function(err, user) {

         if (err) {

           return done(err);

          }


         if (!user) {

           return done(null, false);

         }

         if (user.password != password) {

           return done(null, false);

         }

          return done(null, user);

       });

     });

   }));

   passport.deserializeUser(function(user, done) {

     done(null, user);

   });

   //Sign up

   passport.serializeUser(function(user, done) {

     done(null, user);

   });

   passport.deserializeUser(function(user, done) {

     done(null, user);

   });

   passport.use(new LocalStrategy(function(newusername, newpassword, done) {

     process.nextTick(function() {

        UserDetails.findOne({

         'username': newusername, 

       }, function(err, user) {

         if (err) {

           return done(err);

         }       

         if (!user) {

             var newUser = new UserDetails({"username":newusername, "password":newpassword});

             newUser.save(function (err, newUser) {

               if (err) 

               {

                   return console.error(err);

               }

               console.log("Document Saved!");

               });

            return done(null, true);

         }

       if(user){

           return done(null,false);

       }


         return done(null, user);

       });

     });

    }));
2
  • 1
    First of all, what you should do is : - improve formating. It's just unreadable - only show the part of your code that doesn't work/that's related to your issue. For example, we do not need your error handlers here. However, we do need to see how you authenticate users (ie where passport.authenticate is called). - shorten your title Aug 29, 2014 at 8:48
  • Alright, Thanks a lot. I will improve such problems next time. Sep 4, 2014 at 4:02

1 Answer 1

0

Regarding your code issue, I think it's because you have two different but unnamed Local Strategies. Thus, the second overwrites the first one.

You can give names to strategies, by doing : passport.use('local-signup',new LocalStrategy(..)) and passport.use('local-signin',new LocalStrategy(..)) .

Then, in your router, you can call the specific strategy by passing its name as the first parameter of passport.authenticate().

For example,

app.get('/signin', passport.authenticate('local-signin',...);
app.get('/signup', passport.authenticate('local-signup',...);
0

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.