Skip to content

Commit 5a6a19c

Browse files
authored
Merge pull request #2 from GuyPago/infra/avoid-double-service-container
Replace `BuildServiceProvider` call
2 parents 909a083 + ecf9500 commit 5a6a19c

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

aspnetcore.ntier.API/Program.cs

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,18 @@
1010

1111
var builder = WebApplication.CreateBuilder(args);
1212

13-
// Add services to the container.
13+
// Add configuration
14+
var jwtConfigSection = builder.Configuration.GetSection(nameof(JwtSettings));
15+
builder.Services.Configure<JwtSettings>(jwtConfigSection);
1416

17+
// Add services to the container.
1518
builder.Services.AddControllers();
1619

1720
builder.Services
1821
.AddAuthentication(JwtBearerDefaults.AuthenticationScheme)
1922
.AddJwtBearer(options =>
2023
{
21-
var jwtSettings = builder.Services.BuildServiceProvider().GetRequiredService<IOptions<JwtSettings>>().Value;
24+
var jwtSettings = jwtConfigSection.Get<JwtSettings>();
2225

2326
options.TokenValidationParameters = new TokenValidationParameters
2427
{
@@ -29,7 +32,6 @@
2932
};
3033
});
3134

32-
builder.Services.Configure<JwtSettings>(builder.Configuration.GetSection(nameof(JwtSettings)));
3335
builder.Services.RegisterDALDependencies(builder.Configuration);
3436
builder.Services.RegisterBLLDependencies(builder.Configuration);
3537

0 commit comments

Comments
 (0)