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

[CI] Grab 'dotnet-install.sh' script directly from GitHub. #9699

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

jpobst
Copy link
Contributor

@jpobst jpobst commented Jan 21, 2025

Our CI builds are frequently failing to download the dotnet-install.sh script from the official .NET website due to an AuthenticationException.

Instead, grab the dotnet-install.sh script directly from GitHub to avoid the issue.

@jpobst jpobst marked this pull request as ready for review January 22, 2025 01:25
@jpobst jpobst requested a review from grendello as a code owner January 22, 2025 01:25
// AuthenticationException: The remote certificate is invalid because of errors in the certificate chain: RevocationStatusUnknown
// For now we'll grab it directly from GitHub
// public static readonly Uri DotNetInstallScript = new Uri ("https://dot.net/v1/dotnet-install.sh");
public static readonly Uri DotNetInstallScript = new Uri ("https://raw.githubusercontent.com/dotnet/install-scripts/refs/heads/main/src/dotnet-install.sh");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are no releases or tags, so main is probably best option for now:

@jonathanpeppers jonathanpeppers merged commit 108f7a3 into main Jan 22, 2025
55 of 58 checks passed
@jonathanpeppers jonathanpeppers deleted the dev/jpobst/install-dotnet-github branch January 22, 2025 16:06
grendello added a commit that referenced this pull request Jan 22, 2025
* main:
  [XABT] Create separate assembly preparation tasks. (#9637)
  Add `activity-alias` support (#9654)
  [CI] Grab 'dotnet-install.sh' script directly from GitHub. (#9699)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants