Skip to content

Freshness review: Overview of ASP.NET Core Authentication#36810

Open
Copilot wants to merge 2 commits intomainfrom
copilot/review-aspnet-core-article-again
Open

Freshness review: Overview of ASP.NET Core Authentication#36810
Copilot wants to merge 2 commits intomainfrom
copilot/review-aspnet-core-article-again

Conversation

Copy link
Contributor

Copilot AI commented Feb 26, 2026

Freshness review of aspnetcore/security/authentication/index.md addressing stale metadata, style inconsistencies, and prose/structure issues across moniker sections.

Metadata

  • Updated ms.date from 10/18/2022 to 02/26/2026
  • Added ai-usage: ai-assisted

>= aspnetcore-7.0 section

  • Removed extra blank line before ## Additional resources
  • Fixed double space after [Finbuckle.MultiTenant](…)

< aspnetcore-6.0 section

  • Removed ### Orchard Core / ### ABP Framework subsection headers; rewrote as prose to match the style of the other two monikers
  • Fixed https://orchardcore.net/https://www.orchardcore.net/ (consistent with other monikers)
  • Fixed asp.net core application frameworksASP.NET Core application frameworks

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • openid.net
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Contributes to #36100


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.


Internal previews

📄 File 🔗 Preview link
aspnetcore/security/authentication/index.md aspnetcore/security/authentication/index

Co-authored-by: meaghanlewis <10103121+meaghanlewis@users.noreply.github.com>
Copilot AI changed the title [WIP] Review ASP.NET Core article for freshness and accuracy Freshness review: Overview of ASP.NET Core Authentication Feb 26, 2026
@meaghanlewis meaghanlewis marked this pull request as ready for review February 26, 2026 20:04
### ABP Framework
[Orchard Core](https://www.orchardcore.net/) is an open-source, modular, and multi-tenant app framework built with ASP.NET Core that also provides a content management system (CMS). See the [Orchard Core](https://github.com/OrchardCMS/OrchardCore) source for an example of authentication providers per tenant.

[ABP Framework](https://abp.io/) supports various architectural patterns including modularity, microservices, domain driven design, and multi-tenancy. See [ABP Framework source on GitHub](https://github.com/abpframework/abp).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[ABP Framework](https://abp.io/) supports various architectural patterns including modularity, microservices, domain driven design, and multi-tenancy. See [ABP Framework source on GitHub](https://github.com/abpframework/abp).
[ABP Framework](https://abp.io/) supports various architectural patterns including modularity, microservices, domain-driven design, and multi-tenancy. See [ABP Framework source on GitHub](https://github.com/abpframework/abp).


## Authentication providers per tenant

ASP.NET Core doesn't have a built-in solution for multi-tenant authentication. While it's possible for customers to write one using the built-in features, we recommend customers to consider [Orchard Core](https://www.orchardcore.net/) or [ABP Framework](https://abp.io/) for multi-tenant authentication.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ASP.NET Core doesn't have a built-in solution for multi-tenant authentication. While it's possible for customers to write one using the built-in features, we recommend customers consider [Orchard Core](https://www.orchardcore.net/) or [ABP Framework](https://abp.io/) for multi-tenant authentication.

Copy link
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.

Looks great! Approved. I did make two super minor suggestions that you could take or leave.

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.

3 participants