Skip to content

Commit 2b9e0c5

Browse files
authored
Merge pull request #112 from marcominerva/develop
Set AuthenticationType of JwtBearer.TokenValidationParameters to SchemeName from the settings
2 parents 5077548 + 418f63e commit 2b9e0c5

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/SimpleAuthentication/JwtBearer/JwtBearerService.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ internal class JwtBearerService(IOptions<JwtBearerSettings> jwtBearerSettingsOpt
1212

1313
public string CreateToken(string userName, IList<Claim>? claims = null, string? issuer = null, string? audience = null, DateTime? absoluteExpiration = null)
1414
{
15-
claims ??= new List<Claim>();
15+
claims ??= [];
1616
claims.Update(jwtBearerSettings.NameClaimType, userName);
1717
claims.Update(JwtRegisteredClaimNames.Jti, Guid.NewGuid().ToString());
1818
claims.Remove(JwtRegisteredClaimNames.Aud);
@@ -37,6 +37,7 @@ public ClaimsPrincipal ValidateToken(string token, bool validateLifetime)
3737
{
3838
var tokenValidationParameters = new TokenValidationParameters
3939
{
40+
AuthenticationType = jwtBearerSettings.SchemeName,
4041
NameClaimType = jwtBearerSettings.NameClaimType,
4142
RoleClaimType = jwtBearerSettings.RoleClaimType,
4243
ValidateIssuer = jwtBearerSettings.Issuers?.Any() ?? false,

src/SimpleAuthentication/SimpleAuthenticationExtensions.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ static void CheckAddJwtBearer(AuthenticationBuilder builder, IConfigurationSecti
9696
{
9797
options.TokenValidationParameters = new()
9898
{
99+
AuthenticationType = settings.SchemeName,
99100
NameClaimType = settings.NameClaimType,
100101
RoleClaimType = settings.RoleClaimType,
101102
ValidateIssuer = settings.Issuers?.Any() ?? false,

0 commit comments

Comments
 (0)