Skip to content

Fix loading dotnet user-jwts config - #59425

Merged
halter73 merged 1 commit into
mainfrom
halter73/58996
Dec 13, 2024
Merged

Fix loading dotnet user-jwts config#59425
halter73 merged 1 commit into
mainfrom
halter73/58996

Conversation

@halter73

Copy link
Copy Markdown
Member

Fixes #58996

@ghost ghost added the area-commandlinetools Includes: Command line tools, dotnet-dev-certs, dotnet-user-jwts, and OpenAPI label Dec 11, 2024
@halter73
halter73 requested review from MackinnonBuck and captainsafia and removed request for captainsafia December 11, 2024 02:46
ValidAudience = audience,
ValidateIssuerSigningKey = true,
IssuerSigningKeys = GetIssuerSigningKeys(configSection, issuers),
IssuerSigningKeys = GetIssuerSigningKeys(configSection, [issuer, ..issuers]),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fact that we didn't pass the standalone issuer here was always a bug, correct?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was a regression caused in .NET 9 by #52821. I plan to backport this once approved.

@halter73
halter73 merged commit 97d1829 into main Dec 13, 2024
@halter73
halter73 deleted the halter73/58996 branch December 13, 2024 19:04
@halter73

Copy link
Copy Markdown
Member Author

/backport to release/9.0

@dotnet-policy-service dotnet-policy-service Bot added this to the 10.0-preview1 milestone Dec 13, 2024
@github-actions

Copy link
Copy Markdown
Contributor

Started backporting to release/9.0: https://github.com/dotnet/aspnetcore/actions/runs/12321516859

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-commandlinetools Includes: Command line tools, dotnet-dev-certs, dotnet-user-jwts, and OpenAPI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

.NET 9: dotnet user-jwts not working

2 participants