-
Notifications
You must be signed in to change notification settings - Fork 18.3k
Description
What version of Go are you using (go version
)?
$ go version 1.13.8
Version of x/crypto
: 1d94cc7ab1c630336ab82ccb9c9cda72a875c382
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env
)?
go env
Output
$ go env n/a
What did you do?
I tried to connect to an OpenSSH 8.2 server with the ssh-rsa
host key algorithm disabled on the server. I also tried to run an x/crypto/ssh
server and connect from an OpenSSH client with ssh-rsa
disabled. Lastly, I tried to sign a host certificate with ssh.Certificate.SignCert
with a SHA-2 based signature.
What did you expect to see?
I expected the RSA host key and certificate to validate successfully with the new SHA-2 based signatures introduced in RFC 8332. I also expected to be able to sign host certificates and have them automatically received a SHA-2 based signature.
OpenSSH has already deprecated ssh-rsa
(i.e. SHA-1 based) signatures in host certificates in version 8.2 because of safety reasons. They can still be used by the host key algorithm must be manually specified.
What did you see instead?
I was unable to connect either as a server or a client if ssh-rsa
wasn't enabled while using RSA host keys or host certificates. I was able to sign a certificate with the AlgorithmSigner
wrapper approach (i.e. by forcefully overriding Sign
) proposed by @stoggi in #36261, but it's not a great experience for users.
Activity
hansnielsen commentedon Feb 18, 2020
I spent some time hacking away at a solution for this and believe I have something largely ready to make a CL from: hansnielsen/golang-x-crypto@master...hans-rsa-sha2-support
The main change is that when given an
ssh-rsa
-type key or an RSAcrypto.Signer
(and not a customssh.Signer
), the code will automatically register thessh-rsa
,rsa-sha2-256
, andrsa-sha2-512
host key algorithms for use. This means that for users who haven't specified explicit algorithm preferences, they'll get the new SHA-2 based signatures just by updating. There shouldn't be any external-facing changes beyond algorithm support and the certificate signing choice mentioned below.The approach OpenSSH chose for these new signature types is somewhat interesting: they have the same key type (
ssh-rsa
) but different signature algorithms. This makes it slightly tricky to integrate while keeping the existing tests working. There's a little more explicit special-casing of the RSA SHA-2 signature family than I'd like.The one main choice I made in here (beyond just adding support for the new signature types) is that certificates now default to
rsa-sha2-512
instead ofssh-rsa
. I think given the already-deprecated nature ofssh-rsa
plus the reasonable threat model for host certificates, this is the correct choice. The "wrappedAlgorithmSigner
" approach can still be used to force an RSA SHA-1 signature and there's no need to encourage this.Feedback welcome! If the approach in the code above looks good, I'll work on submitting a CL.
josharian commentedon Feb 18, 2020
cc @hanwen @FiloSottile
FiloSottile commentedon Feb 18, 2020
Sounds good on both adding the algorithms and changing the certificate default.
OpenSSH 8.2 already stopped supporting
ssh-rsa
signed certificates, right?alex commentedon Feb 18, 2020
Only for host certificates I think, client certs still can use it.
hansnielsen commentedon Feb 18, 2020
I considered removing the SHA-1 RSA signature for host certs but wanted to keep this issue to algorithm addition. There are some choices in the client host key algorithm list that are probably worth revisiting as well and it’d probably be best to handle that as one issue.
FiloSottile commentedon Feb 18, 2020
How long has OpenSSH supported
rsa-sha2-512
certificates?hansnielsen commentedon Feb 18, 2020
OpenSSH 7.3 was the first version with
rsa-sha2-512
certificate support, released on 2016-08-01. Ubuntuxenial
stands out as being the largest distro that's still on 7.2 with backported security fixes.alex commentedon Feb 18, 2020
I always regret asking things like this, but surely support non-SHA1 signature algorithms is as deserving of a security backport as an actual CVE?
gopherbot commentedon Feb 19, 2020
Change https://golang.org/cl/220037 mentions this issue:
ssh: support RSA SHA-2 (RFC8332) signatures
rsc commentedon Feb 26, 2020
Based on the discussion above, this seems like a likely accept.
105 remaining items