Commit 59422e16 by Kevin Falcone

Merge pull request #2956 from edx/jibsheet/nginx-p3p

Add P3P headers to the IDAs
parents 0f6510f0 9d8e2ee6
...@@ -63,6 +63,9 @@ server { ...@@ -63,6 +63,9 @@ server {
proxy_redirect off; proxy_redirect off;
proxy_pass http://{{ role_name }}_app_server; proxy_pass http://{{ role_name }}_app_server;
} }
# Prevent invalid display courseware in IE 10+ with high privacy settings
add_header P3P {{ '{{' }} NGINX_P3P_MESSAGE {{ '}}' }}
# Nginx does not support nested condition or or conditions so # Nginx does not support nested condition or or conditions so
# there is an unfortunate mix of conditonals here. # there is an unfortunate mix of conditonals here.
......
...@@ -127,3 +127,4 @@ nginx_cfg: ...@@ -127,3 +127,4 @@ nginx_cfg:
NGINX_ROBOT_RULES: [ ] NGINX_ROBOT_RULES: [ ]
NGINX_EDXAPP_EMBARGO_CIDRS: [] NGINX_EDXAPP_EMBARGO_CIDRS: []
NGINX_P3P_MESSAGE: 'CP="Open edX does not have a P3P policy."'
...@@ -43,7 +43,7 @@ error_page {{ k }} {{ v }}; ...@@ -43,7 +43,7 @@ error_page {{ k }} {{ v }};
{% endif %} {% endif %}
# Prevent invalid display courseware in IE 10+ with high privacy settings # Prevent invalid display courseware in IE 10+ with high privacy settings
add_header P3P 'CP="Open EdX does not have a P3P policy."'; add_header P3P '{{ NGINX_P3P_MESSAGE }}';
# Nginx does not support nested condition or or conditions so # Nginx does not support nested condition or or conditions so
# there is an unfortunate mix of conditonals here. # there is an unfortunate mix of conditonals here.
......
...@@ -32,6 +32,9 @@ server { ...@@ -32,6 +32,9 @@ server {
listen {{ CREDENTIALS_NGINX_PORT }} {{ default_site }}; listen {{ CREDENTIALS_NGINX_PORT }} {{ default_site }};
{% endif %} {% endif %}
# Prevent invalid display courseware in IE 10+ with high privacy settings
add_header P3P '{{ NGINX_P3P_MESSAGE }}';
location ~ ^/static/(?P<file>.*) { location ~ ^/static/(?P<file>.*) {
root {{ COMMON_DATA_DIR }}/{{ credentials_service_name }}; root {{ COMMON_DATA_DIR }}/{{ credentials_service_name }};
try_files /staticfiles/$file =404; try_files /staticfiles/$file =404;
......
...@@ -30,6 +30,9 @@ server { ...@@ -30,6 +30,9 @@ server {
add_header Strict-Transport-Security "max-age=31536000; includeSubDomains"; add_header Strict-Transport-Security "max-age=31536000; includeSubDomains";
{% endif %} {% endif %}
# Prevent invalid display courseware in IE 10+ with high privacy settings
add_header P3P '{{ NGINX_P3P_MESSAGE }}';
# Nginx does not support nested condition or or conditions so # Nginx does not support nested condition or or conditions so
# there is an unfortunate mix of conditonals here. # there is an unfortunate mix of conditonals here.
{% if NGINX_REDIRECT_TO_HTTPS %} {% if NGINX_REDIRECT_TO_HTTPS %}
......
...@@ -7,6 +7,9 @@ upstream {{ edx_notes_api_service_name }}_app_server { ...@@ -7,6 +7,9 @@ upstream {{ edx_notes_api_service_name }}_app_server {
server { server {
listen {{ edx_notes_api_nginx_port }} default_server; listen {{ edx_notes_api_nginx_port }} default_server;
# Prevent invalid display courseware in IE 10+ with high privacy settings
add_header P3P '{{ NGINX_P3P_MESSAGE }}';
# Nginx does not support nested condition or or conditions so # Nginx does not support nested condition or or conditions so
# there is an unfortunate mix of conditonals here. # there is an unfortunate mix of conditonals here.
{% if NGINX_REDIRECT_TO_HTTPS %} {% if NGINX_REDIRECT_TO_HTTPS %}
......
...@@ -32,6 +32,9 @@ location @proxy_to_app { ...@@ -32,6 +32,9 @@ location @proxy_to_app {
proxy_redirect off; proxy_redirect off;
proxy_pass http://insights_app_server; proxy_pass http://insights_app_server;
} }
# Prevent invalid display courseware in IE 10+ with high privacy settings
add_header P3P '{{ NGINX_P3P_MESSAGE }}';
# Nginx does not support nested condition or or conditions so # Nginx does not support nested condition or or conditions so
# there is an unfortunate mix of conditonals here. # there is an unfortunate mix of conditonals here.
......
...@@ -63,7 +63,7 @@ error_page {{ k }} {{ v }}; ...@@ -63,7 +63,7 @@ error_page {{ k }} {{ v }};
{% endif %} {% endif %}
# Prevent invalid display courseware in IE 10+ with high privacy settings # Prevent invalid display courseware in IE 10+ with high privacy settings
add_header P3P 'CP="Open EdX does not have a P3P policy."'; add_header P3P '{{ NGINX_P3P_MESSAGE }}';
# Nginx does not support nested condition or or conditions so # Nginx does not support nested condition or or conditions so
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment