Skip to content

Commit

Permalink
Use IConfigurationSection in EnableTokenAcquistion Internal and fix 1…
Browse files Browse the repository at this point in the history
…583 (#1582)

* use IConfigSection

* fix warning
  • Loading branch information
jennyf19 authored Jan 7, 2022
1 parent f2888f2 commit 20013e4
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/Microsoft.Identity.Web/WebApiExtensions/WebApiBuilders.cs
Original file line number Diff line number Diff line change
Expand Up @@ -23,28 +23,29 @@ public static class WebApiBuilders
/// <param name="configureConfidentialClientApplicationOptions">The action to configure <see cref="ConfidentialClientApplicationOptions"/>.</param>
/// <param name="authenticationScheme">Authentication scheme.</param>
/// <param name="services">The services being configured.</param>
/// <param name="configuration">Configuration.</param>
/// <param name="configuration">IConfigurationSection.</param>
/// <returns>The authentication builder to chain.</returns>
public static MicrosoftIdentityAppCallsWebApiAuthenticationBuilder EnableTokenAcquisition(
Action<ConfidentialClientApplicationOptions> configureConfidentialClientApplicationOptions,
string authenticationScheme,
IServiceCollection services,
IConfiguration? configuration)
IConfigurationSection? configuration)
{
services.AddOptions<ConfidentialClientApplicationOptions>(authenticationScheme)
.Configure<IOptionsMonitor<MergedOptions>>((
ccaOptions, mergedOptionsMonitor) =>
{
configureConfidentialClientApplicationOptions(ccaOptions);
MergedOptions mergedOptions = mergedOptionsMonitor.Get(authenticationScheme);
configuration.Bind(mergedOptions);
MergedOptions.UpdateMergedOptionsFromConfidentialClientApplicationOptions(ccaOptions, mergedOptions);
});

services.AddTokenAcquisition();

return new MicrosoftIdentityAppCallsWebApiAuthenticationBuilder(
services,
configuration as IConfigurationSection);
configuration);
}
}
}

0 comments on commit 20013e4

Please sign in to comment.