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.

Sign up for GitHub

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

Jump to bottom

Logging in to a Linkedin profile that doesn't have an email attached causes undefined offset 0 #347

Closed
HoughtonChristopher opened this issue Feb 26, 2019 · 2 comments
Closed

Logging in to a Linkedin profile that doesn't have an email attached causes undefined offset 0 #347

HoughtonChristopher opened this issue Feb 26, 2019 · 2 comments
Labels
bug

Comments

@HoughtonChristopher
Copy link

ln 98 in LinkedinProvider return json_decode($response->getBody(), true)['elements'][0]['handle~'];
@driesvints driesvints added the bug label Feb 26, 2019
@driesvints
Copy link
Member

Okay, I can see this going wrong but how is it possible that a profile doesn't has an email attached?

@HoughtonChristopher
Copy link
Author

I have absolutely no idea but I just experienced it.

Maybe not verifying it or signing up with a phone number? I'm not entirely sure.

@driesvints driesvints mentioned this issue Mar 29, 2019
Merged
drakeford97 added a commit to drakeford97/Socialite that referenced this issue May 25, 2022
@drakeford97
Fix bug with no LinkedIn email addresses
f31bee4 
In certain situations it's possible that LinkedIn doesn't returns any email addresses because they're private and not available through the API. At the moment the current provider always expects one so this change allows also to retrieve the user without expecting an email address.

Fixes laravel/socialite#347
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@driesvints @HoughtonChristopher