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.

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

Authorization with Google is not working #121

Open
jainarpit014 opened this issue Aug 21, 2018 · 7 comments
Open

Authorization with Google is not working #121

jainarpit014 opened this issue Aug 21, 2018 · 7 comments

Comments

@jainarpit014
Copy link

It doesn't work on the demo as well.

  1. Try to Signin with google
  2. Logout
  3. Sign in again with google
    http://nodge.ru/yii-eauth/demo2/login
@web-rider
Copy link

I solve this poblem:
In /vendor/nodge/yii2-eauth/src/oauth2
in function getCallbackUrl need replace
foreach (['code', 'state', 'redirect_uri'] as $param) {

to

foreach (['code', 'state', 'redirect_uri', 'scope'] as $param) {

@manos-k
Copy link

manos-k commented Sep 20, 2018

@web-rider works like a charm. Thank you

@xr0m3oz
Copy link

xr0m3oz commented Sep 27, 2018

@web-rider works,thanks

@Ravi-Seffcon
Copy link

Yes, it works. Thanks a lot!

@xr0m3oz
Copy link

xr0m3oz commented Sep 28, 2018

Need to add a fix to the main code, to be able to update through the composer. @Nodge

@Rjgoolsby
Copy link

There's already a pull-request for this. I don't think this repo is maintained anymore. Also, you may want to follow this pull request and move that specifically to the Google OAuth Class so it has no detrimental impacts to any other services

Pull Request #120

@ramssesable
Copy link

ramssesable commented Jun 20, 2019

There is an other one to unset 'hd' after that pull request #120

It must be like this way

foreach (['code', 'state', 'redirect_uri', 'scope', 'authuser', 'prompt', 'session_state', 'hd'] as $param) {
	if (isset($route[$param])) {
		unset($route[$param]);
	}
}

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

No branches or pull requests

7 participants