From: Tom Hughes <tom@compton.nu>
Date: Sun, 14 Dec 2014 00:28:26 +0000 (+0000)
Subject: Don't render the offline page for XHR requests
X-Git-Tag: live~4861
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/4e6fe811a03e77d2bba22c7666716ae6a6bd8f04?ds=sidebyside

Don't render the offline page for XHR requests
---

diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index 90a967276..bd4a62948 100644
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -192,14 +192,22 @@ class ApplicationController < ActionController::Base
 
   def check_database_readable(need_api = false)
     if STATUS == :database_offline or (need_api and STATUS == :api_offline)
-      redirect_to :controller => 'site', :action => 'offline'
+      if request.xhr?
+        report_error "Database offline for maintenance", :service_unavailable
+      else
+        redirect_to :controller => 'site', :action => 'offline'
+      end
     end
   end
 
   def check_database_writable(need_api = false)
     if STATUS == :database_offline or STATUS == :database_readonly or
        (need_api and (STATUS == :api_offline or STATUS == :api_readonly))
-      redirect_to :controller => 'site', :action => 'offline'
+      if request.xhr?
+        report_error "Database offline for maintenance", :service_unavailable
+      else
+        redirect_to :controller => 'site', :action => 'offline'
+      end
     end
   end