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

Update Apache configuration #145

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

th-certbund
Copy link

No description provided.

@ignisf
Copy link

ignisf commented Jul 17, 2018

Hello,

What is the rationale of this change? Why deviate from the cipher string recommended by the authors and used throughout this guide (see section 3.2.3. Recommended cipher suites)?

@krono
Copy link
Contributor

krono commented Jul 17, 2018

What this does is essentially

  1. Disable session tickets
  2. Change Cipher
  3. add comments related to a mozilla guide.

I think 1. is interesting, and there should be some general discussion in the text about session tickets (and a note for apache in particular.

For 2. this is not a good idea to just change it here in the config file. Rather, for this please engage in a disucssion on the ACH mailing list.

For 3. we maybe need a general disclaimer that, while we understand the merits of the Mozilla cipher generator and such, this is not the place to just dump the results of it…

=-=-=-=-=

@th-certbund You indicated that this was tested with a recent apache on Debian 9.

  • Does it also work with an unmodified config file?
  • Is the session syntax new to that version?

@th-certbund
Copy link
Author

@ignisf:
The old cipher list includes "+SSLv3" while it should be disabled.
CAMELLIA is not supported by web browsers like Firefox.
The list should start with strong/fast ciphers widely supported by web browsers today (like CHACHA20-POLY1305) as highest priority if SSLHonorCipherOrder is used.
etc.

@krono
Copy link
Contributor

krono commented Jul 17, 2018

@th-certbund That is true per se. Have you looked at Section 3.2.3 "Recommended cipher suites" in the guide? It explains the two cipher combos recommended and why people might want the more compatible "Ciphersuite B".

Note that all examples in the repository use the B variant. It is intended that people opting for the A variant (namely EDH+aRSA+AES256:EECDH+aRSA+AES256:!SSLv3) replace that themselves. Maybe it would be a good Idea to make two toplevel configs with one using the A and one using the B variant. but this is for a new issue i think.

@aaronkaplan
Copy link
Member

aaronkaplan commented Jul 17, 2018 via email

@krono
Copy link
Contributor

krono commented Jul 17, 2018

@aaronkaplan ok, great. This probably means fixing the suites, right?

@krono
Copy link
Contributor

krono commented Jul 17, 2018

The problem might be that we get a myriad of different suites amongst all the different config files, at which point they don't serve any good purpose anymore :/

@aaronkaplan
Copy link
Member

aaronkaplan commented Jul 17, 2018 via email

@martinberg212
Copy link

Why is this PR not being merged?
It has been a year since the last comment was made and the current apache config is still very much dated.

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.

5 participants