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

Update .gitmodules to use https urls #192

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dark64
Copy link

@dark64 dark64 commented Jan 12, 2022

Recent changes in the Git protocol security on GitHub https://github.blog/2021-09-01-improving-git-protocol-security-github/ broke our build script that clones libsnark. I found that some submodules referenced in .gitmodules use the unencrypted git protocol and some use https, which produces an error The unauthenticated git protocol on port 9418 is no longer supported if the repository is cloned using the https protocol.

@AntoineRondelet
Copy link
Collaborator

AntoineRondelet commented Jan 12, 2022

Ah yes good catch @dark64 thanks for this. I actually updated it on clearmatics/libsnark#37 but forgot to propagate here...

@AntoineRondelet AntoineRondelet self-requested a review January 12, 2022 12:09
Copy link
Collaborator

@AntoineRondelet AntoineRondelet left a comment

Choose a reason for hiding this comment

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

LGTM

@AntoineRondelet
Copy link
Collaborator

@ValarDragon is there any plan to release a new version of libsnark at some point in the near future? If so, I think this PR should probably target staging for now and will hit master on the new release.

@mkumar-cognam
Copy link

we can also use git to change protocol to https using below command

git config --global url."https://".insteadOf git://

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