Skip to content

Commit

Permalink
Fix Id Web build (#2991)
Browse files Browse the repository at this point in the history
Co-authored-by: Franco Fung <francofung@microsoft.com>
  • Loading branch information
FuPingFranco and Franco Fung authored Aug 21, 2024
1 parent 23f6d94 commit fcd715e
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 1 deletion.
2 changes: 2 additions & 0 deletions src/Microsoft.Identity.Web.TokenAcquisition/MergedOptions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -222,6 +222,8 @@ internal static void UpdateMergedOptionsFromMicrosoftIdentityOptions(MicrosoftId
mergedOptions.AdditionalAuthorizationParameters.Add(parameter.Key, parameter.Value);
}
}

mergedOptions.PushedAuthorizationBehavior = microsoftIdentityOptions.PushedAuthorizationBehavior;
#endif
mergedOptions.TokenValidationParameters = microsoftIdentityOptions.TokenValidationParameters.Clone();
mergedOptions.UsePkce |= microsoftIdentityOptions.UsePkce;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -544,6 +544,8 @@ internal static void PopulateOpenIdOptionsFromMergedOptions(
options.AdditionalAuthorizationParameters.Add(parameter.Key, parameter.Value);
}
}

options.PushedAuthorizationBehavior = mergedOptions.PushedAuthorizationBehavior;
#endif
}
}
Expand Down
2 changes: 1 addition & 1 deletion tests/Microsoft.Identity.Web.Test/WebAppExtensionsTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -913,7 +913,7 @@ public void PreventChangesInOpenIdConnectOptionsToBeOverlooked()
// and diff the files to find what are the new properties.
int numberOfProperties = typeof(OpenIdConnectOptions).GetProperties(System.Reflection.BindingFlags.Public | System.Reflection.BindingFlags.Instance).Length;
#if NET9_0_OR_GREATER
int expectedNumberOfProperties = 61;
int expectedNumberOfProperties = 62;
//string filePath = @"C:\temp\net9.txt";
#elif NET8_0
int expectedNumberOfProperties = 60;
Expand Down

0 comments on commit fcd715e

Please sign in to comment.