Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BWA with WinAuth article #34708

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

guardrex
Copy link
Collaborator

@guardrex guardrex commented Feb 12, 2025

Fixes #34707

Notes

❓***QUESTION:***❓ We want to show claims and how to set a policy for a Win SID. Is there anything else that we need this sample app to show, or is this enough to get folks going?

Thanks @markat1 for requesting this sample and coverage.


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/security/blazor-web-app-with-entra.md aspnetcore/blazor/security/blazor-web-app-with-entra
aspnetcore/blazor/security/blazor-web-app-with-oidc.md aspnetcore/blazor/security/blazor-web-app-with-oidc
aspnetcore/blazor/security/blazor-web-app-with-windows-authentication.md aspnetcore/blazor/security/blazor-web-app-with-windows-authentication
aspnetcore/toc.yml aspnetcore/toc

@guardrex guardrex self-assigned this Feb 12, 2025
Copy link
Contributor

@mikekistler mikekistler left a comment

Choose a reason for hiding this comment

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

This is a new area to me, so some of my comments may be just from lack of knowledge. Take them with a grain of salt.

@guardrex
Copy link
Collaborator Author

guardrex commented Feb 13, 2025

Made some updates on the last two commits ☝️ 👇 ...

  • I added the WinAuth ASP.NET Core article cross-link to the introductory remarks.
  • I decided to drop the sections from H3 to H2 because I want them to show up in the article's sidebar ToC.
  • Fleshed out the "fallback policy" language. I didn't need to do much with the "default policy" language because it already basically said that the policy is to require an authenticated user.

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.

Add BWA with Windows Auth article
2 participants