Skip to content
This repository was archived by the owner on Dec 13, 2018. It is now read-only.

Commit 658f462

Browse files
committed
#1208 Clean up JWT and OIDC issuer handling
1 parent 1c30f33 commit 658f462

File tree

2 files changed

+8
-19
lines changed

2 files changed

+8
-19
lines changed

src/Microsoft.AspNetCore.Authentication.JwtBearer/JwtBearerHandler.cs

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -90,17 +90,11 @@ protected override async Task<AuthenticateResult> HandleAuthenticateAsync()
9090
var validationParameters = Options.TokenValidationParameters.Clone();
9191
if (_configuration != null)
9292
{
93-
if (validationParameters.ValidIssuer == null && !string.IsNullOrEmpty(_configuration.Issuer))
94-
{
95-
validationParameters.ValidIssuer = _configuration.Issuer;
96-
}
97-
else
98-
{
99-
var issuers = new[] { _configuration.Issuer };
100-
validationParameters.ValidIssuers = (validationParameters.ValidIssuers == null ? issuers : validationParameters.ValidIssuers.Concat(issuers));
101-
}
93+
var issuers = new[] { _configuration.Issuer };
94+
validationParameters.ValidIssuers = validationParameters.ValidIssuers?.Concat(issuers) ?? issuers;
10295

103-
validationParameters.IssuerSigningKeys = (validationParameters.IssuerSigningKeys == null ? _configuration.SigningKeys : validationParameters.IssuerSigningKeys.Concat(_configuration.SigningKeys));
96+
validationParameters.IssuerSigningKeys = validationParameters.IssuerSigningKeys?.Concat(_configuration.SigningKeys)
97+
?? _configuration.SigningKeys;
10498
}
10599

106100
List<Exception> validationFailures = null;

src/Microsoft.AspNetCore.Authentication.OpenIdConnect/OpenIdConnectHandler.cs

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1137,16 +1137,11 @@ private ClaimsPrincipal ValidateToken(string idToken, AuthenticationProperties p
11371137

11381138
if (_configuration != null)
11391139
{
1140-
if (string.IsNullOrEmpty(validationParameters.ValidIssuer))
1141-
{
1142-
validationParameters.ValidIssuer = _configuration.Issuer;
1143-
}
1144-
else if (!string.IsNullOrEmpty(_configuration.Issuer))
1145-
{
1146-
validationParameters.ValidIssuers = validationParameters.ValidIssuers?.Concat(new[] { _configuration.Issuer }) ?? new[] { _configuration.Issuer };
1147-
}
1140+
var issuer = new[] { _configuration.Issuer };
1141+
validationParameters.ValidIssuers = validationParameters.ValidIssuers?.Concat(issuer) ?? issuer;
11481142

1149-
validationParameters.IssuerSigningKeys = validationParameters.IssuerSigningKeys?.Concat(_configuration.SigningKeys) ?? _configuration.SigningKeys;
1143+
validationParameters.IssuerSigningKeys = validationParameters.IssuerSigningKeys?.Concat(_configuration.SigningKeys)
1144+
?? _configuration.SigningKeys;
11501145
}
11511146

11521147
var principal = Options.SecurityTokenValidator.ValidateToken(idToken, validationParameters, out SecurityToken validatedToken);

0 commit comments

Comments
 (0)