semian 0.25.1 → 0.25.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8b506d59cbcde1eb9cb0adc3f406ed33f710ce6aa8f8773cfa142bd80acf7622
-  data.tar.gz: f74a8dfeb7469173e0b6b545ff8ec691ca281cc25331c33f80b375507fc2cbfb
+  metadata.gz: c6daa64288b6b4677a2abc6f4e547aa72c0cd47696b1b47fd0d7a2116b5a5cc0
+  data.tar.gz: 6fee5050c2d5d3720b22be523303bf23247ecb720361228abf7d1d15f9ed3c65
 SHA512:
-  metadata.gz: f0671d6ca7561db895eef1091ca594687f851bbbb6a00f29b7849000641eacebd2ab21cf26bd727ba47f432b285190fccd5c76152aa9f1f7c06fad360bb3cb41
-  data.tar.gz: 71749f811814a2b93f97f09321846f54d151b043ec51358b72bbeea33be6998b6b5f1b786e6d940164e7b87a31c4ad63388c57f89180850246adc7445e653f42
+  metadata.gz: 7bf4baa41c3a9ccfacc8406b15f2bd85c39138593be9318f0712da906f8c816dc94b90d5097603d94b9f341fc8923b8d61854dad5f5197dfcda2dc2050a9d83c
+  data.tar.gz: 55c9dcf87364ca17819475fbcd183f17ad8aadd3b8f32e80ce8cb3bc15a9459a02435c709f7ab057dba1eefb0c6c8e622d87acccc96dec0c493147c05fac5d12

data/lib/semian/adapter.rb

@@ -49,7 +49,8 @@ module Semian
       last_error = nil unless last_error.is_a?(Exception) # Net::HTTPServerError is not an exception
       raise self.class::CircuitOpenError.new(semian_identifier, message), cause: last_error
     rescue ::Semian::BaseError => error
-      raise self.class::ResourceBusyError.new(semian_identifier, error.message)
+      message = "Semian Resource #{semian_identifier}: #{error.message}"
+      raise self.class::ResourceBusyError.new(semian_identifier, message)
     rescue *resource_exceptions => error
       error.semian_identifier = semian_identifier if error.respond_to?(:semian_identifier=)
       raise

data/lib/semian/circuit_breaker.rb

@@ -39,7 +39,7 @@ module Semian
     def acquire(resource = nil, &block)
       transition_to_half_open if transition_to_half_open?
 
-      raise OpenCircuitError unless request_allowed?
+      raise OpenCircuitError, "Circuit breaker is open for resource: #{@name}" unless request_allowed?
 
       result = nil
       begin

data/lib/semian/configuration_validator.rb

@@ -10,7 +10,7 @@ module Semian
 
       unless @force_config_validation
         Semian.logger.info(
-          "Semian is running in log-mode for configuration validation. This means that Semian will not raise an error if the configuration is invalid. This is not recommended for production environments.\n\n[IMPORTANT] PLEASE UPDATE YOUR CONFIGURATION TO USE `force_config_validation: true` TO ENABLE STRICT CONFIGURATION VALIDATION.\n---\n",
+          "Semian Resource #{@name} is running in log-mode for configuration validation. This means that Semian will not raise an error if the configuration is invalid. This is not recommended for production environments.\n\n[IMPORTANT] PLEASE UPDATE YOUR CONFIGURATION TO USE `force_config_validation: true` TO ENABLE STRICT CONFIGURATION VALIDATION.\n---\n",
         )
       end
     end
@@ -29,6 +29,7 @@ module Semian
     end
 
     def raise_or_log_validation_required!(message)
+      message = "Semian Resource #{@name}: #{message}"
       if @force_config_validation
         raise ArgumentError, message
       else
@@ -50,7 +51,7 @@ module Semian
     end
 
     def validate_bulkhead_configuration!
-      return if ENV.key?("SEMIAN_BULKHEAD_DISABLED")
+      return if ENV.key?("SEMIAN_BULKHEAD_DISABLED") || !Semian.semaphores_enabled?
       return unless @configuration.fetch(:bulkhead, true)
 
       tickets = @configuration[:tickets]

data/lib/semian/redis_client.rb

@@ -126,6 +126,7 @@ module Semian
 
   module RedisClientPool
     include RedisClientCommon
+
     define_method(:semian_resource, Semian::Adapter.instance_method(:semian_resource))
     define_method(:clear_semian_resource, Semian::Adapter.instance_method(:clear_semian_resource))
   end

data/lib/semian/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Semian
-  VERSION = "0.25.1"
+  VERSION = "0.25.3"
 end

data/lib/semian.rb

@@ -199,9 +199,9 @@ module Semian
     # If consumer who retrieved / registered by a Semian::Adapter, keep track
     # of who the consumer was so that we can clear the resource reference if needed.
     consumer = args.delete(:consumer)
-    if consumer&.class&.include?(Semian::Adapter)
-      consumers[name] ||= []
-      consumers[name] << WeakRef.new(consumer)
+    if consumer&.class&.include?(Semian::Adapter) && !args[:dynamic]
+      consumer_set = (consumers[name] ||= ObjectSpace::WeakMap.new)
+      consumer_set[consumer] = true
     end
     self[name] || register(name, **args)
   end
@@ -233,14 +233,8 @@ module Semian
     resource = resources.delete(name)
     if resource
       resource.bulkhead&.unregister_worker
-      consumers_for_resource = consumers.delete(name) || []
-      consumers_for_resource.each do |consumer|
-        if consumer.weakref_alive?
-          consumer.clear_semian_resource
-        end
-      rescue WeakRef::RefError
-        next
-      end
+      consumers_for_resource = consumers.delete(name) || {}
+      consumers_for_resource.each_key(&:clear_semian_resource)
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: semian
 version: !ruby/object:Gem::Version
-  version: 0.25.1
+  version: 0.25.3
 platform: ruby
 authors:
 - Scott Francis
@@ -78,7 +78,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.7.1
+rubygems_version: 3.7.2
 specification_version: 4
 summary: Bulkheading for Ruby with SysV semaphores
 test_files: []