-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Option: Email address confirmation not required when using social account #145
Comments
Unfortunately, it is not that trivial.
I'll keep this ticket open. TODO:
|
But if an option would be defined one could choose. Maybe on a per site basis? |
What do you mean? This is not something optional. If you set e-mail verification to mandatory then e-mails must be verified. If Facebook does not guarantee verified e-mails then the current behavior (sending out e-mail verifications) is correct. Making this optional somehow defeats the purpose of e-mail verification. In that case, simply turn e-mail verification off. |
I understand that email addresses from Facebook are not really verified while those from Google are. So you are probably right that only the Google addresss should be marked as verified. |
I am not very fond of adding a setting that breaks the semantics of the e-mail verified flag. If stopping fake accounts is your goal, then please consider putting a captcha on your signup form. Making the user jump through hoops like handling verification mails when you are not really after a verified e-mail address does not seem very user friendly. If you insist, we could pass along the social login to |
mm, I think most websites with social signup work this way?: either you signup using an email account that you have to verify, or you sign up using a social networkign account. django-allauth is the only place this is different that I know about, so I don't think this is such a strange thing to e wondering about. There are two parts to this as far a I can see: One part is that email addresses that are verified already are treated as if they were not. This is the case with Gmail at least and the current behavior of verifying gmail-addresses twice seems broken. In the case of Facebook, I would think someone would have to recheck the claim that they show nonverified emails. Maybe they changed some things around, but I have at least not been able to sign up for an acocunt and get it verified without clicking on the verification link. I could not find the part about a verified Faceook account not having a verified email address in the Facebook develoepr documentation either. I would really like to know how to do this. The second part is to have an option to only require a verified email if you do traditional signup, which is what I gather from the above. I haven't looked enough at the allauth code to say where one could do this, but if you say that the place to do this is in the adapter, then I would be all for this. Thanks for taking the time looking into this! |
The reason I am cautious about the semantics of the verified flag is that there are production sites up and running that rely on it to be correct. For example, users that have a verified e-mail matching a certain domain automatically gain extra privileges. So it involves potential security problems. It is still not clear to me why you are using email verification in the first place? As for the points you make:
|
As for the first three points we agree. As for the fourth point: A captcha
|
Just checking: suppose someone signs up with Twitter. No e-mail address is handed over by Twitter, so the user will need to fill it in after authenticating with Twitter. I assume that in this case the user has already shown that his account is not purely fake -- there is a valid Twitter account after all. So, there is no need for e-mail verification to kick in. Agreed? This would mean that the requirement is not to stamp the e-mail address as "verified", but rather that you need some form of control over whether or not e-mail verification is to kick in in the first place... |
That specific use case would be us, correct. But additionally, say one
|
I just checked the Facebook case, The documentation over at https://developers.facebook.com/docs/reference/api/user/ clearly mentions:
So verified=true over at Facebook does not imply email.verified=true for allauth. |
Ok, but have you tried whether it will return a non-verified email address On Mon, Jan 7, 2013 at 5:09 PM, Raymond Penners [email protected]:
Johannes Wilm |
I haven't tried yet. Still, even if it works like that now, it is no guarantee that it stays that way as it is clearly not documented that way. I am still pondering about an approach to move this forward. How about the following approach:
This way, you can add your own custom logic and return Would this suit your needs? |
Added a new FB provider setting: VERIFIED_EMAIL |
Usually I do want the email address to be confirmed, however when this email address is taken from a social networking site the probability that this email address is valid is pretty high. Why would I still want to send out a confirmation link? It would be great if there would be a way to disable this, while leaving this functionality intact for non-social accounts.
The text was updated successfully, but these errors were encountered: