Skip to content

Remove IdentityModel source-build workaround - #55557

Open
mthalman wants to merge 1 commit into
release/9.0.1xxfrom
mthalman-remove-identitymodel-workaround
Open

Remove IdentityModel source-build workaround#55557
mthalman wants to merge 1 commit into
release/9.0.1xxfrom
mthalman-remove-identitymodel-workaround

Conversation

@mthalman

Copy link
Copy Markdown
Member

The aspnetcore fix that made the IdentityModel version override unnecessary has flowed into this branch. Remove the temporary source-only IdentityModelVersion pin so aspnetcore uses its repository-defined version again.

Fixes dotnet/source-build#5617

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: d769c502-f3c6-4f02-9e0d-db729faebcb4
Copilot AI review requested due to automatic review settings July 31, 2026 18:29
@mthalman
mthalman requested review from a team as code owners July 31, 2026 18:29
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@mthalman

Copy link
Copy Markdown
Member Author

/azp run sdk-source-build

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

Copilot AI 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.

Pull request overview

Removes a temporary source-build-only IdentityModelVersion MSBuild property override for the aspnetcore source-build repository project now that the upstream aspnetcore fix has flowed into this branch.

Changes:

  • Drop /p:IdentityModelVersion=8.19.2 from the source-only BuildArgs passed to aspnetcore builds.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants