Skip to content

Document AddIdentityCore as prerequisite for ConfigureApplicationCookie#36803

Merged
meaghanlewis merged 3 commits intomainfrom
copilot/update-cookie-settings-documentation
Feb 27, 2026
Merged

Document AddIdentityCore as prerequisite for ConfigureApplicationCookie#36803
meaghanlewis merged 3 commits intomainfrom
copilot/update-cookie-settings-documentation

Conversation

Copy link
Contributor

Copilot AI commented Feb 24, 2026

ConfigureApplicationCookie has an undocumented dependency on AddIdentityCore — it must be called after AddIdentityCore just as it must be after AddIdentity or AddDefaultIdentity. The docs only mentioned the latter two, leaving AddIdentityCore users without guidance.

Changes

  • Cookie settings section (both >= aspnetcore-6.0 and < aspnetcore-6.0 monikers): Updated the prerequisite note to include AddIdentityCore:

    ConfigureApplicationCookie must be called after calling AddIdentityCore, AddIdentity, or AddDefaultIdentity.

Original prompt

This section details on the original issue you should resolve

<issue_title>ConfigureApplicationCookie after AddIdentityCore</issue_title>
<issue_description>In the section cookie-settings you say:
"must be called after calling AddIdentity or AddDefaultIdentity."
We found that there is also a dependency to AddIdentityCore that has to be called before ConfigureApplicationCookie
If this is correct this could be changed to "must be called after calling AddIdentityCore an before AddIdentity or AddDefaultIdentity."
We found no reference of AddIdentityCore in the docs of AddIdentity or AddDefaultIdentity. Maybe the dependency should be document there as well.


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.


Internal previews

📄 File 🔗 Preview link
aspnetcore/security/authentication/identity-configuration.md Configure ASP.NET Core Identity

Co-authored-by: meaghanlewis <10103121+meaghanlewis@users.noreply.github.com>
Copilot AI changed the title [WIP] Update documentation for ConfigureApplicationCookie dependency Document AddIdentityCore as prerequisite for ConfigureApplicationCookie Feb 24, 2026
Copilot AI requested a review from meaghanlewis February 24, 2026 22:57
@meaghanlewis meaghanlewis marked this pull request as ready for review February 24, 2026 23:40
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.

Approved. Looks great. one minor item, a missing metadata for ai-usage, see commit suggestion.

Co-authored-by: Wade Pickett <wpickett@microsoft.com>
@meaghanlewis meaghanlewis merged commit 7870718 into main Feb 27, 2026
4 checks passed
@meaghanlewis meaghanlewis deleted the copilot/update-cookie-settings-documentation branch February 27, 2026 20:15
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.

ConfigureApplicationCookie after AddIdentityCore

3 participants