Stop with the sleep + redirect since it can consume PG connections

reclaim-the-stack · Jun 18, 2024 · 1b9eccd · 1b9eccd
1 parent eb75d18
commit 1b9eccd
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/app/controllers/configs_controller.rb b/app/controllers/configs_controller.rb
@@ -17,11 +17,7 @@ def show
       render plain: server.machine_config.generate_config
     else
       server.update!(last_configured_at: nil, last_request_for_configuration_at: Time.now)
-
-      sleep 15
-
-      attempt = params[:attempt] || 0
-      redirect_to get_config_path(attempt: attempt.to_i + 1)
+      render plain: "No configuration found for server with IP #{server.ip}", status: 420
     end
   end