Merge branch 'saml_based_federated_single_sign_on' into sso_bug_changing_orientation

[pub/Android/ownCloud.git] / src / com / owncloud / android / authentication / SsoWebViewClient.java

diff --git a/src/com/owncloud/android/authentication/SsoWebViewClient.java b/src/com/owncloud/android/authentication/SsoWebViewClient.java
index 9d78d6a..ec86667 100644 (file)
--- a/src/com/owncloud/android/authentication/SsoWebViewClient.java
+++ b/src/com/owncloud/android/authentication/SsoWebViewClient.java
@@ -17,15 +17,18 @@
 
 package com.owncloud.android.authentication;
 
 
 package com.owncloud.android.authentication;
 

-import android.content.Context;
+import java.lang.ref.WeakReference;
+
+import com.owncloud.android.Log_OC;
+

 import android.graphics.Bitmap;
 import android.graphics.Bitmap;

+import android.os.Handler;
+import android.os.Message;

 import android.view.View;
 import android.webkit.CookieManager;
 import android.webkit.WebView;
 import android.webkit.WebViewClient;
 import android.view.View;
 import android.webkit.CookieManager;
 import android.webkit.WebView;
 import android.webkit.WebViewClient;

-import android.widget.Toast;

 
 

-import com.owncloud.android.Log_OC;

 
 /**
  * Custom {@link WebViewClient} client aimed to catch the end of a single-sign-on process 
 
 /**
  * Custom {@link WebViewClient} client aimed to catch the end of a single-sign-on process 


@@ -40,12 +43,20 @@ public class SsoWebViewClient extends WebViewClient {
         
     private static final String TAG = SsoWebViewClient.class.getSimpleName();
     
         
     private static final String TAG = SsoWebViewClient.class.getSimpleName();
     

-    private Context mContext;
+    public interface SsoWebViewClientListener {
+        public void onSsoFinished(String sessionCookie);
+    }
+    
+    private Handler mListenerHandler;
+    private WeakReference<SsoWebViewClientListener> mListenerRef;

     private String mTargetUrl;
     private String mTargetUrl;

+    private String mLastReloadedUrlAtError;

     
     

-    public SsoWebViewClient (Context context) {
-        mContext = context;
+    public SsoWebViewClient (Handler listenerHandler, SsoWebViewClientListener listener) {
+        mListenerHandler = listenerHandler;
+        mListenerRef = new WeakReference<SsoWebViewClient.SsoWebViewClientListener>(listener);

         mTargetUrl = "fake://url.to.be.set";
         mTargetUrl = "fake://url.to.be.set";

+        mLastReloadedUrlAtError = null;

     }
     
     public String getTargetUrl() {
     }
     
     public String getTargetUrl() {


@@ -58,85 +69,104 @@ public class SsoWebViewClient extends WebViewClient {
 
     @Override
     public void onPageStarted (WebView view, String url, Bitmap favicon) {
 
     @Override
     public void onPageStarted (WebView view, String url, Bitmap favicon) {

-        //Log_OC.e(TAG, "onPageStarted : " + url);
+        Log_OC.d(TAG, "onPageStarted : " + url);

         if (url.startsWith(mTargetUrl)) {
             view.setVisibility(View.GONE);
             CookieManager cookieManager = CookieManager.getInstance();
         if (url.startsWith(mTargetUrl)) {
             view.setVisibility(View.GONE);
             CookieManager cookieManager = CookieManager.getInstance();

-            String cookies = cookieManager.getCookie(url);
-            Toast.makeText(mContext, "got cookies: " + cookies, Toast.LENGTH_LONG).show();
+            final String cookies = cookieManager.getCookie(url);
+            //Log_OC.d(TAG, cookies);
+            if (mListenerHandler != null && mListenerRef != null) {
+                // this is good idea because onPageStarted is not running in the UI thread
+                mListenerHandler.post(new Runnable() {
+                    @Override
+                    public void run() {
+                        SsoWebViewClientListener listener = mListenerRef.get();
+                        if (listener != null) {
+                            listener.onSsoFinished(cookies);
+                        }
+                    }
+                });
+            }

         }
     }
     
     @Override
         }
     }
     
     @Override

+    public void onFormResubmission (WebView view, Message dontResend, Message resend) {
+        Log_OC.d(TAG, "onFormResubMission ");
+
+        // necessary to grant reload of last page when device orientation is changed after sending a form
+        resend.sendToTarget();
+    }
+
+    @Override

     public boolean shouldOverrideUrlLoading(WebView view, String url) {
     public boolean shouldOverrideUrlLoading(WebView view, String url) {

-        //view.loadUrl(url);

         return false;
     }
     
     @Override
     public void onReceivedError (WebView view, int errorCode, String description, String failingUrl) {
         return false;
     }
     
     @Override
     public void onReceivedError (WebView view, int errorCode, String description, String failingUrl) {

-        Log_OC.e(TAG, "onReceivedError : " + failingUrl);
+        Log_OC.e(TAG, "onReceivedError : " + failingUrl + ", code " + errorCode + ", description: " + description);
+        if (!failingUrl.equals(mLastReloadedUrlAtError)) {
+            view.reload();
+            mLastReloadedUrlAtError = failingUrl;
+        } else {
+            mLastReloadedUrlAtError = null;
+        }

     }
     
     }
     

-    /*
-

     @Override
     @Override

-    public void doUpdateVisitedHistory (WebView view, String url, boolean isReload) {
-        Log_OC.e(TAG, "doUpdateVisitedHistory : " + url);
+    public void onPageFinished (WebView view, String url) {
+        Log_OC.d(TAG, "onPageFinished : " + url);
+        mLastReloadedUrlAtError = null;

     }
     
     }
     

+    /*

     @Override
     @Override

-    public void onPageFinished (WebView view, String url) {
-        Log_OC.e(TAG, "onPageFinished : " + url);
+    public void doUpdateVisitedHistory (WebView view, String url, boolean isReload) {
+        Log_OC.d(TAG, "doUpdateVisitedHistory : " + url);

     }
     
     @Override
     public void onReceivedSslError (WebView view, SslErrorHandler handler, SslError error) {
     }
     
     @Override
     public void onReceivedSslError (WebView view, SslErrorHandler handler, SslError error) {

-        Log_OC.e(TAG, "onReceivedSslError : " + error);
+        Log_OC.d(TAG, "onReceivedSslError : " + error);

     }
     
     @Override
     public void onReceivedHttpAuthRequest (WebView view, HttpAuthHandler handler, String host, String realm) {
     }
     
     @Override
     public void onReceivedHttpAuthRequest (WebView view, HttpAuthHandler handler, String host, String realm) {

-        Log_OC.e(TAG, "onReceivedHttpAuthRequest : " + host);
+        Log_OC.d(TAG, "onReceivedHttpAuthRequest : " + host);

     }
 
     @Override
     public WebResourceResponse shouldInterceptRequest (WebView view, String url) {
     }
 
     @Override
     public WebResourceResponse shouldInterceptRequest (WebView view, String url) {

-        Log_OC.e(TAG, "shouldInterceptRequest : " + url);
+        Log_OC.d(TAG, "shouldInterceptRequest : " + url);

         return null;
     }
     
     @Override
     public void onLoadResource (WebView view, String url) {
         return null;
     }
     
     @Override
     public void onLoadResource (WebView view, String url) {

-        Log_OC.e(TAG, "onLoadResource : " + url);            
-    }
-    
-    @Override
-    public void onFormResubmission (WebView view, Message dontResend, Message resend) {
-        Log_OC.e(TAG, "onFormResubMission ");
-        super.onFormResubmission(view, dontResend, resend);
+        Log_OC.d(TAG, "onLoadResource : " + url);   

     }
     
     @Override
     public void onReceivedLoginRequest (WebView view, String realm, String account, String args) {
     }
     
     @Override
     public void onReceivedLoginRequest (WebView view, String realm, String account, String args) {

-        Log_OC.e(TAG, "onReceivedLoginRequest : " + realm + ", " + account + ", " + args);
+        Log_OC.d(TAG, "onReceivedLoginRequest : " + realm + ", " + account + ", " + args);

     }
     
     @Override
     public void onScaleChanged (WebView view, float oldScale, float newScale) {
     }
     
     @Override
     public void onScaleChanged (WebView view, float oldScale, float newScale) {

-        Log_OC.e(TAG, "onScaleChanged : " + oldScale + " -> " + newScale);
+        Log_OC.d(TAG, "onScaleChanged : " + oldScale + " -> " + newScale);
+        super.onScaleChanged(view, oldScale, newScale);

     }
     }

-    
+

     @Override
     public void onUnhandledKeyEvent (WebView view, KeyEvent event) {
     @Override
     public void onUnhandledKeyEvent (WebView view, KeyEvent event) {

-        Log_OC.e(TAG, "onUnhandledKeyEvent : " + event);
+        Log_OC.d(TAG, "onUnhandledKeyEvent : " + event);

     }
     
     @Override
     public boolean shouldOverrideKeyEvent (WebView view, KeyEvent event) {
     }
     
     @Override
     public boolean shouldOverrideKeyEvent (WebView view, KeyEvent event) {

-        Log_OC.e(TAG, "shouldOverrideKeyEvent : " + event);
+        Log_OC.d(TAG, "shouldOverrideKeyEvent : " + event);

         return false;
     }
         return false;
     }

-    

     */
 }
     */
 }