Added cancellation checks in SynchronizeFolderOperation before every RemoteOperation...

[pub/Android/ownCloud.git] / src / com / owncloud / android / operations / DetectAuthenticationMethodOperation.java

diff --git a/src/com/owncloud/android/operations/DetectAuthenticationMethodOperation.java b/src/com/owncloud/android/operations/DetectAuthenticationMethodOperation.java
index a30829b..5afc421 100644 (file)
--- a/src/com/owncloud/android/operations/DetectAuthenticationMethodOperation.java
+++ b/src/com/owncloud/android/operations/DetectAuthenticationMethodOperation.java
@@ -31,11 +31,11 @@ import com.owncloud.android.lib.common.operations.OnRemoteOperationListener;
 import com.owncloud.android.lib.common.operations.RemoteOperation;
 import com.owncloud.android.lib.common.operations.RemoteOperationResult;
 import com.owncloud.android.lib.common.operations.RemoteOperationResult.ResultCode;
 import com.owncloud.android.lib.common.operations.RemoteOperation;
 import com.owncloud.android.lib.common.operations.RemoteOperationResult;
 import com.owncloud.android.lib.common.operations.RemoteOperationResult.ResultCode;

+import com.owncloud.android.lib.common.utils.Log_OC;

 import com.owncloud.android.lib.resources.files.ExistenceCheckRemoteOperation;
 
 import android.content.Context;
 import android.net.Uri;
 import com.owncloud.android.lib.resources.files.ExistenceCheckRemoteOperation;
 
 import android.content.Context;
 import android.net.Uri;

-import android.util.Log;

 
 /**
  * Operation to find out what authentication method requires
 
 /**
  * Operation to find out what authentication method requires


@@ -69,6 +69,7 @@ public class DetectAuthenticationMethodOperation extends RemoteOperation {
      * Constructor
      * 
      * @param context       Android context of the caller.
      * Constructor
      * 
      * @param context       Android context of the caller.

+     * @param webdavUrl

      */
     public DetectAuthenticationMethodOperation(Context context) {
         mContext = context;
      */
     public DetectAuthenticationMethodOperation(Context context) {
         mContext = context;


@@ -90,15 +91,18 @@ public class DetectAuthenticationMethodOperation extends RemoteOperation {
         AuthenticationMethod authMethod = AuthenticationMethod.UNKNOWN;
         
         RemoteOperation operation = new ExistenceCheckRemoteOperation("", mContext, false);
         AuthenticationMethod authMethod = AuthenticationMethod.UNKNOWN;
         
         RemoteOperation operation = new ExistenceCheckRemoteOperation("", mContext, false);

-        client.setBasicCredentials("", "");
+        client.clearCredentials();

         client.setFollowRedirects(false);
         
         // try to access the root folder, following redirections but not SAML SSO redirections
         client.setFollowRedirects(false);
         
         // try to access the root folder, following redirections but not SAML SSO redirections

-        do {
-            result = operation.execute(client);
+        result = operation.execute(client);
+        String redirectedLocation = result.getRedirectedLocation(); 
+        while (redirectedLocation != null && redirectedLocation.length() > 0 && 
+                !result.isIdPRedirection()) {

             client.setBaseUri(Uri.parse(result.getRedirectedLocation()));
             client.setBaseUri(Uri.parse(result.getRedirectedLocation()));

-            
-        } while (result.isTemporalRedirection() && !result.isIdPRedirection());
+            result = operation.execute(client);
+            redirectedLocation = result.getRedirectedLocation();
+        } 

 
         // analyze response  
         if (result.getCode() == ResultCode.UNAUTHORIZED) {
 
         // analyze response  
         if (result.getCode() == ResultCode.UNAUTHORIZED) {


@@ -118,8 +122,11 @@ public class DetectAuthenticationMethodOperation extends RemoteOperation {
             authMethod = AuthenticationMethod.SAML_WEB_SSO;
         }
         // else - fall back to UNKNOWN
             authMethod = AuthenticationMethod.SAML_WEB_SSO;
         }
         // else - fall back to UNKNOWN

-        Log.d(TAG, "Authentication method found: " + authenticationMethodToString(authMethod));
+        Log_OC.d(TAG, "Authentication method found: " + authenticationMethodToString(authMethod));

         
         

+        if (!authMethod.equals(AuthenticationMethod.UNKNOWN)) {
+            result = new RemoteOperationResult(true, result.getHttpCode(), null);
+        }

         ArrayList<Object> data = new ArrayList<Object>();
         data.add(authMethod);
         result.setData(data);
         ArrayList<Object> data = new ArrayList<Object>();
         data.add(authMethod);
         result.setData(data);