Skip to content

Added Exception Log and fixed Google provider#18

Open
LorenzoRogai wants to merge 1 commit intoUseMuffin:masterfrom
LorenzoRogai:master
Open

Added Exception Log and fixed Google provider#18
LorenzoRogai wants to merge 1 commit intoUseMuffin:masterfrom
LorenzoRogai:master

Conversation

@LorenzoRogai
Copy link

Added Exception Log on the _authenticate function

Added the support for Google provider, Google now returns an array with multiple emails. The script will now take the first one. This should prevent newUser from firing on each Google oAuth 2 login

Added Exception Log on the _authenticate function

Added the support for Google provider, Google now returns an array with multiple emails. The script will now take the first one. This should prevent newUser from firing on each Google oAuth 2 login
@Spriz
Copy link
Contributor

Spriz commented Dec 26, 2017

I personally think checking whether $data[$this->config('fields.username')] is an array or not is more suitable :-) This doesn't respect configurations neither.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants