-
Notifications
You must be signed in to change notification settings - Fork 12
Migrate all WebApp projects to .NET 10 #88
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: fume <2969850+fume@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR migrates two ASP.NET Core web applications from .NET 8 to .NET 10 by updating the target framework and associated NuGet packages. The migration addresses both framework compatibility requirements and security vulnerabilities (CVE GHSA-m5vv-6r4h-3vj9 in Azure.Identity).
- Updated target framework from
net8.0tonet10.0in both projects - Upgraded ASP.NET Core packages from
8.0.xto10.0.0 - Updated Azure and Microsoft Identity packages with security fixes and compatibility improvements
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| WebApps/Proxy/Microsoft.SPID.Proxy/Microsoft.SPID.Proxy.csproj | Updated target framework to .NET 10 and upgraded 9 packages including Azure.Identity (security fix), ASP.NET Core, and System packages |
| WebApps/CNS.Auth.Web/CNS.Auth.Web.csproj | Updated target framework to .NET 10 and upgraded 4 packages including ASP.NET Core packages and Microsoft.IdentityModel.Tokens.Saml |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <PackageReference Include="Microsoft.AspNetCore.Authentication.Certificate" Version="10.0.0" /> | ||
| <PackageReference Include="Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation" Version="10.0.0" /> | ||
| <PackageReference Include="Microsoft.IdentityModel.Tokens.Saml" Version="8.3.1" /> | ||
| <PackageReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Design" Version="8.0.8" /> |
Copilot
AI
Nov 23, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Microsoft.VisualStudio.Web.CodeGeneration.Design package is still on version 8.0.8 while the project targets .NET 10. Although this package is a development-time tool and may work across versions, consider updating to version 10.0.0 if available for consistency with other package versions and to ensure full .NET 10 compatibility.
| <PackageReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Design" Version="8.0.8" /> | |
| <PackageReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Design" Version="10.0.0" /> |
Updates all C# projects from .NET 8 to .NET 10 target framework with compatible package versions.
Changes
Project Files
net8.0→net10.0net8.0→net10.0Package Updates
Microsoft.SPID.Proxy:
1.11.0→1.13.1(resolves CVE GHSA-m5vv-6r4h-3vj9)1.3.1→1.3.28.0.x→10.0.08.0.0→10.0.0CNS.Auth.Web:
8.0.x→10.0.06.14.0→8.3.1Notes
Deprecation warnings introduced (SYSLIB0057, ASPDEPR005, SYSLIB0028) are expected for .NET 10. These APIs remain functional but should be migrated to recommended alternatives in future work:
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.