-
Notifications
You must be signed in to change notification settings - Fork 52
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
Twitter e-mail validation #2
Comments
That's a good enhancement request. As Twitter's OAuth contains no extra parameters for specifying state, the only way that can be done is by appending parameters to Due to the parameter being handled internally by Opauth's TwitterStrategy, the extra parameters would be lost when Opauth does a callback to parent app. I have marked this issue as enhancement and will be worked on to add state support. Thanks @alex21 for your feedback and suggestion. |
Thank you for answer. Waiting for this enhancement:) |
From what i understand of twitter api, there is no way you can obtain or validate an email address by twitter. Can you provide a link where i can info on this subject? |
@ceeram I believe this story is asking for state support (or I believe can be implemented with state support – uzyn/opauth#21) State support simply allows you to pass in a string and obtain a string back after all the callbacks and redirects. |
Twitter no longer allows variable get query parameters in callback urls, the url must match exactly the one you had set in the app settings. https://github.com/opauth/facebook/blob/wip/1.0/Strategy.php#L49 So it is very much dependent on the strategy provider |
Before send request to twitter, I ask user for his e-mail. Can I add an extra parameter(like scope in google strategy) to send to twitter e-mail and validate it. I expect to receive in response something like this:
"email_verified" => 1(or 0)
Can I do this? Or how can I validate e-mail in other way?
Thanks.
The text was updated successfully, but these errors were encountered: