Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026

MSAL 4.82.0-preview-1 resolves CAE support for Managed Identity flows, allowing previously skipped tests to be re-enabled.

Changes

  • Dependency update: Bumped MicrosoftIdentityClientVersion from 4.81.0 to 4.82.0-preview-1 in Directory.Build.props

  • Binding redirects: Updated all Microsoft.Identity.Client binding redirects in OWIN test apps to 4.82.0.0:

    • tests/DevApps/aspnet-mvc/OwinWebApp/Web.config (2 instances)
    • tests/DevApps/aspnet-mvc/OwinWebApi/Web.config (2 instances)
  • Test enablement: Removed Skip attributes from 5 CAE tests that were disabled for issue Disabled MSI tests due to instance discovery changes in MSAL for MSI #3669:

    • ManagedIdentityTests: 4 tests covering bearer header generation, claims-based cache bypass, UAMI caching, and client capabilities forwarding
    • FederatedIdentityCaeTests: 1 test for FIC with claims-based cache bypass

All re-enabled tests pass on .NET 10.0 with the new MSAL version.

Original prompt

Goal: Update Microsoft.Identity.Client to 4.82.0-preview-1 and re-enable Managed Identity CAE-related tests that were previously skipped for issue #3669.

Repository: AzureAD/microsoft-identity-web

Scope of changes

  1. Bump MSAL package version
  • File: Directory.Build.props
  • Change <MicrosoftIdentityClientVersion> from 4.81.0 to 4.82.0-preview-1.
  1. Update binding redirects to match the new version
  • File: tests/DevApps/aspnet-mvc/OwinWebApp/Web.config
    • Update bindingRedirect for Microsoft.Identity.Client to oldVersion="0.0.0.0-4.82.0.0" and newVersion="4.82.0.0".
  • File: tests/DevApps/aspnet-mvc/OwinWebApi/Web.config
    • Update bindingRedirect for Microsoft.Identity.Client to oldVersion="0.0.0.0-4.82.0.0" and newVersion="4.82.0.0".
  1. Re-enable Managed Identity CAE tests (remove Skip for issue 3669)
  • File: tests/Microsoft.Identity.Web.Test/ManagedIdentityCaeTests.cs
    • Remove Skip = "See https://github.com/AzureAD/microsoft-identity-web/issues/3669" from:
      • ManagedIdentity_ReturnsBearerHeader
      • ManagedIdentity_WithClaims_HeaderBypassesCache
      • UserAssigned_MI_Caching_and_Claims
      • SystemAssigned_MSI_Forwards_ClientCapabilities_InQuery
  • File: tests/Microsoft.Identity.Web.Test/FederatedIdentityCaeTests.cs
    • Remove Skip = "See https://github.com/AzureAD/microsoft-identity-web/issues/3669" from ManagedIdentityWithFic_WithClaims_BypassesCache.

Testing

  • Run the affected Managed Identity CAE test classes to confirm they pass with MSAL 4.82.0-preview-1.

If any tests still fail due to incomplete CAE support in the preview, note the failing cases and, if necessary, selectively reapply Skip with an updated rationale referencing the new MSAL build.

This pull request was created from Copilot chat.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: gladjohn <90415114+gladjohn@users.noreply.github.com>
Co-authored-by: gladjohn <90415114+gladjohn@users.noreply.github.com>
@gladjohn gladjohn changed the title [WIP] Update Microsoft.Identity.Client to 4.82.0-preview-1 and enable tests [DRAFT] Update Microsoft.Identity.Client to 4.82.0-preview-1 and enable tests (DO NOT MERGE) Jan 16, 2026
Copilot AI changed the title [DRAFT] Update Microsoft.Identity.Client to 4.82.0-preview-1 and enable tests (DO NOT MERGE) Update MSAL to 4.82.0-preview-1 and re-enable Managed Identity CAE tests Jan 16, 2026
Copilot AI requested a review from gladjohn January 16, 2026 00:02
@gladjohn gladjohn changed the title Update MSAL to 4.82.0-preview-1 and re-enable Managed Identity CAE tests [for testing only - do not merge] Update MSAL to 4.82.0-preview-1 and re-enable Managed Identity CAE tests Jan 16, 2026
@gladjohn
Copy link
Contributor

@trwalke with the preview version of MSAL, CAE tests are passing. I will release MSAL version and then enable these tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants