Skip to content

Custom auth policies with IARD overhaul#36866

Merged
guardrex merged 6 commits intomainfrom
guardrex/custom-auth-policies-IAuthorizationRequirementData
Mar 16, 2026
Merged

Custom auth policies with IARD overhaul#36866
guardrex merged 6 commits intomainfrom
guardrex/custom-auth-policies-IAuthorizationRequirementData

Conversation

@guardrex
Copy link
Collaborator

@guardrex guardrex commented Mar 11, 2026

Fixes #36857

Tom ...

UPDATE: I changed the "iard" file name, and it's still complaining ... "with" and the API name. We can change it if you want.


Internal previews

📄 File 🔗 Preview link
aspnetcore/mvc/security/authorization/custom-authorization-policies-with-iauthorizationrequirementdata-in-mvc.md aspnetcore/mvc/security/authorization/custom-authorization-policies-with-iauthorizationrequirementdata-in-mvc
aspnetcore/security/authorization/custom-authorization-policies-with-iauthorizationrequirementdata.md aspnetcore/security/authorization/custom-authorization-policies-with-iauthorizationrequirementdata
aspnetcore/toc.yml aspnetcore/toc

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Refactors the IAuthorizationRequirementData documentation to emphasize Blazor scenarios, while adding a dedicated ASP.NET Core MVC companion article and exposing it in the MVC Authorization TOC.

Changes:

  • Updates security/authorization/iard.md to reference a Blazor Web App sample and adds a Minimal API-based demonstration flow.
  • Adds a new MVC-specific article at mvc/security/authorization/iard.md with JWT + curl testing steps.
  • Adds the new MVC article to aspnetcore/toc.yml under MVC → Security and Identity → Authorization.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
aspnetcore/toc.yml Adds the MVC IARD article under the MVC Authorization section.
aspnetcore/security/authorization/iard.md Repositions the IARD article toward Blazor-focused guidance and updates sample references and demo.
aspnetcore/mvc/security/authorization/iard.md Introduces an MVC-specific companion article and links back to the main IARD page.

@guardrex guardrex marked this pull request as ready for review March 11, 2026 22:02
@guardrex guardrex requested a review from tdykstra March 11, 2026 22:02
@guardrex
Copy link
Collaborator Author

@halter73 ... Could you take a look at #33353 and let me know if changes are required to react to his comments?

The samples for the two articles are at ...

Copy link
Contributor

@tdykstra tdykstra left a comment

Choose a reason for hiding this comment

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

LGTM

@guardrex
Copy link
Collaborator Author

guardrex commented Mar 16, 2026

Tom ... Great idea. I made 5001 (HTTPS) the port, and I also put 5000 (HTTP) on both samples for those who would like to test without having to deal with certificate config. Sample updates were made here ...

https://github.com/dotnet/AspNetCore.Docs.Samples/pull/293/changes

We're good to go here for the primary updates. I've messaged Stephen to ask about the other open issue for this article. I can't follow the reasoning of the ask on that issue, so he'll need to figure out if that's a valid concern and exactly what changes we'd need to make.

@guardrex guardrex merged commit aae76ec into main Mar 16, 2026
4 checks passed
@guardrex guardrex deleted the guardrex/custom-auth-policies-IAuthorizationRequirementData branch March 16, 2026 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor 'Custom authorization policies with IAuthorizationRequirementData' for Blazor focus

3 participants