fix(rnta-msal): address deprecations (#3101)

microsoft · Apr 22, 2024 · af82a1f · af82a1f
1 parent 9516a64
commit af82a1f
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 8 deletions.
diff --git a/.changeset/twelve-jeans-wait.md b/.changeset/twelve-jeans-wait.md
@@ -0,0 +1,5 @@
+---
+"@rnx-kit/react-native-test-app-msal": patch
+---
+
+Migrate away from deprecated `acquireTokenSilent(Array<(out) String!>, IAccount, String): IAuthenticationResult!`
diff --git a/...p-msal/android/src/main/java/com/microsoft/reacttestapp/msal/MicrosoftAccountsActivity.kt b/...p-msal/android/src/main/java/com/microsoft/reacttestapp/msal/MicrosoftAccountsActivity.kt
@@ -76,7 +76,7 @@ class MicrosoftAccountsActivity : AppCompatActivity() {
 
         val sharedPreferences = getPreferences(MODE_PRIVATE)
 
-        val selectedAccountObserver = Observer<Account> { account ->
+        val selectedAccountObserver = Observer<Account?> { account ->
             withTokenBroker { tokenBroker ->
                 if (tokenBroker.selectedAccount == account) {
                     return@withTokenBroker

diff --git a/...native-test-app-msal/android/src/main/java/com/microsoft/reacttestapp/msal/TokenBroker.kt b/...native-test-app-msal/android/src/main/java/com/microsoft/reacttestapp/msal/TokenBroker.kt
@@ -4,6 +4,7 @@ import android.app.Activity
 import android.content.Context
 import android.content.res.Resources.NotFoundException
 import com.microsoft.identity.client.AcquireTokenParameters
+import com.microsoft.identity.client.AcquireTokenSilentParameters
 import com.microsoft.identity.client.AuthenticationCallback
 import com.microsoft.identity.client.IAuthenticationResult
 import com.microsoft.identity.client.IMultipleAccountPublicClientApplication
@@ -19,14 +20,13 @@ class TokenBroker private constructor(context: Context) {
         const val EMPTY_GUID = "00000000-0000-0000-0000-000000000000"
 
         @Volatile
-        private var INSTANCE: TokenBroker? = null
+        private var instance: TokenBroker? = null
 
-        fun getInstance(context: Context): TokenBroker =
-            INSTANCE ?: synchronized(this) {
-                INSTANCE ?: TokenBroker(context).also {
-                    INSTANCE = it
-                }
+        fun getInstance(context: Context): TokenBroker = instance ?: synchronized(this) {
+            instance ?: TokenBroker(context).also {
+                instance = it
             }
+        }
     }
 
     var selectedAccount: Account? = null
@@ -152,7 +152,13 @@ class TokenBroker private constructor(context: Context) {
 
         val authority = multiAccountApp.configuration.defaultAuthority.authorityURL.toString()
         try {
-            val result = multiAccountApp.acquireTokenSilent(scopes, account, authority)
+            val result = multiAccountApp.acquireTokenSilent(
+                AcquireTokenSilentParameters.Builder()
+                    .forAccount(account)
+                    .fromAuthority(authority)
+                    .withScopes(scopes.toList())
+                    .build()
+            )
             val redirectUri = multiAccountApp.configuration.redirectUri
             onTokenAcquired(AuthResult(result, redirectUri), null)
         } catch (_: UiRequiredException) {