Skip to content

Conversation

igsilya
Copy link
Contributor

@igsilya igsilya commented Mar 18, 2024

HTTPS has much lower chances to be blocked in certain internal networks as well as more resilient to having issues with changing ssh keys on a GitHub server side. HTTPS is also the default method recommended by GitHub.

Which issue is resolved by this Pull Request:
I didn't open an issue for this, but the problem I have is that GitHub periodically changes SSH keys and automatic taxonomy clone fails due to a difference with a key in the known hosts file. And I know that there are companies who has git protocol blocked in their internal networks.

Description of your changes:
Changing all git/ssh clone paths to HTTPS.

HTTPS has much lower chances to be blocked in certain internal networks
as well as more resilient to having issues with changing ssh keys on
a GitHub server side.  HTTPS is also the default method recommended by
GitHub.

Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
@xukai92 xukai92 added the hold In-progress PR. Tag should be removed before merge. label Mar 19, 2024
@xukai92
Copy link
Member

xukai92 commented Mar 19, 2024

we plan to make these changes when the repos go public.
for now using https will require some old authentication like password

@xukai92
Copy link
Member

xukai92 commented Mar 19, 2024

i will close this for now. feel free to reopen later.

@xukai92 xukai92 closed this Mar 19, 2024
@igsilya
Copy link
Contributor Author

igsilya commented Mar 19, 2024

we plan to make these changes when the repos go public. for now using https will require some old authentication like password

@xukai92 HTTPS works just fine with authentication via access tokens, which isn't old. But I see how ssh keys can be slightly easier to manage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold In-progress PR. Tag should be removed before merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants