Skip to content

Upgrade to 4.7 for linkedin oauth2 provider#2

Open
eucuepo wants to merge 126 commits intojolivares:masterfrom
3pillarlabs:master
Open

Upgrade to 4.7 for linkedin oauth2 provider#2
eucuepo wants to merge 126 commits intojolivares:masterfrom
3pillarlabs:master

Conversation

@eucuepo
Copy link
Copy Markdown
Collaborator

@eucuepo eucuepo commented Mar 9, 2015

No description provided.

tarun-nagpal and others added 30 commits May 15, 2014 16:23
1) The profile and contact URLs for Flikr have changed
2) Provider Id was not set.
Please accept this plugin.

At the plugin in the In the provider.FlickrImpl.java in the function getPluginsList (line 493)
list.add("org.brickred.socialauth.plugin.flickr.AlbumsPluginImpl");
google plus token expiry is returned as:

"expires_in" : 3596

OAuth2 class only handles strings:

            if (jObj.has("expires_in")) {
                String str = jObj.getString("expires_in");
                if (str != null && str.length() > 0) {
                    expires = Integer.valueOf(str);
                }
            }

this code result in a exception being raised:
org.json.JSONException: JSONObject["expires_in"] not a string.

and ultimately fails with:

org.brickred.socialauth.exception.SocialAuthException: Unexpected auth response from https://accounts.google.com/o/oauth2/token


fix issue #30
Conflicts:
	socialauth/src/main/java/org/brickred/socialauth/oauthstrategy/OAuth2.java
sayantam and others added 30 commits May 8, 2017 12:02
Fixed hotmail/microsoft live login issue
Update master with merged pull requests from develop
Simple Travis Configuration
Added Travis CI build status badge
initial commit for GitHub pages
Standard structure and content from labs site
Merging pages from dev to master
Bump up Facebook API to 2.10
Merge changes from dev to master
Add support for retrieving email address of authenticated Twitter User
Merging changes for Twitter email
Preparing for new Maven release
Upgrade Facebook Graph API to v3.2
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.