Skip to content

Revise IAuthorizationRequirementData article details - #37347

Merged
guardrex merged 6 commits into
mainfrom
guardrex-patch-4
Jul 29, 2026
Merged

Revise IAuthorizationRequirementData article details#37347
guardrex merged 6 commits into
mainfrom
guardrex-patch-4

Conversation

@guardrex

@guardrex guardrex commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

Fixes #37340

The original set of changes included remarks on support for IAuthorizationRequirementData, but we're moving that over to Ilona's PR at #37393.

What this PR is going to do now is just call out that the BWA demonstrates use with a Minimal API endpoint ...

... using the same AuthorizationHandler that I found in the original MVC sample.


Internal previews

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

Updated the article to clarify that the IAuthorizationRequirementData interface is supported only in Minimal APIs. Adjusted the sample app description to specify the endpoint and the age requirement.
@guardrex
guardrex requested a review from halter73 July 17, 2026 10:57
@guardrex guardrex self-assigned this Jul 17, 2026
guardrex added 3 commits July 17, 2026 09:10
Clarified that IAuthorizationRequirementData isn't supported in Razor components.
Clarified that IAuthorizationRequirementData is not supported for certain authorization attributes, including Razor components and SignalR hub classes.

@wadepickett wadepickett left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@guardrex, looks great! Approved.

@guardrex

Copy link
Copy Markdown
Collaborator Author

Thx @wadepickett!

@halter73 ... Do u want to review?

@guardrex

Copy link
Copy Markdown
Collaborator Author

@halter73 ... Do you want to review this?

guardrex added 2 commits July 29, 2026 11:27
Clarified the context of the article and removed unsupported authorization attributes list.
Update the date for the document on custom authorization policies.
@guardrex

Copy link
Copy Markdown
Collaborator Author

@wadepickett ... May I have a re-approval on the reduced set of updates? The rest of this has been moved over to Ilona's PR, so this just makes a minor update WRT the BWA sample app's use of a Minimal API (and mentioning the endpoint in passing).

@wadepickett wadepickett left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@guardrex: Looks good, re-approved.

@guardrex
guardrex merged commit 9c02a6b into main Jul 29, 2026
4 checks passed
@guardrex
guardrex deleted the guardrex-patch-4 branch July 29, 2026 19:41
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.

[Blazor][Signalr] Add a note about IAuthorizationRequirementData only being supported in Minimal APIs

2 participants