Skookum/passport — comment on commit 3751d2917d08eb64a6e04f2f68f782e3ba8fd98b
File:  line 0
By hunterloftis:

Jared,

Thanks for the feedback - after making this branch to just get something done quickly, I tried passing (req, res) to the .authenticate() method and it works great. What threw me off at first was an example in the comments that shows the callback but is missing the (req, res), which of course doesn't work:

https://github.com/jaredhanson/passport/blob/master/lib/passport/middleware/authenticate.js#L35

Digging passport so far, and I'd love to work with you on it. I've tried several other node auth solutions and they are all too heavy and require a particular auth pattern and implementation changes. I like that passport just does one thing well and gets out of your way.

A couple questions:

- Is there a way to request more permissions from facebook beyond just logging in? There's a 'profile' object passed back after fb auth, but I don't see anywhere to actually set the fb connect permissions ('email', 'info', etc).

- What is the serialization for? I've included it just as you have in your code, but I don't understand the need for serialization and deserialization, so I think I'm misunderstanding some Passport features.

cheers,
Hunter