-
Notifications
You must be signed in to change notification settings - Fork 146
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
New clients added #67
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Ahh... so cool! You rock @luchianenco! Check out our clients list now - https://github.com/knpuniversity/oauth2-client-bundle#step-1-download-the-client-library - 35! Woo! |
thanks ;) |
Many in our list are outdated? Or many other client libraries are outdated (so we can't add them)? |
Many client libraries doesn't support ResourceOwnerInterface and therefore we cannot use them as I understand. |
Hmm, yea that makes sense -we require them to return that. Well, probably in those cases, those libraries aren't very used anyways :) |
Some of them, what I would like to add in the first row are Envato, MeetUp, Reddit, Twitch and some others that have really big usage...there are many also that we can add but as I understand they are not so popular. |
Hmm. Yea, I looked at a few of those. They're indeed implemented incorrectly - e.g. https://github.com/tpavlek/oauth2-twitch/blob/ead8e17d393610b35d06f6e42723a75590833a3c/src/Provider/Twitch.php#L130 should return a That's a bug in their code. So, to support these, the libraries themselves would need to be improved, or replaced with new libs :/ |
Another portion of clients: Salesforce, Zendesk, Strava, Uber, Unsplash, PSN, Mollie are added