Upgrade MSAL dependencies to 4.59.1

[Haard30]

Upgrading MSAL version to 4.59.1

Why not the latest 4.60.3?

There are few unresolved issues in 4.60.x version of MSAL for example:
[Bug] Broker-based flows fail for MSA accounts with latest MSAL.

Also, looking at the changelog, there does not seem to be an immediate need to move to 4.60.x, hence updating to a more stable version for now.

Selected improvements and bug fixes which can be helpful:

4.59.0 -
• Fixed a scenario when the same tokens are cached under different cache keys when an identity provider sends scopes in a different order.
• Fixed SemaphoreFullException happening in managed identity flows.

4.58.1 -
• Fixed an issue with sending an incorrect operating system descriptor in silent flows on Mac.
• Added WithForceRefresh support for silent flows using the Windows broker.

4.57 -
• Included missing Windows broker-related exception data when serializing MSAL exceptions.
• Added additional logging in the cache.

4.56 -
• Added AuthenticationResult.AuthenticationResultMetadata.Telemetry that currently contains telemetry from the Windows broker (WAM).
• Added clarity to the Windows broker logs.

Upgrading microsoft-authentication-extensions-for-dotnet

• The MSAL Extensions repository has been archived and is maintained alongside MSAL.
• Microsoft.Identity.Client.Extensions.Msal is now in sync with MSAL version. Note that repository mentions there are no breaking changes between 2.x and 4.x pic below)

Testing

Benchmark test

4.59.1

BenchmarkDotNet=v0.13.2, OS=Windows 11 (10.0.22631.3447)
Intel Core i7-10700 CPU 2.90GHz, 1 CPU, 16 logical and 8 physical cores
.NET SDK=8.0.204
  [Host]     : .NET 6.0.29 (6.0.2924.17105), X64 RyuJIT AVX2
  DefaultJob : .NET 6.0.29 (6.0.2924.17105), X64 RyuJIT AVX2

Method	Mean	Error	StdDev
NativeBrokerBenchmark	56.71 ms	0.307 ms	0.272 ms
ManagedBrokerBenchmark	56.89 ms	1.070 ms	0.948 ms

4.55.0

BenchmarkDotNet=v0.13.2, OS=Windows 11 (10.0.22631.3447)
Intel Core i7-10700 CPU 2.90GHz, 1 CPU, 16 logical and 8 physical cores
.NET SDK=8.0.204
  [Host]     : .NET 6.0.29 (6.0.2924.17105), X64 RyuJIT AVX2
  DefaultJob : .NET 6.0.29 (6.0.2924.17105), X64 RyuJIT AVX2

Method	Mean	Error	StdDev
NativeBrokerBenchmark	55.98 ms	0.265 ms	0.247 ms
ManagedBrokerBenchmark	55.50 ms	0.512 ms	0.453 ms

Other

Tested following commands on both mac and windows:

azureauth --help
azureauth aad --help
azureauth ado pat --help
azureauth ado token --help
azureauth ado pat scopes --help
azureauth info --help
azureauth ado token
azureauth ado pat
azureauth aad (with different output and auth modes)

[mijpeterson]

It's nice to see many of our dependency versions converging.

[evandigby]

Approving as my question is non-blocking--just something to keep in mind as we merge.

[mvanchaa]

Thanks for adding a detailed description, @Haard30! Super helpful!