Web.config migration should not include ASPNET-specific settings

dotnet / upgrade-assistant

A tool to assist developers in upgrading .NET Framework applications to .NET 6 and beyond

MIT License

1.11k stars 162 forks source link

Web.config migration should not include ASPNET-specific settings #298

Open mtaylorfsmb opened 3 years ago

mtaylorfsmb commented 3 years ago

ASP.NET MVC projects have a set of ASPNET-specific appsettings. These shouldn't be needed in ASP.NET Core and shouldn't be automatically migrated to the appsettings.json file. Without looking too deep I see the following appsettings that should be ignored when migrating the app settings.

aspnet:*
webpages:*
ClientValidationEnabled
UnobtrusiveJavaScriptEnabled

There may be more.

KSchlobohm commented 3 years ago

Also applies to the nested web.config in the Views folder

oteinone commented 3 years ago

I also found these which have likely been legacy already for MVC5

    <add key="ChartImageHandler" value="storage=memory;deleteAfterServicing=true" />
    <add key="PreserveLoginUrl" value="true" />