Merge pull request #39 from rogerhu/fix_intent

New intent:// scheme breaks this check. Disable for now
codepath · May 12, 2017 · 593c27d · 593c27d
2 parents 44e0199 + 83f27e6
commit 593c27d
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git 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
@@ -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
@@ -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();
         }