Merge pull request #5708 from betagouv/remove_rack_mini_profiler_from_prod

Remove rack_mini_profiler from prod

Gemfile

@@ -60,7 +60,6 @@ gem 'premailer-rails'
 gem 'puma' # Use Puma as the app server
 gem 'pundit'
 gem 'rack-attack'
-gem 'rack-mini-profiler'
 gem 'rails'
 gem 'rails-i18n' # Locales par défaut
 gem 'rake-progressbar', require: false
@@ -104,6 +103,7 @@ group :development do
   gem 'brakeman', require: false
   gem 'haml-lint'
   gem 'letter_opener_web'
+  gem 'rack-mini-profiler'
   gem 'rails-erd', require: false # generates `doc/database_models.pdf`
   gem 'rubocop', require: false
   gem 'rubocop-rails_config'

app/controllers/application_controller.rb

@@ -12,7 +12,6 @@ class ApplicationController < ActionController::Base
   before_action :load_navbar_left_pannel_partial_url
   before_action :set_raven_context
   before_action :redirect_if_untrusted
-  before_action :authorize_request_for_profiler
   before_action :reject, if: -> { feature_enabled?(:maintenance_mode) }
 
   before_action :staging_authenticate
@@ -30,12 +29,6 @@ class ApplicationController < ActionController::Base
     end
   end
 
-  def authorize_request_for_profiler
-    if feature_enabled?(:mini_profiler)
-      Rack::MiniProfiler.authorize_request
-    end
-  end
-
   def load_navbar_left_pannel_partial_url
     controller = request.controller_class
     method = params[:action]

config/initializers/rack_mini_profiler.rb

@@ -1 +1,3 @@
-Rack::MiniProfiler.config.authorization_mode = :whitelist
+if Rails.env.development?
+  Rack::MiniProfiler.config.authorization_mode = :whitelist
+end