Roll Blazor into the Simple Auth article#36650
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors the "Simple authorization" article (aspnetcore/security/authorization/simple.md) to center on Blazor Razor component examples, per the guidance from issue #35816 to consolidate authorization content under corresponding technology areas. Two new companion articles are created for MVC and Razor Pages, the old .NET Core 3.1 Razor Pages sample app is deleted, and cross-reference links throughout the repo are updated.
Changes:
- Rewrites the main
simple.mdarticle to focus on Blazor, with new MVC (mvc/security/authorization/simple.md) and Razor Pages (razor-pages/security/authorization/simple.md) articles for technology-specific guidance. - Adds "Security and Identity > Authorization" TOC nodes under both the Razor Pages and MVC sections.
- Replaces all
[!INCLUDE[](~/includes/requireAuth.md)]usages with inline<xref:>links and deletes the now-unused include file and the 3.1 sample app.
Reviewed changes
Copilot reviewed 38 out of 47 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
aspnetcore/security/authorization/simple.md |
Rewritten to use Blazor examples; links to new MVC/RP articles |
aspnetcore/mvc/security/authorization/simple.md |
New MVC-specific simple authorization article |
aspnetcore/razor-pages/security/authorization/simple.md |
New Razor Pages-specific article with handler authorization guidance |
aspnetcore/toc.yml |
Adds Security and Identity > Authorization nodes under Razor Pages and MVC |
aspnetcore/security/authorization/secure-data.md |
Updates #rau anchor links to #require-authenticated-users |
aspnetcore/security/authorization/policies.md |
Replaces requireAuth.md include with inline xref |
aspnetcore/security/authorization/policies/includes/policies5.md |
Replaces requireAuth.md include with inline xref |
aspnetcore/security/authentication/index.md |
Updates #rau anchor links to #require-authenticated-users |
aspnetcore/security/authentication/identity.md |
Replaces requireAuth.md include with inline xref |
aspnetcore/security/authentication/identity/includes/identity6.md |
Replaces requireAuth.md include with inline xref |
aspnetcore/security/authentication/identity-configuration.md |
Replaces requireAuth.md include with inline xref |
aspnetcore/razor-pages/index.md |
Updates link text/anchor for Razor Pages auth section |
aspnetcore/razor-pages/index/includes/index35.md |
Updates link text/anchor for Razor Pages auth section |
aspnetcore/includes/requireAuth.md |
Deleted (no longer used) |
aspnetcore/security/authorization/simple/samples/3.1/PageHandlerAuth/* |
Deleted old sample app (many files) |
Contributor
tdykstra
approved these changes
Mar 5, 2026
Contributor
tdykstra
left a comment
There was a problem hiding this comment.
A few suggestions, nothing major
Co-authored-by: Tom Dykstra <tdykstra@microsoft.com>
…-circuited on auth failure (#36839) * Initial plan * Fix AuthorizeIndexPageHandlerFilter to short-circuit pipeline on failed authorization Co-authored-by: guardrex <1622880+guardrex@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: guardrex <1622880+guardrex@users.noreply.github.com>
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 #35816
cc: @tdykstra (article metadata author) and @wadepickett for discussion
Dan stated (and confirmed offline) ...
Internal previews