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

Disable SSLHonorCipherOrder #247

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mueller-ma
Copy link

Copy link

This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

@github-actions github-actions bot added the stale label May 20, 2024
@mueller-ma
Copy link
Author

The PR is still required to match the recommendations by Mozilla.

@github-actions github-actions bot removed the stale label May 27, 2024
@ravik694
Copy link

ravik694 commented Jul 2, 2024

It's recommended by https://ssl-config.mozilla.org/#server=apache&version=2.4.41&config=modern&openssl=1.1.1k&guideline=5.7 to disable SSLHonorCipherOrder.

According to https://wiki.mozilla.org/Security/Server_Side_TLS#Modern_compatibility:

Cipher preference: client chooses
[...]

  • Rationale:
    • All cipher suites are forward secret and authenticated
    • The cipher suites are all strong and so we allow the client to choose, as they will know best if they have support for hardware-accelerated AES

The Modern config is only meant for TLS versions/ciphers where all ciphers are strong/support forward secrecy. Setting a hardcoded SSLHonorCipherOrder Off in this role could actually be less secure if the TLS versions/ciphers chosen are not all strong. The default config for the role does not enforce TLS versions/ciphers that are all strong.

@mueller-ma
Copy link
Author

I can make it configurable, but IMO it should be "secure by default". The TLS versions and ciphers in https://github.com/geerlingguy/ansible-role-apache/blob/master/defaults/main.yml should be updated as well, but in a different PR.

Copy link

github-actions bot commented Nov 4, 2024

This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

@github-actions github-actions bot added the stale label Nov 4, 2024
@mueller-ma
Copy link
Author

The PR is still required to match the recommendations by Mozilla.

@github-actions github-actions bot removed the stale label Nov 11, 2024
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

Successfully merging this pull request may close these issues.

2 participants