Skip to content

Commit

Permalink
Update sshd config based on feedback
Browse files Browse the repository at this point in the history
- As pointed out by @kushaldas, chacha20-poly1305 mostly mobile-specific cipher, and while historically present in the sshd configuration for SecureDrop, is not necessary to support Debian-based ssh clients.

- The UsePrivilegeSeparation option has been deprecated in OpenSSH 7.5 [1]. UsePrivilegeSeparation has defaulted to 'sandbox' since 6.1 [2] and to 'yes' since 3.3 [3].

[1] https://www.openssh.com/txt/release-7.5
[2] https://www.openssh.com/txt/release-6.1
[3] https://www.openssh.com/txt/release-3.3
  • Loading branch information
emkll committed Jan 7, 2021
1 parent 54abdb2 commit 3f4f6ac
Showing 1 changed file with 1 addition and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@ HostKey /etc/ssh/ssh_host_rsa_key
HostKey /etc/ssh/ssh_host_ecdsa_key
HostKey /etc/ssh/ssh_host_ed25519_key

# Enforce privilege separation by creating unprivileged child process
UsePrivilegeSeparation yes
KeyRegenerationInterval 3600
ServerKeyBits 4096

Expand Down Expand Up @@ -39,7 +37,7 @@ UseDNS no

# Cipher selection

Ciphers [email protected],[email protected],[email protected],aes256-ctr,aes128-ctr
Ciphers [email protected],[email protected],aes256-ctr,aes128-ctr
# Don't use SHA1 for kex
KexAlgorithms [email protected],diffie-hellman-group-exchange-sha256
# Don't use SHA1 for hashing, don't use encrypt-and-MAC mode
Expand Down

0 comments on commit 3f4f6ac

Please sign in to comment.