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

Installer of aspnetcore still contains wrong outdated .NET dotnet license information that makes it problematic to distribute #54434

Closed
1 task done
mgexo opened this issue Mar 8, 2024 · 14 comments
Assignees
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework investigate
Milestone

Comments

@mgexo
Copy link

mgexo commented Mar 8, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

There are wrong outdated license agreements and license files mentioned in the installer of aspnetcore, which developers must distribute as dependency e.g. for CoreWCF to clients in their installers.

This may cause software distributors of .NET Core - which is licensed under MIT - to get false warnings that they would violate some old Microsoft .NET Framework licenses which should not apply to .NET Core software components like aspnetcore (especially not the Desktop/Client part)

Is that intentional?

Example:
Go to https://dotnet.microsoft.com/en-us/download/dotnet/8.0
Download https://dotnet.microsoft.com/en-us/download/dotnet/thank-you/runtime-aspnetcore-8.0.2-windows-x64-installer
Run the installer
In the first screen, click on the "license term".

It links to https://go.microsoft.com/fwlink/?LinkId=329770, which then links to https://dotnet.microsoft.com/en-us/dotnet_library_license.htm which seems to be unrelated to .net Core and does NOT reference MIT but some old Microsoft license which makes it much more difficult to distribute.

The correct link should probably this one https://github.com/dotnet/core/blob/main/license-information-windows.md

This license text may even be contained in the installer.

In ticket 40835 a related issue has been reported for some other part.

Expected Behavior

No more reference to old Microsoft .NET framework licenses but correct dotnetcore MIT licenses.

Steps To Reproduce

Go to https://dotnet.microsoft.com/en-us/download/dotnet/8.0
Download https://dotnet.microsoft.com/en-us/download/dotnet/thank-you/runtime-aspnetcore-8.0.2-windows-x64-installer
Run the installer
In the first screen, click on the "license term".

It links to https://go.microsoft.com/fwlink/?LinkId=329770, which then links to https://dotnet.microsoft.com/en-us/dotnet_library_license.htm which seems to be unrelated to .net Core and does NOT reference MIT but some old Microsoft license which makes it much more difficult to distribute.

Exceptions (if any)

No response

.NET Version

No response

Anything else?

No response

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Mar 8, 2024
@wtgodbe wtgodbe added this to the 9.0-preview6 milestone Mar 12, 2024
@wtgodbe wtgodbe self-assigned this Mar 12, 2024
@wtgodbe
Copy link
Member

wtgodbe commented May 1, 2024

Hi, thanks for the report. It's expected that the hosting bundle uses the Microsoft license, as it redistributes some assets from dotnet/runtime & IIS that are not covered by the MIT license.

@wtgodbe wtgodbe closed this as not planned Won't fix, can't repro, duplicate, stale May 1, 2024
@mgexo
Copy link
Author

mgexo commented May 1, 2024

@wtgodbe but how can customers then redistribute the parts that are required for client-software only e.g. for the .NET Core WCF implementation? Do we need to create a reduced installer or can .NET CoreWCF not be shipped to clients through even though it is meant to be for client customers? We (and most likely all customers of CoreWCF ) need a redistributable installer for the respective parts

@wtgodbe
Copy link
Member

wtgodbe commented May 1, 2024

Generally the MIT license only applies to the code itself, not the compiled bits or the installers. If you need to redistribute code or bits from WCF I recommend filing an issue in https://github.com/dotnet/wcf

@richlander
Copy link
Member

We did a push on fixing up licensed a few years ago, mostly targeting on Linux. It looks like there is still some missing pieces.

Let's start with intent

@richlander richlander reopened this May 3, 2024
@mgexo
Copy link
Author

mgexo commented May 6, 2024

@wtgodbe can you maybe comment (or adjust your comments) based on the new information that had been provided by
@mconnew in dotnet/wcf#5522 ?

@richlander
Copy link
Member

Here's the intent:

  • Windows builds should use .NET Library license.
  • Linux and macOS builds should use MIT.

zip/tar.gz vs installer doesn't make a difference.

Windows builds can carry the .NET Library license and should ideally also point to https://github.com/dotnet/core/blob/main/license-information-windows.md.

@mgexo
Copy link
Author

mgexo commented Aug 2, 2024

@richlander @wtgodbe will the license fix make it into dotnet9 ?

@richlander
Copy link
Member

It seems like this fwlink is the source of it: https://go.microsoft.com/fwlink/?LinkId=329770

I don't have permission to see the link definition. If it is soley used by our installers, then we can change it.

Do you have access to the link @leecow?

@leecow
Copy link
Member

leecow commented Aug 2, 2024

Yes, I can access the link. Where do we want it to point?

@richlander
Copy link
Member

If the link is specific to our installers, it can point here:

https://github.com/dotnet/core/blob/main/license-information.md

I think that page needs an update to make it the Windows differences more clear. I can do that.

They are described here: https://github.com/dotnet/core/blob/main/license-information-windows.md. We could link just there but then users would miss the bigger picture.

@leecow
Copy link
Member

leecow commented Aug 2, 2024

If the link is specific to our installers ...

Looks like it is - https://github.com/search?q=org%3Adotnet%20https%3A%2F%2Fgo.microsoft.com%2Ffwlink%2F%3FLinkId%3D329770&type=code

@richlander
Copy link
Member

Feel free to make the link change.

In parallel, I made this PR. Hopefully we can merge it soon. I think the new text is clearer and provides some links to useful information.

dotnet/core#9440

@leecow
Copy link
Member

leecow commented Aug 2, 2024

Link updated

@richlander
Copy link
Member

Resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework investigate
Projects
None yet
Development

No branches or pull requests

4 participants