Skip to content

Commit 76af292

Browse files
JonathanCrdCopilot
andauthored
[Identity] Update Broker's Azure.Identity dependency to unlock beta release (Azure#51243)
* Fix tests with MacBrokerRedirectUri with temporary constant while Azure.Identity gets that feature out. * Depend on latest Identity Beta to unblock release * Update sdk/identity/Azure.Identity.Broker/src/TemporaryConstants.cs Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
1 parent 2da6e62 commit 76af292

File tree

4 files changed

+20
-4
lines changed

4 files changed

+20
-4
lines changed

sdk/identity/Azure.Identity.Broker/src/Azure.Identity.Broker.csproj

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,7 @@
1313
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
1414
</PropertyGroup>
1515
<ItemGroup>
16-
<!--<PackageReference Include="Azure.Identity" VersionOverride="1.14.0-beta.3" />-->
17-
<ProjectReference Include="..\..\Azure.Identity\src\Azure.Identity.csproj" />
16+
<PackageReference Include="Azure.Identity" VersionOverride="1.14.0-beta.4" />
1817
<PackageReference Include="Microsoft.Identity.Client.Broker" />
1918
</ItemGroup>
2019
</Project>

sdk/identity/Azure.Identity.Broker/src/DevelopmentBrokerOptions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public DevelopmentBrokerOptions() : base()
3636
// Set default value for UseDefaultBrokerAccount on macOS
3737
if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX))
3838
{
39-
RedirectUri = new(Constants.MacBrokerRedirectUri);
39+
RedirectUri = new(TemporaryConstants.MacBrokerRedirectUri);
4040
}
4141
}
4242

sdk/identity/Azure.Identity.Broker/src/InteractiveBrowserCredentialBrokerOptions.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public InteractiveBrowserCredentialBrokerOptions(IntPtr parentWindowHandle) : ba
3737
// Set default value for UseDefaultBrokerAccount on macOS
3838
if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX))
3939
{
40-
RedirectUri = new(Constants.MacBrokerRedirectUri);
40+
RedirectUri = new(TemporaryConstants.MacBrokerRedirectUri);
4141
}
4242
}
4343

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
// Copyright (c) Microsoft Corporation. All rights reserved.
2+
// Licensed under the MIT License.
3+
4+
namespace Azure.Identity.Broker
5+
{
6+
/// <summary>
7+
/// Temporary constants for Azure.Identity.Broker until Azure.Identity dependency is updated.
8+
/// </summary>
9+
internal static class TemporaryConstants
10+
{
11+
/// <summary>
12+
/// Redirect URI for macOS broker authentication.
13+
/// TODO: Remove this once Azure.Identity dependency includes Constants.MacBrokerRedirectUri
14+
/// </summary>
15+
internal const string MacBrokerRedirectUri = "msauth.com.msauth.unsignedapp://auth";
16+
}
17+
}

0 commit comments

Comments
 (0)