Add missing SSL attributes to nginx config

rackspace-cookbooks / elkstack

Elasticsearch, logstash, and kibana stack

Other

81 stars 54 forks source link

Closed neilh23 closed 8 years ago

neilh23 commented 8 years ago

Cipher and protocols where defined in attributed, but never passed through to the nginx configuration!

Also, deprecated TLSv1.0 by default

martinb3 commented 8 years ago

Hi there -- tests are failing, likely because of:

default['nginx']['disable_http'] = true

Would you mind updating the tests to match your changes? Thanks!

neilh23 commented 8 years ago

Good point. Have set the default to false for now, seems reasonable to allow port 80 by default

martinb3 commented 8 years ago

Hi there -- the tests on this still aren't passing. Would you mind reviewing those?