Auth: Correctly handle permission for requests with token request opt…

…ions (#2322)
microg · May 5, 2024 · d2169ad · d2169ad
1 parent b1520c4
commit d2169ad
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/play-services-core/src/main/kotlin/org/microg/gms/auth/signin/extensions.kt b/play-services-core/src/main/kotlin/org/microg/gms/auth/signin/extensions.kt
@@ -96,11 +96,11 @@ fun getServerAuthTokenManager(context: Context, packageName: String, options: Go
 
 suspend fun performSignIn(context: Context, packageName: String, options: GoogleSignInOptions?, account: Account, permitted: Boolean = false): GoogleSignInAccount? {
     val authManager = getOAuthManager(context, packageName, options, account)
-    if (permitted) authManager.isPermitted = true
     val authResponse = withContext(Dispatchers.IO) {
         if (options?.includeUnacceptableScope == true) {
             authManager.setTokenRequestOptions(consentRequestOptions)
         }
+        if (permitted) authManager.isPermitted = true
         authManager.requestAuth(true)
     }
     var consentResult:String ?= null
@@ -110,7 +110,7 @@ suspend fun performSignIn(context: Context, packageName: String, options: Google
     } else {
         if (authResponse.auth == null) return null
     }
-    Log.d(TAG, "id token requested: ${options?.isIdTokenRequested == true}, serverClientId = ${options?.serverClientId}")
+    Log.d(TAG, "id token requested: ${options?.isIdTokenRequested == true}, serverClientId = ${options?.serverClientId}, permitted = ${authManager.isPermitted}")
     val idTokenResponse = getIdTokenManager(context, packageName, options, account)?.let {
         it.isPermitted = authManager.isPermitted
         consentResult?.let { result -> it.putDynamicFiled(CONSENT_RESULT, result) }