Merge Release integration/24.3.0 to dev, Fixes AB#3618269 (#3131)

[AB#3618269](https://identitydivision.visualstudio.com/fac9d424-53d2-45c0-91b5-ef6ba7a6bf26/_workitems/edit/3618269)

---------

Co-authored-by: Azure DevOps Pipeline <build-agent@azuredevops.com>
Co-authored-by: fadidurah <88730756+fadidurah@users.noreply.github.com>
Co-authored-by: Shahzaib Jameel <shjameel@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: 5 people

changelog.txt

@@ -1,5 +1,8 @@
 vNext
 ----------
+
+Version 24.3.0
+----------
 - [MINOR] Add SwitchBrowserRedirectActivity for non-broker Switch Browser protocol flows. BrokerBrowserRedirectActivity now extends this shared base class (#3133)
 - [PATCH] Fix Token Endpoint Server Telemetry Parsing (#3128)
 - [MINOR] Enable passkey registration by default in CommonFlight (ENABLE_PASSKEY_REGISTRATION) (#3132)

common4j/src/main/com/microsoft/identity/common/java/providers/microsoft/microsoftsts/AbstractMicrosoftStsTokenResponseHandler.java

@@ -46,7 +46,6 @@
 import com.microsoft.identity.common.java.util.ResultUtil;
 import com.microsoft.identity.common.java.util.StringUtil;
 
-
 import java.net.HttpURLConnection;
 import java.util.HashMap;
 import java.util.List;

common4j/src/test/com/microsoft/identity/common/java/providers/microsoft/microsoftsts/MicrosoftStsTokenResponseHandlerTest.java

@@ -145,47 +145,36 @@ public void testHandleTokenResponse_withUrlEncodedClientDataHeader_attributesEmi
         // eSTS URL-encodes pipe separators in the response header (e.g. "%7C" for "|").
         // Real-world example: x-ms-clientdata=[m%7C0x800482A5%7C%7Cmicrosoftonline.com%7Cnone]
         final String encodedClientDataHeader = "m%7C0x800482A5%7C%7Cmicrosoftonline.com%7Cnone";
-
         final HashMap<String, List<String>> headers = new HashMap<>();
         headers.put("Content-Type", Collections.singletonList("application/json; charset=utf-8"));
         headers.put(HttpConstants.HeaderField.X_MS_CLIENTDATA,
                 Collections.singletonList(encodedClientDataHeader));
-
         final HttpResponse response = new HttpResponse(200, MOCK_TOKEN_SUCCESS_RESPONSE, headers);
         final MicrosoftStsTokenResponseHandler handler = new MicrosoftStsTokenResponseHandler();
-
         final Span mockSpan = mock(Span.class);
         when(mockSpan.setAttribute(Mockito.anyString(), Mockito.anyString())).thenReturn(mockSpan);
-
         try (MockedStatic<SpanExtension> mockedExtension = Mockito.mockStatic(SpanExtension.class)) {
             mockedExtension.when(SpanExtension::current).thenReturn(mockSpan);
-
             final TokenResult tokenResult = handler.handleTokenResponse(response);
-
             Assert.assertNotNull(tokenResult);
             Assert.assertTrue(tokenResult.getSuccess());
             Assert.assertNotNull(tokenResult.getClientDataInfo());
             verify(mockSpan).setAttribute(AttributeName.server_error.name(), "0x800482A5");
             verify(mockSpan).setAttribute(AttributeName.account_type.name(), "MSA");
         }
     }
-
     @SneakyThrows
     @Test
     public void testHandleTokenResponse_withMalformedPercentEncoding_doesNotCrash() {
         // Lone '%' is invalid percent-encoding; decode should fail gracefully and skip ClientDataInfo.
         final String malformedHeader = "e|AADSTS50058|%ZZ|us|public";
-
         final HashMap<String, List<String>> headers = new HashMap<>();
         headers.put("Content-Type", Collections.singletonList("application/json; charset=utf-8"));
         headers.put(HttpConstants.HeaderField.X_MS_CLIENTDATA,
                 Collections.singletonList(malformedHeader));
-
         final HttpResponse response = new HttpResponse(200, MOCK_TOKEN_SUCCESS_RESPONSE, headers);
         final MicrosoftStsTokenResponseHandler handler = new MicrosoftStsTokenResponseHandler();
-
         final TokenResult tokenResult = handler.handleTokenResponse(response);
-
         Assert.assertNotNull(tokenResult);
         Assert.assertTrue(tokenResult.getSuccess());
         // Malformed encoding => null ClientDataInfo, but token result still valid.

common4j/versioning/version.properties

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

versioning/version.properties

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