Fix statsd UDP sockets not being cleaned up in Sidekiq (#11230)

5 years ago · 1a747f7020
parent bc60d794f8
commit 1a747f7020
1 changed files with 5 additions and 3 deletions
--- a/app/lib/sidekiq_error_handler.rb
+++ b/app/lib/sidekiq_error_handler.rb
@ -3,9 +3,11 @@
 class SidekiqErrorHandler
  def call(*)
    yield
-  rescue Mastodon::HostValidationError => e
-    Rails.logger.error "#{e.class}: #{e.message}"
-    Rails.logger.error e.backtrace.join("\n")
+  rescue Mastodon::HostValidationError
    # Do not retry
+  ensure
+    socket = Thread.current[:statsd_socket]
+    socket&.close
+    Thread.current[:statsd_socket] = nil
  end
 end