From 83f27e65667bac221cb98240d32e26d980686a9d Mon Sep 17 00:00:00 2001 From: Roger Hu Date: Thu, 11 May 2017 22:50:04 -0700 Subject: [PATCH] New intent:// scheme breaks this check. Disable for now --- README.md | 2 +- app/build.gradle | 2 +- app/src/main/java/com/codepath/oauth/OAuthBaseClient.java | 8 +++----- 3 files changed, 5 insertions(+), 7 deletions(-) diff --git a/README.md b/README.md index ca75014..e2a05aa 100755 --- a/README.md +++ b/README.md @@ -24,7 +24,7 @@ Next, add this line to your `app/build.gradle` file: ```gradle dependencies { - compile 'com.codepath.libraries:android-oauth-handler:1.2.3' + compile 'com.codepath.libraries:android-oauth-handler:1.2.4' } ``` diff --git a/app/build.gradle b/app/build.gradle index 0f6bf5c..d75b50d 100644 --- a/app/build.gradle +++ b/app/build.gradle @@ -8,7 +8,7 @@ ext { GROUP = 'com.codepath.libraries' BASE_VERSION = "1.2" - VERSION_NAME = "1.2.3" + VERSION_NAME = "1.2.4" POM_PACKAGING = "aar" POM_DESCRIPTION = "CodePath OAuth Handler" diff --git a/app/src/main/java/com/codepath/oauth/OAuthBaseClient.java b/app/src/main/java/com/codepath/oauth/OAuthBaseClient.java index 2b981c1..1db4680 100755 --- a/app/src/main/java/com/codepath/oauth/OAuthBaseClient.java +++ b/app/src/main/java/com/codepath/oauth/OAuthBaseClient.java @@ -122,11 +122,9 @@ public void connect() { public void authorize(Uri uri, OAuthAccessHandler handler) { this.accessHandler = handler; if (checkAccessToken() == null && uri != null) { - String uriServiceCallback = uri.getScheme() + "://" + uri.getHost(); - // check if the authorize callback matches this service before trying to get an access token - if (uriServiceCallback.equals(callbackUrl)) { - client.fetchAccessToken(getOAuth1RequestToken(), uri); - } + // TODO: check UriServiceCallback with intent:// scheme + client.fetchAccessToken(getOAuth1RequestToken(), uri); + } else if (checkAccessToken() != null) { // already have access token this.accessHandler.onLoginSuccess(); }