Some refactoring. What is nicer?

code.js

var passport = require('passport');
var BasicStrategy = require('passport-http').BasicStrategy;
var config = require('./config');

passport.use(new BasicStrategy(function(user, pass, done) {
  if (user !== config.auth.user) {
    return done(null, false, {
      message: 'Unknown user ' + user
    });
  }
  if (pass !== config.auth.pass) {
    return done(null, false, {
      message: 'Invalid password'
    });
  }
  done(null, config.auth);
}));

// sends 401 if unauthorized
module.exports = passport.authenticate('basic', { session: false });

refactored-code.js

var passport = require('passport');
var BasicStrategy = require('passport-http').BasicStrategy;
var config = require('./config');


var strategy = new BasicStrategy(function (user, password, done) {

  userMatches(user, done) && passMatches(password, done) && done(null, config.auth);

});


passport.use(strategy);


function userMatches (user, done) { 
  if (user === config.auth.user) return true;
  var err = { message: 'Unknown user ' + user };
  done(null, false, err);
}


function passMatches (password, done) {
  if (password === config.auth.password) return true;
  var err = { message: 'Invalid password' };
  done(null, false, err);
}


// sends 401 if unauthorized
module.exports = passport.authenticate('basic', { session: false });