Skip to content

Remove experimental flag requirement from IAuthenticationOperation#5699

Merged
gladjohn merged 4 commits intomainfrom
copilot/remove-experimental-flag-authentication
Feb 3, 2026
Merged

Remove experimental flag requirement from IAuthenticationOperation#5699
gladjohn merged 4 commits intomainfrom
copilot/remove-experimental-flag-authentication

Conversation

Copy link
Contributor

Copilot AI commented Feb 2, 2026

Changes proposed in this request

Removed experimental feature gate from IAuthenticationOperation and WithAuthenticationExtension. Developers can now use custom authentication operations without calling .WithExperimentalFeatures(true).

Two validations removed:

  • WithAuthenticationOperation in AbstractAcquireTokenParameterBuilder.cs
  • WithAdditionalCacheParameters in AbstractConfidentialClientAcquireTokenParameterBuilderExtension.cs

Updated tests in AuthenticationOperationTests.cs to remove .WithExperimentalFeatures(true) calls, demonstrating that the feature works without experimental flags.

Before:

var app = ConfidentialClientApplicationBuilder
    .Create(clientId)
    .WithClientSecret(secret)
    .WithExperimentalFeatures(true)  // Required
    .Build();

app.AcquireTokenForClient(scopes)
    .WithAuthenticationExtension(authExtension)
    .ExecuteAsync();

After:

var app = ConfidentialClientApplicationBuilder
    .Create(clientId)
    .WithClientSecret(secret)
    .Build();

app.AcquireTokenForClient(scopes)
    .WithAuthenticationExtension(authExtension)  // No experimental flag needed
    .ExecuteAsync();

Testing

Updated existing tests in AuthenticationOperationTests.cs:

  • Should_UseCustomRequestHeaders_And_StoreAdditionalParameters
  • Should_UseCustomRequestHeaders_And_StoreAdditionalParametersWithCaching
  • Should_UseEmptyExtension_And_Parameters

All tests now run without .WithExperimentalFeatures(true) and pass successfully, proving that WithAuthenticationExtension works correctly for:

  • AuthenticationOperation alone
  • AdditionalCacheParameters alone
  • Both combined

Performance impact

None. Removed validation calls, no new logic added.

Documentation

  • All relevant documentation is updated.
Original prompt

This section details on the original issue you should resolve

<issue_title>[Feature Request] Remove experimental flag from IAuthenticationOperation</issue_title>
<issue_description>### MSAL client type

Confidential

Problem statement

WithAuthenticationExtension should not be experimental, to avoid app developers having to set this flag.

Proposed solution

No response

Alternatives

No response</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…ticationExtension

Co-authored-by: bgavrilMS <12273384+bgavrilMS@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove experimental flag from IAuthenticationOperation Remove experimental flag requirement from IAuthenticationOperation Feb 2, 2026
Copilot AI requested a review from bgavrilMS February 2, 2026 17:35
…rement

Co-authored-by: bgavrilMS <12273384+bgavrilMS@users.noreply.github.com>
@bgavrilMS bgavrilMS marked this pull request as ready for review February 2, 2026 19:21
@bgavrilMS bgavrilMS requested a review from a team as a code owner February 2, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] Remove experimental flag from IAuthenticationOperation

4 participants