Policy-based authorization article overhaul#37231
Open
guardrex wants to merge 12 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR modernizes and restructures the policy-based authorization documentation by shifting MVC and Razor Pages–specific guidance into their own new articles and refocusing the main article on Blazor-oriented examples, while also updating related cross-references and Entra ID links across the docs set.
Changes:
- Overhauls
security/authorization/policies.mdto use Razor component examples and updated samples (including a new external-service sample section). - Adds new companion articles for MVC and Razor Pages policy-based authorization and wires them into
aspnetcore/toc.yml. - Updates multiple related docs to point to the new anchors/pages and refreshes Microsoft Entra ID links +
ms.datemetadata.
Reviewed changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| aspnetcore/toc.yml | Adds TOC entries for the new MVC and Razor Pages policy-based authorization articles. |
| aspnetcore/security/how-to-choose-identity-solution.md | Updates Entra link + ms.date metadata. |
| aspnetcore/security/authorization/resource-based.md | Updates xref fragments and sample namespace/path text to match the new policies doc structure. |
| aspnetcore/security/authorization/policies/samples/stubs/copy_of_IAuthorizationService.cs | Removes copied framework source stub previously used for an include snippet. |
| aspnetcore/security/authorization/policies/includes/policies5.md | Removes pre-6.0 include content that was previously injected into the policies article. |
| aspnetcore/security/authorization/policies.md | Major rewrite: policy concepts + Blazor-focused examples + new external-service sample walkthrough. |
| aspnetcore/security/authorization/iauthorizationpolicyprovider.md | Updates xref fragments + ms.date. |
| aspnetcore/security/authorization/dependencyinjection.md | Updates xref fragments + ms.date. |
| aspnetcore/security/authorization/customizingauthorizationmiddlewareresponse.md | Updates ms.date. |
| aspnetcore/security/authorization/custom-authorization-policies-with-iauthorizationrequirementdata.md | Replaces hard-coded user values with placeholders in decoded JWT example. |
| aspnetcore/release-notes/aspnetcore-5.0.md | Updates Entra link + ms.date. |
| aspnetcore/razor-pages/security/authorization/resource-based.md | Updates xref fragment to new policies doc anchor. |
| aspnetcore/razor-pages/security/authorization/policies.md | New Razor Pages–specific policy-based authorization article. |
| aspnetcore/mvc/security/authorization/resource-based.md | Updates xref fragment to new policies doc anchor. |
| aspnetcore/mvc/security/authorization/policies.md | New MVC-specific policy-based authorization article (including MVC handler Resource guidance). |
| aspnetcore/migration/70-to-80.md | Updates xref to point to the new MVC policies article section. |
| aspnetcore/migration/22-to-30.md | Updates xrefs to the new policies anchor/article for handler resource guidance. |
13 tasks
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…et/AspNetCore.Docs into guardrex/policy-based-authz
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #35817
Tom ... I worked quite long time on this one, longer than usual. The current article (live link) seems like it has a lot of problems to address.
Notes
Internal previews
Toggle expand/collapse