sidekiq-killswitch 1.1.1 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d2c1e5555f67df5c51420ca3946edcf8786c622f7ee6d7c2ef238a3c2e42f639
-  data.tar.gz: 5607f5785571de88fae01a9abca4637bbeb35a2418db9cda87a29f0c6f5a1bcf
+  metadata.gz: eb8937ff18f8950997b4d4c1551ac6ecaeefee18613c3816d9596ddfcfd233f1
+  data.tar.gz: 71bb59018ad34d95516508546db39d9a1fc873ed89a778b8534d5e30c4a22863
 SHA512:
-  metadata.gz: 21e69f7d202a0c5f186d4f317bcb05ec8031522bd24c05bc7720be745e12f11280a5fd1f0902cf431dce750a63c9167409b36c5d4e92b9957fe916031f3a9699
-  data.tar.gz: 4399541da941edfef9a83eb555e80c1c5e98ffcb76160eaf487a06bc5415af4bcb4b5c4d0dcd299ad68d292509538f2b387b80a230fac56389b0431f0e3f1b31
+  metadata.gz: b56410e4ec667161846d8b70113517f04d6fca9abef4dd4deb2fa089e7cb838c37c3c58024d24295b3d6ef40a603f798742b2445ca9b03e204860debcc0b15e6
+  data.tar.gz: fbbe27dbf1a077322e44f48bd39bdbb3863c5ddbae630bd76300d3b3f6eba2f830b0d9334058a0bc3d8b7a9c3dab8373aae24a5fb35f60fbaaefe1461ff3ec2e

data/CHANGES.md

@@ -1,3 +1,6 @@
+### 1.1.2 / 2025-02-03
+* Trim whitespace from worker names when adding / removing from blackhole or dead queue
+
 ### 1.1.1 / 2024-08-27
 * Support for `redis-client` client (used by Sidekiq 7)
 * Remove `Sidekiq::DeadSet#kill` extension.

data/lib/sidekiq/killswitch/version.rb

@@ -2,6 +2,6 @@
 
 module Sidekiq
   module Killswitch
-    VERSION = '1.1.1'
+    VERSION = '1.1.2'
   end
 end

data/lib/sidekiq/killswitch.rb

@@ -86,7 +86,7 @@ module Sidekiq
       end
 
       def class_to_str(class_or_string)
-        class_or_string.is_a?(String) ? class_or_string : class_or_string.name
+        (class_or_string.is_a?(String) ? class_or_string : class_or_string.name).strip
       end
 
       private

data/spec/killswitch_spec.rb

@@ -57,6 +57,16 @@ RSpec.describe Sidekiq::Killswitch do
         expect(redis.hget(Sidekiq::Killswitch::BLACKHOLE_WORKERS_KEY_NAME, 'AnotherWorker')).to eq(time_now.to_s)
       end
     end
+
+    it 'trims whitespaces off worker names' do
+      time_now = stub_time_now
+
+      Sidekiq::Killswitch.blackhole_add_worker("   #{worker_name}   ")
+
+      Sidekiq::Killswitch.redis_pool do |redis|
+        expect(redis.hget(Sidekiq::Killswitch::BLACKHOLE_WORKERS_KEY_NAME, worker_name)).to eq(time_now.to_s)
+      end
+    end
   end
 
   describe '.blackhole_remove_worker' do
@@ -68,6 +78,15 @@ RSpec.describe Sidekiq::Killswitch do
         expect(redis.hexists(Sidekiq::Killswitch::BLACKHOLE_WORKERS_KEY_NAME, worker_name)).to eq(0)
       end
     end
+
+    it 'trims whitespaces off worker names' do
+      Sidekiq::Killswitch.blackhole_add_worker(worker_name)
+      Sidekiq::Killswitch.blackhole_remove_worker("   #{worker_name}   ")
+
+      Sidekiq::Killswitch.redis_pool do |redis|
+        expect(redis.hexists(Sidekiq::Killswitch::BLACKHOLE_WORKERS_KEY_NAME, worker_name)).to eq(0)
+      end
+    end
   end
 
   describe '.blackhole_worker?' do
@@ -105,6 +124,16 @@ RSpec.describe Sidekiq::Killswitch do
         expect(redis.hget(Sidekiq::Killswitch::DEAD_QUEUE_WORKERS_KEY_NAME, worker_name)).to eq(time_now.to_s)
       end
     end
+
+    it 'trims whitespaces off worker names' do
+      time_now = stub_time_now
+
+      Sidekiq::Killswitch.dead_queue_add_worker("   #{worker_name}   ")
+
+      Sidekiq::Killswitch.redis_pool do |redis|
+        expect(redis.hget(Sidekiq::Killswitch::DEAD_QUEUE_WORKERS_KEY_NAME, worker_name)).to eq(time_now.to_s)
+      end
+    end
   end
 
   describe '.dead_queue_remove_worker' do
@@ -116,6 +145,15 @@ RSpec.describe Sidekiq::Killswitch do
         expect(redis.hexists(Sidekiq::Killswitch::DEAD_QUEUE_WORKERS_KEY_NAME, worker_name)).to eq(0)
       end
     end
+
+    it 'trims whitespaces off worker names' do
+      Sidekiq::Killswitch.dead_queue_add_worker(worker_name)
+      Sidekiq::Killswitch.dead_queue_remove_worker("   #{worker_name}   ")
+
+      Sidekiq::Killswitch.redis_pool do |redis|
+        expect(redis.hexists(Sidekiq::Killswitch::DEAD_QUEUE_WORKERS_KEY_NAME, worker_name)).to eq(0)
+      end
+    end
   end
 
   describe '.dead_queue_worker?' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sidekiq-killswitch
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.1.2
 platform: ruby
 authors:
 - Yuriy Naidyon
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-11-04 00:00:00.000000000 Z
+date: 2025-02-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sidekiq
@@ -132,7 +132,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.26
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: Cross-host Sidekiq worker killswitches