Merge pull request #5283 from betagouv/add_maintenance_to_ping

Ajoute un mode maintenance (404) a l'url de ping

app/controllers/ping_controller.rb

@@ -1,11 +1,16 @@
 class PingController < ApplicationController
   def index
     Rails.logger.silence do
-      if (ActiveRecord::Base.connected?)
-        head :ok, content_type: "application/json"
+      status_code = if File.file?(Rails.root.join("maintenance"))
+        # See https://cbonte.github.io/haproxy-dconv/2.0/configuration.html#4.2-http-check%20disable-on-404
+        :not_found
+      elsif (ActiveRecord::Base.connected?)
+        :ok
       else
-        head :internal_server_error, content_type: "application/json"
+        :internal_server_error
       end
+
+      head status_code, content_type: "application/json"
     end
   end
 end

spec/controllers/ping_controller_spec.rb

@@ -11,5 +11,15 @@ describe PingController, type: :controller do
 
       it { expect(subject.status).to eq 500 }
     end
+
+    context 'when a maintenance file is present' do
+      before do
+        allow(File).to receive(:file?).and_return(true)
+      end
+
+      it 'tells HAProxy that the app is in maintenance, but will be available again soon' do
+        expect(subject.status).to eq 404
+      end
+    end
   end
 end