Merge pull request #331 from oskarpearson/ie-compatibility-headers

Set X-UA-Compatible HTTP header
alphagov · Jul 20, 2015 · e2ba8c6 · e2ba8c6
2 parents 534c21e + 0c606ea
commit e2ba8c6
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/config/application.rb b/config/application.rb
@@ -42,5 +42,7 @@ class Application < Rails::Application
     config.action_dispatch.rescue_responses.merge!(
       'Site::ServiceUnavailable' => :service_unavailable
     )
+
+    config.action_dispatch.default_headers.merge!('X-UA-Compatible' => 'IE=edge')
   end
 end
diff --git a/spec/controllers/application_controller_spec.rb b/spec/controllers/application_controller_spec.rb
@@ -10,6 +10,7 @@ def index
   end
 
   let(:cache_control) { response.headers['Cache-Control'] }
+  let(:x_ua_compatible) { response.headers['X-UA-Compatible'] }
 
   it "reloads the site instance on every request" do
     expect(Site).to receive(:reload)
@@ -21,6 +22,11 @@ def index
     expect(cache_control).to eq('no-store, no-cache')
   end
 
+  it "sets X-UA-Compatible control headers" do
+    get :index
+    expect(x_ua_compatible).to eq('IE=edge')
+  end
+
   context "when the site is disabled" do
     before do
       expect(Site).to receive(:enabled?).and_return(false)