Merge pull request #141 from Crown-Commercial-Service/make-503-status…

…-200

Make 503 status page return 200 status code so that Barracuda does not block it

app/controllers/errors_controller.rb

@@ -19,7 +19,7 @@ def internal_error
 
   def service_unavailable
     respond_to do |format|
-      format.html { render status: :service_unavailable, layout: 'error' }
+      format.html { render status: :ok, layout: 'error' }
     end
   end
 end

app/views/layouts/_footer.html.erb

@@ -1,44 +1,48 @@
 <%= render partial: '/layouts/contact_us' %>
 <%= ccs_footer(
   meta: {
-    items: [
-      {
-        text: t('.cookie_policy_link'),
-        href: cookie_policy_path,
-        attributes: {
-          aria: {
-            label: t('.cookie_policy_link_aria_label')
+    items: unless current_page?(service_unavailable_path)
+      [
+        {
+          text: t('.cookie_policy_link'),
+          href: cookie_policy_path,
+          attributes: {
+            aria: {
+              label: t('.cookie_policy_link_aria_label')
+            }
           }
-        }
-      },
-      {
-        text: t('.cookie_settings_link'),
-        href: cookie_settings_path,
-        attributes: {
-          aria: {
-            label: t('.cookie_settings_link_aria_label')
+        },
+        {
+          text: t('.cookie_settings_link'),
+          href: cookie_settings_path,
+          attributes: {
+            aria: {
+              label: t('.cookie_settings_link_aria_label')
+            }
           }
-        }
-      },
-      {
-        text: t('.privacy_link_text'),
-        href: t('.privacy_link'),
-        attributes: {
-          target: '_blank',
-          aria: {
-            label: t('.privacy_link_aria_label')
+        },
+        {
+          text: t('.privacy_link_text'),
+          href: t('.privacy_link'),
+          attributes: {
+            target: '_blank',
+            aria: {
+              label: t('.privacy_link_aria_label')
+            }
           }
-        }
-      },
-      {
-        text: t('.accessibility_statement_link'),
-        href: accessibility_statement_path,
-        attributes: {
-          aria: {
-            label: t('.accessibility_statement_link_aria_label')
+        },
+        {
+          text: t('.accessibility_statement_link'),
+          href: accessibility_statement_path,
+          attributes: {
+            aria: {
+              label: t('.accessibility_statement_link_aria_label')
+            }
           }
         }
-      }
-    ]
+      ]
+    else
+      []
+    end
   }
 ) %>

config/routes.rb

@@ -41,5 +41,5 @@
   get '/404', to: 'errors#not_found'
   get '/422', to: 'errors#unacceptable'
   get '/500', to: 'errors#internal_error'
-  get '/503', '/service-unavailable', to: 'errors#service_unavailable'
+  get '/service-unavailable', to: 'errors#service_unavailable'
 end