Skip to content
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

Closed
willportnoy opened this issue Jan 5, 2022 · 5 comments
Closed
Labels
answered question Further information is requested relv2

Comments

@willportnoy
Copy link

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

@jmprieur
Copy link
Collaborator

jmprieur commented Jan 5, 2022

@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 jmprieur added question Further information is requested answered labels Jan 5, 2022
@willportnoy
Copy link
Author

@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.

@jmprieur
Copy link
Collaborator

jmprieur commented Jan 5, 2022

@willportnoy do you have an example of code that I could see?
It's pretty old ... and I don't think dotnet new, or Visual Studio create these projects any longer.

@willportnoy
Copy link
Author

I followed up offline, to share source code of an ASP.NET Core web app running on Net Framework runtime.

@celluj34
Copy link

celluj34 commented Sep 8, 2022

What was the resolution on this? I am running into this right now and would really like to be able to use AddMicrosoftIdentityWebApi.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
answered question Further information is requested relv2
Projects
None yet
Development

No branches or pull requests

3 participants