-
Notifications
You must be signed in to change notification settings - Fork 76
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
Should compression be opt-in? #78
Comments
@lpirl thanks for raising this question @atomic111 opinion? |
@lpirl we can add the attribute, but the default value should be no. there was some vulnerabilities in the zlib compression. my approach is to reduce attack surface and only activate features that you relly need. i agree to the thread, that it would be really hard to exploit this flaw. My recommendation is to disable the compression stuff. |
@atomic111 right, I completely agree with your comment why it should be turned off – even if it is not a big thing. So you say including the attribute is not crucial since it is disabled per default anyway? I'd expect that explicitly disabling compression would suggest users/admins that it is generally a good idea to disable it since the hardening profile disabled it explicitly. |
@lpirl perfect. |
Looks there is no more discussion on this but just checking if option to disable compression added in future releases. |
Any PR to get this option in is welcome! |
According to this thread, compression can be vulnerable to CRIME/BREACH attacks (if the encrypted data carries public data as well).
I am not into crypto but I guess compression should be opt-in, at least, shouldn't it?
(This issue was migrated here from dev-sec/ansible-ssh-hardening#90)
The text was updated successfully, but these errors were encountered: