Skip to content

Conversation

gladjohn
Copy link
Contributor

Fixes #5352

Changes proposed in this request
This pull request removes the use of the WithExperimentalFeatures method across the codebase, indicating that the associated features are no longer considered experimental. It also updates the test cases to reflect these changes and introduces some minor refactoring in related code.

Removal of WithExperimentalFeatures on WithMtlsProofOfPossession API:

  • The WithExperimentalFeatures method has been removed from various test cases in the MtlsPopTests, ManagedIdentityTests, and other test files, aligning the code with the updated feature status. [1] [2] [3] [4] [5] [6] and others)

Refactoring and Updates:

  • Removed the ValidateUseOfExperimentalFeature call in the WithMtlsProofOfPossession method in AcquireTokenForClientParameterBuilder, simplifying the method implementation.
  • Updated the ExperimentalFeatureTests to use WithSignedHttpRequestProofOfPossession instead of WithMtlsProofOfPossession, reflecting the removal of experimental status.
  • Minor refactoring in MtlsPopTests to simplify variable declarations and improve readability.

These changes ensure that the codebase is consistent with the updated feature status and improve maintainability by removing experimental flags and redundant code.

Testing
tests

Performance impact
n/a

Documentation

  • All relevant documentation is updated.

@gladjohn gladjohn requested a review from a team as a code owner July 23, 2025 19:06
@gladjohn gladjohn changed the title Removal of WithExperimentalFeatures on WithMtlsProofOfPossession API: Removal of ExperimentalFeatures flag on WithMtlsProofOfPossession API: Jul 23, 2025
@gladjohn gladjohn merged commit 48b002d into main Jul 23, 2025
11 checks passed
@gladjohn gladjohn deleted the gladjohn/msal_sni_ga_new branch July 23, 2025 22:13
lilyydu pushed a commit to microsoft/teams-ai that referenced this pull request Aug 20, 2025
#minor 
Updated [Azure.Identity](https://github.com/Azure/azure-sdk-for-net)
from 1.14.2 to 1.15.0.

<details>
<summary>Release notes</summary>

_Sourced from [Azure.Identity's
releases](https://github.com/Azure/azure-sdk-for-net/releases)._

No release notes found for this version range.

Commits viewable in [compare
view](https://github.com/Azure/azure-sdk-for-net/commits).
</details>

Updated
[Microsoft.Identity.Client](https://github.com/AzureAD/microsoft-authentication-library-for-dotnet)
from 4.74.1 to 4.76.0.

<details>
<summary>Release notes</summary>

_Sourced from [Microsoft.Identity.Client's
releases](https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/releases)._

## 4.76.0

## What's Changed
* Removal of `ExperimentalFeatures` flag on `WithMtlsProofOfPossession`
API: by @​gladjohn in
AzureAD/microsoft-authentication-library-for-dotnet#5402
* #​5400 Fixing issue that leads to multiple active access tokens in the
cache for non-tenanted oidc authority by @​andkorsh in
AzureAD/microsoft-authentication-library-for-dotnet#5401
* Add Service Fabric token revocation support by @​gladjohn in
AzureAD/microsoft-authentication-library-for-dotnet#5421
* Update NativeInterop package version to 0.19.4 by @​ashok672 in
AzureAD/microsoft-authentication-library-for-dotnet#5434
* Adding WithExtraBodyParameters api by @​trwalke in
AzureAD/microsoft-authentication-library-for-dotnet#5389
* Enable mTLS Proof‑of‑Possession for Client‑Assertion Delegates by
@​gladjohn in
AzureAD/microsoft-authentication-library-for-dotnet#5409

## New Contributors
* @​andkorsh made their first contribution in
AzureAD/microsoft-authentication-library-for-dotnet#5401

**Full Changelog**:
AzureAD/microsoft-authentication-library-for-dotnet@4.74.1...4.76.0

Commits viewable in [compare
view](AzureAD/microsoft-authentication-library-for-dotnet@4.74.1...4.76.0).
</details>

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore <dependency name> major version` will close this
group update PR and stop Dependabot creating any more for the specific
dependency's major version (unless you unignore this specific
dependency's major version or upgrade to it yourself)
- `@dependabot ignore <dependency name> minor version` will close this
group update PR and stop Dependabot creating any more for the specific
dependency's minor version (unless you unignore this specific
dependency's minor version or upgrade to it yourself)
- `@dependabot ignore <dependency name>` will close this group update PR
and stop Dependabot creating any more for the specific dependency
(unless you unignore this specific dependency or upgrade to it yourself)
- `@dependabot unignore <dependency name>` will remove all of the ignore
conditions of the specified dependency
- `@dependabot unignore <dependency name> <ignore condition>` will
remove the ignore condition of the specified dependency and ignore
conditions


</details>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

[Engineering task] Remove experimental flag on Mtls POP API in CCA + SNI Flows

4 participants