Merge pull request #112 from marcominerva/develop

Set AuthenticationType of JwtBearer.TokenValidationParameters to SchemeName from the settings
marcominerva · Jul 11, 2024 · 2b9e0c5 · 2b9e0c5
2 parents 5077548 + 418f63e
commit 2b9e0c5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/SimpleAuthentication/JwtBearer/JwtBearerService.cs b/src/SimpleAuthentication/JwtBearer/JwtBearerService.cs
@@ -12,7 +12,7 @@ internal class JwtBearerService(IOptions<JwtBearerSettings> jwtBearerSettingsOpt
 
     public string CreateToken(string userName, IList<Claim>? claims = null, string? issuer = null, string? audience = null, DateTime? absoluteExpiration = null)
     {
-        claims ??= new List<Claim>();
+        claims ??= [];
         claims.Update(jwtBearerSettings.NameClaimType, userName);
         claims.Update(JwtRegisteredClaimNames.Jti, Guid.NewGuid().ToString());
         claims.Remove(JwtRegisteredClaimNames.Aud);
@@ -37,6 +37,7 @@ public ClaimsPrincipal ValidateToken(string token, bool validateLifetime)
     {
         var tokenValidationParameters = new TokenValidationParameters
         {
+            AuthenticationType = jwtBearerSettings.SchemeName,
             NameClaimType = jwtBearerSettings.NameClaimType,
             RoleClaimType = jwtBearerSettings.RoleClaimType,
             ValidateIssuer = jwtBearerSettings.Issuers?.Any() ?? false,

diff --git a/src/SimpleAuthentication/SimpleAuthenticationExtensions.cs b/src/SimpleAuthentication/SimpleAuthenticationExtensions.cs
@@ -96,6 +96,7 @@ static void CheckAddJwtBearer(AuthenticationBuilder builder, IConfigurationSecti
             {
                 options.TokenValidationParameters = new()
                 {
+                    AuthenticationType = settings.SchemeName,
                     NameClaimType = settings.NameClaimType,
                     RoleClaimType = settings.RoleClaimType,
                     ValidateIssuer = settings.Issuers?.Any() ?? false,