Manual merging working/release/23.0.0 to release/23.0.0, Fixes AB#3397730 (#2782)

Manual merging working/release/23.0.0 to release/23.0.0 for octoebr
release process.

PBI Item -
[AB#3397730](https://dev.azure.com/IdentityDivision/Engineering/_workitems/edit/3397730)

---------

Co-authored-by: Azure DevOps Pipeline <[email protected]>
Co-authored-by: Shahzaib <[email protected]>
Co-authored-by: p3dr0rv <[email protected]>

Author: 4 people

changelog.txt

@@ -1,5 +1,8 @@
 vNext
 ----------
+
+Version 23.0.2
+----------
 - [MAJOR] Add KeyStoreBackedSecretKeyProvider (#2674)
 - [MINOR] Add Open Id configuration issuer validation reporting in OpenIdProviderConfigurationClient (#2751)
 - [MINOR] Add helper method to record elapsed time (#2768)
@@ -11,6 +14,7 @@ vNext
 - [MINOR] SDK now handles SMS as strong authentication method #2766
 - [MINOR] Added error handling when webcp redirects have browser protocol #2767
 - [PATCH] Fix for app link redirect from CCT due to forced browser preference (#2775)
+- [PATCH] Don't set browser pkg name in CustomTabIntent (#2778)
 
 Version 22.1.0
 ----------

common/build.gradle

@@ -25,7 +25,7 @@ codeCoverageReport {
 
 // In dev, we want to keep the dependencies(common4j, broker4j, common) to 1.0.+ to be able to be consumed by daily dev pipeline.
 // In release/*, we change these to specific versions being consumed.
-def common4jVersion = "1.0.+"
+def common4jVersion = "23.0.2"
 if (project.hasProperty("distCommon4jVersion") && project.distCommon4jVersion != '') {
     common4jVersion = project.distCommon4jVersion
 }
@@ -376,10 +376,10 @@ tasks.configureEach { task ->
         task.dependsOn 'bundleDistReleaseAar'
     }
     if (task.name == "generateMetadataFileForDistDebugPublication") {
-        task.mustRunAfter 'bundleDistReleaseAar'
+        task.mustRunAfter 'bundleDistDebugAar'
     }
     if (task.name == "generateMetadataFileForDistReleasePublication") {
-        task.dependsOn 'bundleDistDebugAar'
+        task.dependsOn 'bundleDistReleaseAar'
     }
 }
 

common/src/main/java/com/microsoft/identity/common/internal/ui/browser/BrowserAuthorizationStrategy.java

@@ -99,6 +99,7 @@ public Future<AuthorizationResult> requestAuthorization(
             if (!mCustomTabManager.bind(context, mBrowser.getPackageName())) {
                 //create browser auth intent
                 authIntent = new Intent(Intent.ACTION_VIEW);
+                authIntent.setPackage(mBrowser.getPackageName());
             } else {
                 authIntent = mCustomTabManager.getCustomTabsIntent().intent;
             }
@@ -109,9 +110,9 @@ public Future<AuthorizationResult> requestAuthorization(
             );
             //create browser auth intent
             authIntent = new Intent(Intent.ACTION_VIEW);
+            authIntent.setPackage(mBrowser.getPackageName());
         }
 
-        authIntent.setPackage(mBrowser.getPackageName());
         final URI requestUrl = authorizationRequest.getAuthorizationRequestAsHttpRequest();
 
         authIntent.setData(Uri.parse(requestUrl.toString()));

common4j/versioning/version.properties

@@ -1,4 +1,4 @@
 #Wed May 12 20:08:39 UTC 2021
-versionName=22.1.0
+versionName=23.0.2
 versionCode=1
 latestPatchVersion=227

versioning/version.properties

@@ -1,4 +1,4 @@
 #Tue Apr 06 22:55:08 UTC 2021
-versionName=22.1.0
+versionName=23.0.2
 versionCode=1
 latestPatchVersion=234