From 24a3724dfe9d6f06a2cea1d06c2eef55b501d2e7 Mon Sep 17 00:00:00 2001 From: Alan Orth Date: Wed, 23 Sep 2015 18:20:31 +0300 Subject: [PATCH] roles/nginx: Remove spdy_headers_comp It was deprecated when nginx added support for HTTP/2. Signed-off-by: Alan Orth --- roles/nginx/defaults/main.yml | 1 - roles/nginx/templates/https.j2 | 3 --- 2 files changed, 4 deletions(-) diff --git a/roles/nginx/defaults/main.yml b/roles/nginx/defaults/main.yml index fb17d79..029c09f 100644 --- a/roles/nginx/defaults/main.yml +++ b/roles/nginx/defaults/main.yml @@ -15,7 +15,6 @@ nginx_ssl_session_cache: shared:SSL:10m nginx_ssl_buffer_size: 1400 nginx_ssl_dhparam: /etc/ssl/certs/dhparam.pem nginx_ssl_protocols: 'TLSv1 TLSv1.1 TLSv1.2' -nginx_spdy_headers_comp: 6 # Enable HTTP Strict Transport Security? # True on production, False on development! diff --git a/roles/nginx/templates/https.j2 b/roles/nginx/templates/https.j2 index 87b75ec..a04889c 100644 --- a/roles/nginx/templates/https.j2 +++ b/roles/nginx/templates/https.j2 @@ -28,9 +28,6 @@ # of such infrastructure, consider turning off session tickets: ssl_session_tickets off; - # enable SPDY header compression - spdy_headers_comp {{ nginx_spdy_headers_comp }}; - {% if nginx_enable_hsts == True %} # Enable this if you want HSTS (recommended, but be careful) # Include all subdomains and indicate to Google that we want this pre-loaded in Chrome's HSTS store