Merge pull request #1579 from AzureAD/shahzaibj/merge-release-2.2.3-t…

…o-master Merge release 2.2.3 to master
AzureAD · Jan 6, 2022 · 10db19e · 10db19e
2 parents 6fd0fea + 4224106
commit 10db19e
Show file tree

Hide file tree

Showing 7 changed files with 40 additions and 5 deletions.
diff --git a/changelog b/changelog
@@ -1,5 +1,11 @@
 MSAL Wiki : https://github.com/AzureAD/microsoft-authentication-library-for-android/wiki
 
+Version 2.2.3
+----------
+- Remove the broker check for MSA FRT saving function (#1571)
+- Fix msal get account crash in Account Matcher due to NPE because account id was missing (#1558)
+- Picks up [email protected]
+
 Version 2.2.2
 ----------
 - Picks up [email protected]

diff --git a/common b/common
diff --git a/msal/build.gradle b/msal/build.gradle
@@ -274,9 +274,9 @@ dependencies {
         transitive = false
     }
 
-    snapshotApi(group: 'com.microsoft.identity', name: 'common', version: '3.6.3', changing: true)
+    snapshotApi(group: 'com.microsoft.identity', name: 'common', version: '3.6.7', changing: true)
 
-    distApi("com.microsoft.identity:common:3.6.3") {
+    distApi("com.microsoft.identity:common:3.6.7") {
         transitive = false
     }
 }

diff --git a/msal/src/main/java/com/microsoft/identity/client/Account.java b/msal/src/main/java/com/microsoft/identity/client/Account.java
@@ -31,6 +31,7 @@
 import com.microsoft.identity.common.internal.providers.microsoft.MicrosoftIdToken;
 import com.microsoft.identity.common.internal.providers.microsoft.azureactivedirectory.AzureActiveDirectoryIdToken;
 import com.microsoft.identity.common.internal.providers.oauth2.IDToken;
+import com.microsoft.identity.common.internal.util.AccountUtil;
 import com.microsoft.identity.common.internal.util.StringUtil;
 
 import java.util.Map;
@@ -47,6 +48,7 @@ public class Account implements IAccount {
     private String mHomeOid;
     private String mHomeTenantId;
     private String mEnvironment;
+    private String mHomeAccountId;
 
     public Account(
             @Nullable final String clientInfo,
@@ -69,6 +71,7 @@ void setId(@Nullable final String id) {
     @NonNull
     @Override
     public String getId() {
+        final String methodName = ":getId";
         String id;
 
         ClientInfo clientInfo = null;
@@ -93,6 +96,24 @@ public String getId() {
             id = mHomeOid;
         }
 
+        if (StringUtil.isEmpty(id)) {
+            com.microsoft.identity.common.logging.Logger.warn(
+                    TAG + methodName,
+                    "Unable to get account id from either ClientInfo or IdToken. Attempting to obtain from home account id."
+            );
+            id = AccountUtil.getUIdFromHomeAccountId(mHomeAccountId);
+        }
+
+        if (StringUtil.isEmpty(id)) {
+            com.microsoft.identity.common.logging.Logger.error(
+                    TAG + methodName,
+                    "Account ID is empty. Returning MISSING_FROM_THE_TOKEN_RESPONSE.",
+                    null
+            );
+
+            id = MISSING_FROM_THE_TOKEN_RESPONSE;
+        }
+
         return id;
     }
 
@@ -162,4 +183,8 @@ public String getAuthority() {
 
         return MISSING_FROM_THE_TOKEN_RESPONSE;
     }
+
+    public void setHomeAccountId(@NonNull final String homeAccountId) {
+        mHomeAccountId = homeAccountId;
+    }
 }
diff --git a/msal/src/main/java/com/microsoft/identity/client/AccountAdapter.java b/msal/src/main/java/com/microsoft/identity/client/AccountAdapter.java
@@ -293,6 +293,8 @@ private static List<IAccount> createRootAccounts(
                     getIdToken(homeCacheRecord)
             );
 
+            ((MultiTenantAccount) rootAccount).setHomeAccountId(homeCacheRecord.getAccount().getHomeAccountId());
+
             // Set the tenant_id
             ((MultiTenantAccount) rootAccount).setTenantId(
                     StringUtil.getTenantInfo(

diff --git a/msal/src/main/java/com/microsoft/identity/client/PublicClientApplication.java b/msal/src/main/java/com/microsoft/identity/client/PublicClientApplication.java
@@ -1227,7 +1227,9 @@ public String getMsaFamilyRefreshToken(@NonNull final String identifier) throws
     @Override
     public void saveMsaFamilyRefreshToken(@NonNull final String refreshToken) throws MsalClientException {
         validateNonNullArgument(refreshToken, "refreshToken");
-        validateBrokerNotInUse();
+
+        // todo: Re-enable this when MSA SSO is fully supported by Broker (PRTv3)
+        //validateBrokerNotInUse();
 
         try {
             mTokenShareUtility.saveMsaFamilyRefreshToken(refreshToken);

diff --git a/msal/versioning/version.properties b/msal/versioning/version.properties
@@ -1,3 +1,3 @@
 #Wed Aug 01 15:24:11 PDT 2018
-versionName=2.2.2
+versionName=2.2.3
 versionCode=0