Skip to content

Commit

Permalink
Fix configuration key in error messages (dotnet#3630)
Browse files Browse the repository at this point in the history
At some point during development of dotnet#3443 the configuration key was changed and these error messages were missed.
  • Loading branch information
drewnoakes authored Apr 12, 2024
1 parent 89e9d9f commit fbdd392
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/Aspire.Dashboard/Configuration/DashboardOptions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -209,10 +209,11 @@ public string[] GetUsernameClaimTypes()
internal bool TryParseOptions([NotNullWhen(false)] out IEnumerable<string>? errorMessages)
{
List<string>? messages = null;

if (string.IsNullOrWhiteSpace(NameClaimType))
{
messages ??= [];
messages.Add("OpenID Connect claim type for name not configured. Specify a Dashboard:OpenIdConnect:NameClaimType value.");
messages.Add("OpenID Connect claim type for name not configured. Specify a Dashboard:Frontend:OpenIdConnect:NameClaimType value.");
}
else
{
Expand All @@ -222,7 +223,7 @@ internal bool TryParseOptions([NotNullWhen(false)] out IEnumerable<string>? erro
if (string.IsNullOrWhiteSpace(UsernameClaimType))
{
messages ??= [];
messages.Add("OpenID Connect claim type for username not configured. Specify a Dashboard:OpenIdConnect:UsernameClaimType value.");
messages.Add("OpenID Connect claim type for username not configured. Specify a Dashboard:Frontend:OpenIdConnect:UsernameClaimType value.");
}
else
{
Expand Down

0 comments on commit fbdd392

Please sign in to comment.