Conversation
There was a problem hiding this comment.
Pull request overview
This PR overhauls the claim-based authorization documentation set to align terminology (“claim-based”), focus the main article on Blazor (Razor component) scenarios, and add dedicated MVC and Razor Pages variants with updated navigation.
Changes:
- Reworked the main claim-based authorization article to use Blazor-focused examples and added cross-links to new MVC/Razor Pages versions.
- Added new MVC and Razor Pages claim-based authorization articles under their respective security/authorization nodes.
- Updated ToC entries and adjusted “claims-based” → “claim-based” wording in related authorization docs.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| aspnetcore/toc.yml | Adds Razor Pages/MVC “Claim-based authorization” nodes and renames the main entry. |
| aspnetcore/security/authorization/roles.md | Updates sample-app link and adds Blazor Server UseAuthorization requirement guidance. |
| aspnetcore/security/authorization/policies/includes/policies5.md | Updates terminology to “claim-based authorization”. |
| aspnetcore/security/authorization/policies.md | Updates terminology to “claim-based authorization”. |
| aspnetcore/security/authorization/claims.md | Major rewrite of the main claim-based authorization article with Blazor-focused examples and new structure. |
| aspnetcore/razor-pages/security/authorization/claims.md | New Razor Pages-focused claim-based authorization article. |
| aspnetcore/mvc/security/authorization/claims.md | New MVC-focused claim-based authorization article. |
| aspnetcore/blazor/security/index.md | Updates wording to “Claim-based authorization”. |
wadepickett
left a comment
There was a problem hiding this comment.
Approved with some minor suggestions. Looks great. Note the issue on "Founders" vs "Founder" for the policy code vs the text referring to it.
Co-authored-by: Wade Pickett <wpickett@microsoft.com> Co-authored-by: Luke Latham <1622880+guardrex@users.noreply.github.com>
|
Thanks @wadepickett ... I refactored the content on ... and @halter73 should look at that bit to make sure that I'm stating it correctly. The 🧀 moves a bit among BWA, Blazor Server, and MVC/RP. I think what I have is correct based on project template API; however, it's a little tricky given the deltas. Note @halter73 that I'm adding a bit Blazor WASM stating that |
|
@wadepickett ... A few small changes this morning ...
|
Fixes #36933
Fixes #17174 ... Note tho that claim-based authz still relies on a policy-based approach. What is "fixing" that issue is that Razor components (Blazor) are demoed as the focused UI stack approach in the main doc set article going forward.
Wade and Stephen .....
NOTE: If you don't get a chance to review today (Wednesday), no rush for sure. A lot was written this morning, and I plan to perform another edit pass on Thursday morning.
UseAuthorizationcall requirement for Blazor Server apps, which was an oversight on the Roles PR. I'm adding it there, and it will appear in the Claims authz article.razor-pages/security/authorizationandmvc/security/authorizationnodes with cross-links.NOTE TO SELF 🦖
💀 the main doc set repo sample apps after review/approval. I'm not going to do it until then because it hoses the diff and makes the PR much harder to review.
Also, 💀 the prior BWA role-based auth sample app. This PR updates the Roles authz article to point to the new sample that will be used for both the role-based and claim-based authz coverage.
Internal previews