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

Fix npm issue to have git always use https #1

Merged
merged 1 commit into from
May 20, 2022
Merged

Conversation

redbugz
Copy link
Contributor

@redbugz redbugz commented May 20, 2022

Recent changes to npm and/or Github have caused npm 6 to take a very long time to resolve github dependencies, likely due to the git protocol being no longer supported by Github

npm/cli#4896
https://github.blog/2021-09-01-improving-git-protocol-security-github/
https://stackoverflow.com/questions/19317264/does-github-support-git-protocol-for-pull/69812472#69812472

This forces all git interactions to be over https instead of the git protocol

@redbugz redbugz requested review from joeycozza, a team and matthewmatician and removed request for a team May 20, 2022 13:48
@redbugz
Copy link
Contributor Author

redbugz commented May 20, 2022

Tested with frontierApp, normal build step 5 minutes, without this fix, aborted at 25 minutes, with this fix back to normal 5 minutes

@joeycozza joeycozza merged commit fef7f7f into master May 20, 2022
@joeycozza joeycozza deleted the git-https branch May 27, 2022 15:57
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.

2 participants