Skip to content

Light Freshness: ASP.NET - 6 Security/Auth topics#37148

Open
GitHubber17 wants to merge 11 commits into
dotnet:mainfrom
GitHubber17:568764-m
Open

Light Freshness: ASP.NET - 6 Security/Auth topics#37148
GitHubber17 wants to merge 11 commits into
dotnet:mainfrom
GitHubber17:568764-m

Conversation

@GitHubber17
Copy link
Copy Markdown
Contributor

@GitHubber17 GitHubber17 commented May 13, 2026

This PR contains light freshness updates to 6 articles:

  • Introduction to authorization in ASP.NET Core
  • Custom Authorization Policy Providers using IAuthorizationPolicyProvider in ASP.NET Core
  • Hash passwords in ASP.NET Core
  • Enable QR code generation for TOTP authenticator apps in ASP.NET Core (moniker range=">= aspnetcore-8.0")
  • Map, customize, and transform claims in ASP.NET Core (moniker range=">= aspnetcore-6.0")
  • Multifactor authentication in ASP.NET Core (moniker range=">= aspnetcore-9.0")
    -- Also updated 1 include
    -- Section Example: OpenID Connect Duende IdentityServer server
    There are several instances of the term "this" that need context for clarity:
    Displays if the Identity comes from an app that requires MFA, but the user hasn't activated this in Identity
    Informs the user and adds a link to activate this
    Because the client sent this with mfa set, this can then be checked

For details, see ADO request 568764.


Internal previews

📄 File 🔗 Preview link
aspnetcore/security/authentication/claims.md customer intent: As an ASP.NET developer, I want to work with claims in my ASP.NET Core app, so I can map claims, do transformations, and make customizations.
aspnetcore/security/authentication/identity-enable-qrcodes.md Enable QR code generation for TOTP authenticator apps in ASP.NET Core
aspnetcore/security/authentication/mfa.md Multifactor authentication in ASP.NET Core
aspnetcore/security/authorization/iauthorizationpolicyprovider.md customer intent: As an ASP.NET developer, I want to use a custom IAuthorizationPolicyProvider in my ASP.NET Core app, so I can dynamically generate authorization policies.
aspnetcore/security/authorization/introduction.md aspnetcore/security/authorization/introduction
aspnetcore/security/data-protection/consumer-apis/password-hashing.md customer intent: As an ASP.NET developer, I want to use the Data Protection APIs, so I can hash passwords in my ASP.NET Core apps.

@GitHubber17 GitHubber17 changed the title {DRAFT] Light Freshness: ASP.NET - 6 Security/Auth topics May 14, 2026
@GitHubber17 GitHubber17 marked this pull request as ready for review May 14, 2026 23:21
@GitHubber17
Copy link
Copy Markdown
Contributor Author

Hi @wadepickett - Refreshed topics ready for your review. See the top comment for details.

Notes:

Thanks

Comment thread aspnetcore/security/authentication/identity-enable-qrcodes.md Outdated
Comment thread aspnetcore/security/authentication/mfa.md Outdated
Comment thread aspnetcore/security/authentication/claims.md Outdated
Comment thread aspnetcore/security/authentication/mfa.md Outdated
[!code-csharp[](~/security/authentication/claims/sample8/WebRPmapClaims/Program.cs?name=snippet_NS8&highlight=20)]

## Extend or add custom claims using `IClaimsTransformation`
## Extend or add custom claims using 'IClaimsTransformation'
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.

Suggested change
## Extend or add custom claims using 'IClaimsTransformation'
## Extend or add custom claims using `IClaimsTransformation`

Code identifier and should have backticks per repo guidelines.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The editorial instructions say not to overload formatting in headers, i.e., don't apply both bold (from the header H2) and also inline code - which is why I swapped the code ticks for single quotes.

@@ -1,92 +1,93 @@
---
title: Enable QR code generation for TOTP authenticator apps in ASP.NET Core
title: Enable QR code generation for TOTP authentication
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.

Suggested change
title: Enable QR code generation for TOTP authentication
title: Enable QR code generation for TOTP authenticator apps in ASP.NET Core

I think it is best in this case to leave "ASP.NET Core" in there to differentiate from similar topics that are not using this framework.

Copy link
Copy Markdown
Contributor Author

@GitHubber17 GitHubber17 May 15, 2026

Choose a reason for hiding this comment

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

The metadata criteria indicates the title attribute length should be 50 characters or less, which is why I made the adjustments. The publishing system essentially limits chars to 50 in the title proper. However, if the titleSuffix attribute is present, the total allowed chars increases to 65. You could add "ASP.NET Core" in the metadata titleSuffix. The preview text in search results would render as "Enable QR code generation for TOTP authentication - ASP.NET Core" - Here's the criteria: Metadata required for content > title and Rules for writing meta titles. I don't usually automatically add an article-level titleSuffix because sometimes the repo defines the attribute at the folder level or repo-level in a build JSON (docfx.json or similar).

Copy link
Copy Markdown
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

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

Approved, assuming the few minor items I suggested are fixed.
Great work.

@tdykstra, leaving this unmerged for a while so you can have a chance to review as well.

Co-authored-by: Wade Pickett <wpickett@microsoft.com>
@GitHubber17
Copy link
Copy Markdown
Contributor Author

GitHubber17 commented May 15, 2026

@wadepickett - Thank you for your review. I applied most of your suggestions. For two edits, I left you return comments.

Also, I wanted to confirm that you saw my note to you in the top comment about clarity for instances of this in the text. You addressed one instance in your notes, but not all. Just confirming the other text is clear as-is.

@GitHubber17 GitHubber17 requested a review from wadepickett May 15, 2026 18:35
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.

2 participants