Merge pull request rails#47728 from p8/docs/health-controller

guilleiguaran · web-flow · commit fedaf03ff796 · 2023-03-22T00:52:14.000-07:00
Use explicit module for Rails::HealthController
diff --git a/railties/lib/rails/health_controller.rb b/railties/lib/rails/health_controller.rb
@@ -31,23 +31,25 @@
 # where your application is being restarted due to a third-party service going
 # bad. Ideally, you should design your application to handle those outages
 # gracefully.
-class Rails::HealthController < ActionController::Base
-  rescue_from(Exception) { render_down }
+module Rails
+  class HealthController < ActionController::Base
+    rescue_from(Exception) { render_down }
 
-  def show
-    render_up
-  end
-
-  private
-    def render_up
-      render html: html_status(color: "green")
+    def show
+      render_up
     end
 
-    def render_down
-      render html: html_status(color: "red"), status: 500
-    end
+    private
+      def render_up
+        render html: html_status(color: "green")
+      end
 
-    def html_status(color:)
-      %(<html><body style="background-color: #{color}"></body></html>).html_safe
-    end
+      def render_down
+        render html: html_status(color: "red"), status: 500
+      end
+
+      def html_status(color:)
+        %(<html><body style="background-color: #{color}"></body></html>).html_safe
+      end
+  end
 end
