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

Meteor.loginWithFacebook not adding email address to user #4743

Closed
KnutSv opened this issue Jul 14, 2015 · 5 comments
Closed

Meteor.loginWithFacebook not adding email address to user #4743

KnutSv opened this issue Jul 14, 2015 · 5 comments

Comments

@KnutSv
Copy link

@KnutSv KnutSv commented Jul 14, 2015

Code for reproducing the issue:
https://github.com/KnutSv/Meteor-facebook-login-email-bug

When the user registers via Facebook the email from Facebook is not added to the user account.

Meteor.loginWithFacebook({requestPermissions: ['email']}, function(error){
  if (error) {
    console.log(error);
  }
});

Possible cause is changes to Facebook API v. 2.4 where fewer fields are returned from graph.
I get the email returned if I add &fields=email to the graph url:
https://graph.facebook.com/v2.4/{{userID}}?access_token={{accessToken}}&fields=name,email

@omeid
Copy link
Contributor

@omeid omeid commented Jul 14, 2015

I can confirm the same issue.

Meteor works with an older App that allows v2.0 of graph but for 2.4 it fails.

omeid added a commit to omeid/meteor that referenced this issue Jul 14, 2015
@tmeasday tmeasday closed this in b6e5780 Jul 14, 2015
@tmeasday
Copy link
Contributor

@tmeasday tmeasday commented Jul 14, 2015

Thanks for the report @KnutSv, see also @glasser's comment on the PR.

We're considering whether on not to do a point release to deal with this.

dgreensp added a commit that referenced this issue Jul 30, 2015
@xpluscal
Copy link

@xpluscal xpluscal commented Aug 1, 2015

Any news on a point release yet?

@dgreensp
Copy link
Contributor

@dgreensp dgreensp commented Aug 1, 2015

Working on it; probably Monday.

@dgreensp
Copy link
Contributor

@dgreensp dgreensp commented Aug 3, 2015

It's now out in 1.1.0.3.

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

Successfully merging a pull request may close this issue.

6 participants