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

Alternative to forking golang.org/x/crypto #313

Open
jpillor-macquarie opened this issue Jan 31, 2024 · 2 comments
Open

Alternative to forking golang.org/x/crypto #313

jpillor-macquarie opened this issue Jan 31, 2024 · 2 comments

Comments

@jpillor-macquarie
Copy link

Thanks for the great work with sshpiper :)

Sorry I posted here because https://github.com/tg123/sshpiper.crypto has issues disabled. This is just a suggestion, feel free to close. I understand that doing this essentially abandons efforts to get this merged upstream to golang.org/x/crypto.

Currently https://github.com/tg123/sshpiper.crypto forks golang.org/x/crypto. This means that we have to do a mod replace for all of golang.org/x/crypto and you potentially miss critical security updates.

As an alternative, sshpiper.crypto could instead be a go module with one package: ssh, which itself imports golang.org/x/crypto

Then users of sshpiper.crypto only import the ssh package; for everything else, they stick to golang.org/x/crypto.

I have done this to avoid the mod replace, I wrote myself a list to update sshpiper.crypto

  • Clone https://github.com/tg123/sshpiper.crypto into tmp
  • Copy tmp/ssh to ./ssh
  • Copy tmp/internal/poly1305 to ./ssh/internal
  • Copy tmp/ssh/internal/bcrypt_pbkdf to ./ssh/internal
  • Alias PublicKey and Signature to x/crypto/ssh to maintain type compatibility
@tg123
Copy link
Owner

tg123 commented Jan 31, 2024

what i have to is watch upstream and update timely

i did not get how your solution works, could you please send a pr?

@hexiaodai
Copy link

We had the same problem.

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

No branches or pull requests

3 participants