-
Notifications
You must be signed in to change notification settings - Fork 228
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
Add Instagram Provider #51
Conversation
@Waynenabors Sorry for the delay! @athix What are your thoughts on rolling this our vs making it a separate gem? |
Fixes an issue where new Facebook Apps with API v2.8 are no longer compatible with v2.2 Oauth callback content/type. Starting with v2.3 the response is valid JSON. So if we can pick our API version we should be able to pick our parser. I would even make the argument that it should default to :automatic since that works now. But for now I just needed access to set |
I met this problem (and monkey patched). I appreciate if this facebook's changes are merged. |
If someone can resolve the conflict, I'll merge this. |
Last update almost year ago, whoops. 😬 |
My first pull request so hopefully it makes sense.
This was started on the old repo here: https://github.com/NoamB/sorcery/pull/625/files
We are using this code with an approved Instagram client_id with public_scope and it works great.