search

jaredhanson / passport-http

HTTP Basic and Digest authentication strategies for Passport and Node.js.
https://www.passportjs.org/packages/passport-http/?utm_source=github&utm_medium=referral&utm_campaign=passport-http&utm_content=about
MIT License
268 stars 110 forks source link

Info in authenticate callback. #19

Closed nickpoorman closed 10 years ago

nickpoorman commented 10 years ago

This is strange:

passport.authenticate('basic', function(err, user, info) {
    console.log("info is: " + util.inspect(info));
}

Maybe info isn't being passed because it outputs: 'Basic realm="Users"' and when I change info param to anything else if fails silently.

jaredhanson commented 10 years ago

I think this strategy doesn't support info yet. L88 should be changed to:

function verified(err, user, info) {
  if (err) { return self.error(err); }
  if (!user) { return self.fail(self._challenge()); }
  self.success(user, info);
}