-
Notifications
You must be signed in to change notification settings - Fork 588
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
Keep an eye on https://github.com/dotnet/core/issues/9671 #2848
Comments
PR: #2849 This line will fail some time in 2025. It will need to change to the new CDN. FAKE/src/legacy/FakeLib/DotNetCLIHelper.fs Line 567 in 048b97b
|
I think there might still be an issue here - the code at FAKE/src/app/Fake.DotNet.Cli/DotNet.fs Line 139 in 78a2562
https://raw.githubusercontent.com/dotnet/cli and from the linked issue I think it should be using https://dot.net/v1/dotnet-install.sh instead?
Observed when looking at the build results from the latest CI build where it got an error whilst trying to download the installer from the old azureedge URL |
Description
I'm not sure how much of an issue this is, but for the record -
Microsoft announced at dotnet/core#9671 that some .NET resources are moving to new domains due to a CDN they use falling over.
As it stands,
Fake.Runtime
usesMicrosoft.Deployment.DotNet.Releases
to download information about .NET product releases - refFAKE/src/app/Fake.Runtime/SdkAssemblyResolver.fs
Line 243 in 048b97b
Looking at
Microsoft.Deployment.DotNet.Releases
I see dotnet/deployment-tools#419 was made to change the default domain used to download the product release list.So - I wonder whether Fake.Runtime will need to be changed to either use the new URL, or be updated to a new
Microsoft.Deployment.DotNet.Releases
if one should be releasedThe text was updated successfully, but these errors were encountered: