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

Request: unify authentication implementation so they can be combined #48

Open
kbrandwijk opened this issue Aug 3, 2017 · 2 comments
Open

Comments

@kbrandwijk
Copy link
Contributor

It would be great if it was easy to combine multiple custom authentication methods into one project. For this, it's necessary to unify the implementations, to make sure none of them use the same query or mutation names, or the same fields in the User Type, etc.

@marktani
Copy link
Contributor

marktani commented Aug 5, 2017

I don't think we should expect different examples to be compatible to each other with no changes.

Instead, I'd like to see a new example that shows how to combine two authentication methods and offers additional functionality (like allowing to authenticate via email/password as well as Facebook for the same user if the email address matches).

@kbrandwijk
Copy link
Contributor Author

No not without changes. The idea was that we might come up with a sort of 'template' for implementing an authentication provider, so that it's easier to combine them.

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

No branches or pull requests

2 participants