Skip to content

Commit 2c212ba

Browse files
authored
Merge pull request CactuseSecurity#3629 from tpurschke/develop
hotfix audience validation in develop
2 parents 4c9f655 + a1378e1 commit 2c212ba

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

roles/lib/files/FWO.Middleware.Client/JwtReader.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,13 +63,12 @@ public async Task<bool> Validate()
6363
{
6464
RequireExpirationTime = true,
6565
RequireSignedTokens = true,
66+
ValidateLifetime = true,
6667
ValidateAudience = true,
68+
ValidAudiences = [FWO.Basics.JwtConstants.Audience],
6769
ValidateIssuer = true,
68-
ValidateLifetime = true,
69-
ValidAudience = FWO.Basics.JwtConstants.Audience,
7070
ValidIssuer = FWO.Basics.JwtConstants.Issuer,
7171
IssuerSigningKey = jwtPublicKey,
72-
7372
};
7473

7574
JsonWebTokenHandler handler = new ();

roles/middleware/files/FWO.Middleware.Server/Program.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,9 @@ await Task.Factory.StartNew(async() =>
128128
RequireExpirationTime = true,
129129
RequireSignedTokens = true,
130130
ValidateAudience = true,
131-
ValidateIssuer = false,
131+
ValidAudiences = [FWO.Basics.JwtConstants.Audience],
132+
ValidateIssuer = true,
133+
ValidIssuers = [FWO.Basics.JwtConstants.Issuer],
132134
ValidateLifetime = true,
133135
RoleClaimType = "role",
134136
IssuerSigningKey = ConfigFile.JwtPublicKey

0 commit comments

Comments
 (0)