Remove experimental flag requirement from IAuthenticationOperation (#5699)

Copilot · bgavrilMS · web-flow · commit 0e7b12a78a92 · 2026-02-03T15:51:36.000-08:00
* Initial plan

* Remove experimental flag from IAuthenticationOperation and WithAuthenticationExtension

Co-authored-by: bgavrilMS &lt;12273384+bgavrilMS@users.noreply.github.com&gt;

* Update AuthenticationOperationTests to remove experimental flag requirement

Co-authored-by: bgavrilMS &lt;12273384+bgavrilMS@users.noreply.github.com&gt;

---------

Co-authored-by: copilot-swe-agent[bot] &lt;198982749+Copilot@users.noreply.github.com&gt;
Co-authored-by: bgavrilMS &lt;12273384+bgavrilMS@users.noreply.github.com&gt;
Co-authored-by: Bogdan Gavril &lt;bogavril@microsoft.com&gt;
diff --git a/src/client/Microsoft.Identity.Client/ApiConfig/AbstractAcquireTokenParameterBuilder.cs b/src/client/Microsoft.Identity.Client/ApiConfig/AbstractAcquireTokenParameterBuilder.cs
@@ -343,7 +343,6 @@ public T WithB2CAuthority(string authorityUri)
 
         internal /* for testing */ T WithAuthenticationOperation(IAuthenticationOperation authOperation)
         {
-            ValidateUseOfExperimentalFeature();
             CommonParameters.AuthenticationOperation = authOperation ?? throw new ArgumentNullException(nameof(authOperation));
             return this as T;
         }
diff --git a/src/client/Microsoft.Identity.Client/Extensibility/AbstractConfidentialClientAcquireTokenParameterBuilderExtension.cs b/src/client/Microsoft.Identity.Client/Extensibility/AbstractConfidentialClientAcquireTokenParameterBuilderExtension.cs
@@ -117,8 +117,6 @@ public static AbstractAcquireTokenParameterBuilder<T> WithAdditionalCacheParamet
                 return builder;
             }
 
-            builder.ValidateUseOfExperimentalFeature();
-
             //Check if the cache parameters are already initialized, if so, add to the existing list
             if (builder.CommonParameters.AdditionalCacheParameters != null)
             {
diff --git a/tests/Microsoft.Identity.Test.Unit/AuthExtension/AuthenticationOperationTests.cs b/tests/Microsoft.Identity.Test.Unit/AuthExtension/AuthenticationOperationTests.cs
@@ -29,7 +29,6 @@ public async Task Should_UseCustomRequestHeaders_And_StoreAdditionalParameters()
                     ConfidentialClientApplicationBuilder.Create(TestConstants.ClientId)
                                                               .WithClientSecret(TestConstants.ClientSecret)
                                                               .WithHttpManager(httpManager)
-                                                              .WithExperimentalFeatures(true)
                                                               .BuildConcrete();
 
                 HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, new Uri(ProtectedUrl));
@@ -78,7 +77,6 @@ public async Task Should_UseCustomRequestHeaders_And_StoreAdditionalParametersWi
                     ConfidentialClientApplicationBuilder.Create(TestConstants.ClientId)
                                                               .WithClientSecret(TestConstants.ClientSecret)
                                                               .WithHttpManager(httpManager)
-                                                              .WithExperimentalFeatures(true)
                                                               .BuildConcrete();
 
                 HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, new Uri(ProtectedUrl));
@@ -158,7 +156,6 @@ public async Task Should_UseEmptyExtension_And_Parameters()
                     ConfidentialClientApplicationBuilder.Create(TestConstants.ClientId)
                                                               .WithClientSecret(TestConstants.ClientSecret)
                                                               .WithHttpManager(httpManager)
-                                                              .WithExperimentalFeatures(true)
                                                               .BuildConcrete();
 
                 HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, new Uri(ProtectedUrl));

Original file line number	Diff line number	Diff line change
`@@ -343,7 +343,6 @@ public T WithB2CAuthority(string authorityUri)`
`343`	`343`
`344`	`344`	`internal /* for testing */ T WithAuthenticationOperation(IAuthenticationOperation authOperation)`
`345`	`345`	`{`
`346`		`- ValidateUseOfExperimentalFeature();`
`347`	`346`	`CommonParameters.AuthenticationOperation = authOperation ?? throw new ArgumentNullException(nameof(authOperation));`
`348`	`347`	`return this as T;`
`349`	`348`	`}`
Original file line number	Diff line number	Diff line change
`@@ -117,8 +117,6 @@ public static AbstractAcquireTokenParameterBuilder<T> WithAdditionalCacheParamet`
`117`	`117`	`return builder;`
`118`	`118`	`}`
`119`	`119`
`120`		`- builder.ValidateUseOfExperimentalFeature();`
`121`		`-`
`122`	`120`	`//Check if the cache parameters are already initialized, if so, add to the existing list`
`123`	`121`	`if (builder.CommonParameters.AdditionalCacheParameters != null)`
`124`	`122`	`{`