-
Notifications
You must be signed in to change notification settings - Fork 623
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
Build and document strategy for testing with identity providers #3
Comments
I'd like to help with this one |
Do you need to provide the secretKey for a facebook login? On another app I'm working on I'm not using the secretKey just the AppId, which is public. But I'm still running on local emulators. |
Here's javascript code snip it for facebook login using cordovaOauth $cordovaOauth.facebook(appKeys.appId, ["email", "user_friends, user_location, user_posts, user_photos"]).then(function (result) {
console.info("login success");
$localStorage.accessToken = result.access_token;
q.resolve($localStorage.accessToken);
}, function (error) {
console.error(error);
alert("There was a problem signing in! See the console for logs\n" + error);
q.reject(error);
});
return q.promise; |
@tonysurma I think an easy strategy here would be to include a quick readme for setting this up. UserSecrets is pretty easy to get going. I could write this. They would need to, of course, create their own apps/app secret keys to make this work, but that isn't an insurmountable task. |
great thanks! |
Task edit/create with command/query
I believe this has been addressed by the work where we implemented the 3rd party OAuth work. Can @MisterJames or @tonysurma re-open if I'm mistaken. |
as mentioned in the readme the keys for the identify providers will be secrets that are configured in deployment for testing and production.
We need a strategy for how individual contributors can test with identity providers without sharing the 'secret' codes from deployment in production.
The text was updated successfully, but these errors were encountered: