Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Link to ooth, a user accounts library built on top of passport.js #621

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

nickredmark
Copy link

Passport.js is a great solution for authentication for node.js. A concern that passport.js delegates to its users is what to do with the received credentials (e.g. emails, usernames, passwords). Usually people solve this by copying some snippets of code online. But this isn't a trivial issue. Apart from the inefficiency of people reimplementing the same thing and from security risks such as storing passwords in plain text, there are many tricky user account flow details that people usually don't think about such as logging in with different strategies (what if I log in with facebook with an email that I already used to log in with a local strategy?). These are the things solved by ooth. Ooth, just like passport.js, is modular and builds on top of passport.js strategies. With 144 commits since February it moved out of alpha today to 1.0. In this pull request I link Ooth as a linked library. I believe it will solve many people a headache.

@coveralls
Copy link

coveralls commented Nov 13, 2017

Coverage Status

Coverage remained the same at 98.87% when pulling 668303d on nmaro:master into 821a474 on jaredhanson:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants