-
Notifications
You must be signed in to change notification settings - Fork 219
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
[Bug] unclear support for Net Framework, code excluded from compile #1577
Comments
@willportnoy : Microsoft.Identity.Web has limited support for .NET Framework as explained in: Minimal support for .NET FW Classic We have a goal of providing the full experience for .NET framework, but, to be honest, this is not the highest priority, given ASP.NET FW is in maintenance mode (wheareas a lot of innovation happens in ASP.NET for .NET Core / .NET 5.0, .NET 6.0, .NET 7.0. Where would you apply AddMicrosoftIdentityWebAppAuthentication in an ASP.NET application? On the app builder? |
@jmprieur I'm using ASP.NET Core with Net Framework. ASP.Net Core (web framework) is packaged as Net Standard (library format) and can be used in Net Framework (runtime environment). Moving our runtime environment from Net Framework to Net Core requires a substantial effort e.g. all library dependencies have to be available as netstandard or netcore. |
@willportnoy do you have an example of code that I could see? |
I followed up offline, to share source code of an ASP.NET Core web app running on Net Framework runtime. |
What was the resolution on this? I am running into this right now and would really like to be able to use |
Most of the entry points (e.g. AddMicrosoftIdentityWebAppAuthentication) seem to be excluded from the Net Framework compilation of this library:
https://github.com/AzureAD/microsoft-identity-web/blob/master/src/Microsoft.Identity.Web/Microsoft.Identity.Web.csproj#L112
Is this library usable within Net Framework?
It seems to be a NuGet target framework:
https://www.nuget.org/packages/Microsoft.Identity.Web/1.21.1
and issues imply "yes":
#1086
The text was updated successfully, but these errors were encountered: