feedback

Author: p3dr0rv

common/src/main/java/com/microsoft/identity/common/adal/internal/AuthenticationConstants.java

@@ -545,12 +545,12 @@ public static final class SWITCH_BROWSER {
         public static final String CODE = "code";
 
         /**
-         * String Query parameter key for the endpoint.
+         * String Query parameter key for the switchBrowser action uri.
          */
         public static final String ACTION_URI = "action_uri";
 
         /**
-         * String Query parameter key for the endpoint.
+         * String Query parameter key for the switchBrowser action.
          */
         public static final String ACTION = "action";
 

common/src/main/java/com/microsoft/identity/common/internal/ui/webview/AzureActiveDirectoryWebViewClient.java

@@ -339,7 +339,8 @@ protected boolean processSwitchToBrowserRequest(@NonNull final Uri uri) {
         final String code = uri.getQueryParameter(AuthenticationConstants.SWITCH_BROWSER.CODE);
         if (action_uri == null || code == null) {
             // This should never happen, but if it does, we should log it and return.
-            Logger.error(TAG, "Switch browser action URI or code is null. Cannot switch browser.", null);
+            Logger.error(TAG, "Switch browser action URI is null: " + StringUtil.isNullOrEmpty(action_uri), null);
+            Logger.error(TAG, "Switch browser code is null: " + StringUtil.isNullOrEmpty(code), null);
             return false;
         }
         final HashMap<String, String> queryParams = new HashMap<>();