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

Updated to PFS only configs for Nginx & Apache2 #24

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

Conversation

MorganDavis
Copy link
Contributor

This is based upon recommendation from Qualys...which seems pretty reasonable.

@micahflee
Copy link

The Apache config isn't valid:

SSLCipherSuite SSLCipherSuite EECDH+ECDSA+AESGCM EECDH+aRSA+AESGCM EECDH+ECDSA+SHA384 EECDH+ECDSA+SHA256 EECDH+aRSA+SHA384 EECDH+aRSA+SHA256 EECDH+aRSA+RC4 EECDH EDH+aRSA RC4 !aNULL !eNULL !LOW !3DES !MD5 !EXP !PSK !SRP !DSS

It says SSLCipherSuite twice, and it's space-separated instead of colon-separated, and the ciphers are connected with pluses instead of dashes, so if you put this in an Apache config it fails saying: SSLCipherSuite takes one argument, Colon-delimited list of permitted SSL Ciphers (XXX:...:XXX' - see manual)`

Changing that line to this seems to work, but for some reason the Qualys test still says PFS is disabled:

SSLCipherSuite EECDH-ECDSA-AESGCM:EECDH-aRSA-AESGCM:EECDH-ECDSA-SHA384:EECDH-ECDSA-SHA256:EECDH-aRSA-SHA384:EECDH-aRSA-SHA256:EECDH-aRSA-RC4:EECDH:EDH-aRSA:RC4:!aNULL:!eNULL:!LOW:!3DES:!MD5:!EXP:!PSK:!SRP:!DSS

@ioerror
Copy link
Owner

ioerror commented Sep 25, 2013

If this is updated to take @micahflee's comment into account, I'll merge it.

@MorganDavis
Copy link
Contributor Author

It has not been updated yet, but I expect to in the next few days and will notify.

@Strubbl
Copy link

Strubbl commented Jan 8, 2014

@MorganDavis Next days are over?

Also RC4 should be disabled.

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.

4 participants