Merge pull request #39 from rogerhu/fix_intent

New intent:// scheme breaks this check.  Disable for now

Author: rogerhu

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'
 }
 ```
 

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"
 

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();
         }