active_publisher 1.2.4-java → 1.2.5-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9716e4719f25f9859f126740053aa8876da44015057ef541c6473a1999ce3be9
-  data.tar.gz: 05a592d671869f9aaa39b127692454c6c95a5cc53d8cfeac4b7cce34cc42c231
+  metadata.gz: 7a878924c71e32c34435e80052a8f241b357d2f7b70f3582f7b8920b942e63c0
+  data.tar.gz: 60b4eb97d20e389ce4865ef393d69a7d49bdac2e47a3aa26eb6e7ca32e459335
 SHA512:
-  metadata.gz: 7756e073e9213f7a900837748c8753863ff4802ecb673190a952f38a90347bde3c1bd158df9644dca1c15eb9333495388835cee8f93cdf7f92dabe93cfad2c63
-  data.tar.gz: 9bcb7dacca3a4479f6d0dfb5b5573435ea40723477b7ce26ca83198c98d43de978512de5ee4209ae062d1d1759fee49b4bddac953792b68df836f7691e090e49
+  metadata.gz: 8f6399701888401993164b31ccb4fff4f14eb13414a7586d7baacac0c49f2c876dfe33035ce091ec4b62ee4a5b64ad961a7deab79691098401388153404c724b
+  data.tar.gz: 2a1d1347293457b9e5b663d7f4815a176e8d9afdc80e472893c4f1f5206bdfc057bfb6d692067924727aac1e3a6f42bc2fff20cd13698bca12be61e1b72d84bc

data/lib/active_publisher/async/in_memory_adapter/consumer_thread.rb

@@ -52,8 +52,8 @@ module ActivePublisher
         end
 
         def cleanup_up_channel
-          return if @channel.nil?
-          @channel.close
+          return if channel.nil?
+          channel.close
         rescue => error
           ::ActivePublisher.configuration.error_handler.call(error, {:status => "Cleaning up the channel"})
         end
@@ -89,7 +89,6 @@ module ActivePublisher
 
         def start_thread
           return if alive?
-          @channel = make_channel
           @thread = ::Thread.new { start_consuming_thread }
         end
 
@@ -102,13 +101,15 @@ module ActivePublisher
             # If the queue is empty, we should continue to update to "last_tick_at" time.
             next if current_messages.nil?
 
+            @channel ||= make_channel
+
             # We only look at active publisher messages. Everything else is dropped.
             current_messages.select! { |message| message.is_a?(::ActivePublisher::Message) }
 
             begin
               # Only open a single connection for each group of messages to an exchange
               current_messages.group_by(&:exchange_name).each do |exchange_name, messages|
-                publish_all(@channel, exchange_name, messages)
+                publish_all(exchange_name, messages)
                 current_messages -= messages
               end
             rescue *CHANNEL_CLOSED_ERRORS
@@ -135,7 +136,7 @@ module ActivePublisher
           cleanup_up_channel
         end
 
-        def publish_all(channel, exchange_name, messages)
+        def publish_all(exchange_name, messages)
           exchange = channel.topic(exchange_name)
           messages.each do |message|
             fail ::ActivePublisher::ExchangeMismatchError, "bulk publish messages must match publish_all exchange_name" if message.exchange_name != exchange_name
@@ -144,10 +145,10 @@ module ActivePublisher
               exchange.publish(message.payload, options)
             end
           end
-          wait_for_confirms(channel)
+          wait_for_confirms
         end
 
-        def wait_for_confirms(channel)
+        def wait_for_confirms
           return true unless channel.using_publisher_confirms?
           channel.wait_for_confirms(::ActivePublisher.configuration.publisher_confirms_timeout)
         end

data/lib/active_publisher/version.rb

@@ -1,3 +1,3 @@
 module ActivePublisher
-  VERSION = "1.2.4"
+  VERSION = "1.2.5"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_publisher
 version: !ruby/object:Gem::Version
-  version: 1.2.4
+  version: 1.2.5
 platform: java
 authors:
 - Brian Stien
@@ -12,7 +12,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-11-20 00:00:00.000000000 Z
+date: 2020-11-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement