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

OAuth2 client libraries like Facebook needs "league/oauth2-client": "^2.0" to be installed! #48

Closed
giovkanata opened this issue Jan 26, 2017 · 6 comments

Comments

@giovkanata
Copy link

giovkanata commented Jan 26, 2017

I would like to switch to this library from HWIOAuthBundle (we like so much Guard 👍), but client libraries like Facebook, Github, Instagram, Google, Linkedin, Bitbucket (and maybe others...) have a required dependency with "league/oauth2-client": "^2.0" that's locked at the "league/oauth2-client": "^1.0" version in this bundle. How we can resolve the issue?

@bocharsky-bw
Copy link
Member

I'm wondering will"league/oauth2-client": "^1.0|^2.0" be enough for knpuniversity/oauth2-client-bundle?

@weaverryan
Copy link
Member

@bocharsky-bw yep, that's what we need! But, there may have been some BC breaks for version 2. If so, we'll need to write some code so that our library works with both.

OR, if version 2 is just too different, we would need to bump our dependency to oauth2-client version 2 and release a new major version of this bundle.

@weaverryan
Copy link
Member

Actually, if the CHANGELOG is accurate, very little changed: https://github.com/thephpleague/oauth2-client/blob/master/CHANGELOG.md#200

It might be just enough to make that change to composer.json. @bocharsky-bw do you want to make a PR to try it? We can see if the tests pass!

@bocharsky-bw
Copy link
Member

bocharsky-bw commented Jan 26, 2017

@weaverryan Yes, let's see if tests pass first. I opened #49

@weaverryan
Copy link
Member

Done! And new version tagged at 1.9.0.

Thanks for the ping on this! I hadn't noticed the new 2.0 version of the lib

@giovkanata
Copy link
Author

giovkanata commented Jan 26, 2017

Thank you so much! Very fast!

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

3 participants