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
Facebook Authentication Register throws error #4354
Comments
I don´t think that this is a "love to have"-priority. That´s a bug which makes social login with facebook not possible. This should get adressed more quickly. |
I´m not quite sure how the vendors are handled in C5 but the UserData-Extractor from oryzone is outdated. This fixes the error: /concrete/vendor/oryzone/oauth-user-data/src/OAuth/UserData/Extractor/Facebook.php Instead of go with: |
This seems like an outstanding issue with the userdata-extractor library and it doesn't appear to be fixed even though there's an open issue (that's been open since early 2016); should we fork this library @KorvinSzanto ? |
As far as I could test 8.2.1 solved this for most Members but the same error still occurs for some. |
@aembler , could I please request that this is put in queue for the near future? |
It is already there – we'd be happy to accept pull requests for this. It's not personally high on our list of items to work on but I imagine it'll be fixed at some point – probably by someone who needs the functionality. |
Hello everyone, Finally, i found fb login issue solution sharing here. I was using facebook 5.8 and still found fb login issue. I found one issue which is related to token read of facebook and other issue the url as posted by @micrdy . Below is 2 changes which i did after that fb login started working for me 1. concrete5-8.1.0\concrete\vendor\lusitanian\oauth\src\OAuth\OAuth2\Service\Facebook.php IN FUNCTION parseAccessTokenResponse($responseBody) replace With 2. concrete5-8.1.0\concrete\vendor\oryzone\oauth-user-data\src\OAuth\UserData\Extractor\Facebook.php Instead of go with: |
In /dashboard/system/registration/authentication,
When a new user tries to register via faceook, c5 returns a following error.
An exception occurred while executing 'insert into Users (uName, uEmail, uPassword, uIsValidated, uDateAdded, uLastPasswordChange, uIsFullRecord, uDefaultLanguage, uIsActive) values (?, ?, ?, ?, ?, ?, ?, ?, ?)' with params ["", null, "xxxxxxxxxx.", 1, "2016-08-26 17:07:39", "2016-08-26 17:07:39", 1, null, 1]:
SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'uEmail' cannot be null
Facebook document says;
https://developers.facebook.com/docs/facebook-login/permissions#refe...
Note, even if you request the email permission it is not guaranteed you will get an email address. For example, if someone signed up for Facebook with a phone number instead of an email address, the email field may be empty.
Maybe we need to update Facebook Authentication program to allow a user to fill in his/her email. Twitter Authentication process asks a user to fill in his/her email.
The text was updated successfully, but these errors were encountered: