passport.js authentication

Open quarhodron opened this issue 6 years ago • 1 comments

Hi!

What was the reason that you dropped the idea to authenticate with the help of passport.js (I saw your old branch)?

@alex996

Oct 05 '19 13:10 quarhodron

Hi @quarhodron, and sorry for a late reply. I didn't see much benefit for this repo in using Passport. Also, one thing I didn't approve of is how deserializeUser is invoked on every authenticated request, even when you're only fetching e.g. messages or chats and don't need the user record at all. Perhaps I missed something and there's a way around that?

Oct 30 '19 22:10 alex996