-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Fix https://github.com/aspnet/AspNetCore-ManualTests/issues/828 #36451
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix https://github.com/aspnet/AspNetCore-ManualTests/issues/828 #36451
Conversation
javiercn
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good.
@jmprieur I trust you've done due diligence in validating the changes.
|
/backport to main |
|
@jmprieur can you fill out the servicing template for this PR, then we'll just need to ping Kevin on this PR to get approval to merge for 6.0 (or maybe this should go to rc2/thru tactics) Customer Impact Testing Risk |
|
@HaoK : I just improved the description a bit. Is it ok? |
|
Yep I think that should be sufficient, @Pilchie can we get this merged for 6.0? |
|
Approved for .NET 6. |
|
@dotnet/aspnet-build can we get this merged? |
Fix https://github.com/aspnet/AspNetCore-ManualTests/issues/828#issuecomment-916732011
PR Title
Fixes the required scopes in the controller for b2c in the Blasorwasm hosted server
PR Description
Fixes the scopes in the controller in the Blasorwasm hosted server (the resource is in the b2c section).
Without this fix, when creating a b2c blazorwasm hosted application the controller won't find the resource as it's in the "AzureADB2C" section, whereas it was searched in the "AzureAD" section.
For repros and details see: Fixes https://github.com/aspnet/AspNetCore-ManualTests/issues/828
@Claire-kangkang, @HaoK