active_record_streams 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db02bce16951d859a2430c95d561322091feeb07a388d348e31b3a44ba42ddfb
-  data.tar.gz: 2ff33c6f3046974f161d94ded9eb6fac1bbf1f66a107c47c4341d88e909d53eb
+  metadata.gz: 55ce8ad67b9fb07fe31af17d43d354cedaa806eb1ee2256af3267396d509f946
+  data.tar.gz: 47247a17b827034a7e2402acce13b1bb01da23b95b24bbcaf61cb567bcd2196a
 SHA512:
-  metadata.gz: 2accfc57874c853d364c272a74dbe47a26a764071676fcdb022166f11dc6ec7d1c0fba9205f9e2ca27e49e95d372fcdb128656360d044ff9f873f5103911a9a2
-  data.tar.gz: 2a2dfdb5341d1744389f82014671284e602e351822210f873cb230881e5288a71f9ff26759c816439eaf9137fc25bf76a69a8974114d9d743d8122862c5b9da0
+  metadata.gz: d0f83bbbb70f7ccd09d0e771c5815dfbd5dd63a13730fd29a254b60696bcae66026445ce90cbccd462e1c7d7b5186059222f172e57109de3ce0f77ff5e0ff67d
+  data.tar.gz: 6bc49e902ab3a51bdcd2aff33e7e8523843d816e7014c35dc7f88b4dd2961526e46cf02d63d41b40a3b619d6b158ce6fa3074fb5bc14a4c43dba6287cc17b9d0

data/CHANGELOG.md

@@ -26,3 +26,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [0.1.3] - 2019-05-02
 ### Changed
 - `Content-type` header for the HTTP streams is set as a string to fix doubled content-type issue
+
+## [0.1.4] - 2019-05-02
+### Added
+- Error-handler switch for `publish` methods
+
+### Changed
+- README to reflect the error-handler switch

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    active_record_streams (0.1.3)
+    active_record_streams (0.1.4)
       activerecord (~> 4.2.10)
       aws-sdk (~> 2.11.9)
 

data/README.md

@@ -134,7 +134,10 @@ class SampleHttpReSender
   def perform(table_name, message_json)
     message = ActiveRecordStreams::Message.from_json(message_json)
     ActiveRecordStreams.config.streams.each do |stream|
-      stream.publish(table_name, message)
+      # Use `handle_error: false` to raise exceptions instead of
+      # calling `error_handler`, this will let he Sidekiq to
+      # perform retries on it's own 
+      stream.publish(table_name, message, handle_error: false)
     end
   end
 end

data/lib/active_record_streams/publishers/http_stream.rb

@@ -30,7 +30,12 @@ module ActiveRecordStreams
         @error_handler = error_handler
       end
 
-      def publish(table_name, message)
+      ##
+      # @param [String] table_name
+      # @param [ActiveRecordStreams::Message] message
+      # @param [Boolean] handle_error
+
+      def publish(table_name, message, handle_error: true)
         return unless (any_table? && allowed_table?(table_name)) ||
                       table_name == @table_name
 
@@ -38,7 +43,7 @@ module ActiveRecordStreams
         response = http.request(request)
         assert_response_code(response)
       rescue StandardError => e
-        raise e unless @error_handler.is_a?(Proc)
+        raise e unless call_error_handler?(handle_error)
 
         @error_handler.call(self, table_name, message, e)
       end
@@ -82,6 +87,10 @@ module ActiveRecordStreams
 
         raise StandardError, response.body
       end
+
+      def call_error_handler?(handle_error)
+        @error_handler.is_a?(Proc) && handle_error
+      end
     end
   end
 end

data/lib/active_record_streams/publishers/http_stream_spec.rb

@@ -89,6 +89,18 @@ RSpec.describe ActiveRecordStreams::Publishers::HttpStream do
       end
     end
 
+    context 'error response with error handling off' do
+      let(:response) { double(body: 'Error', code: 400) }
+      let(:error_handler) { Proc.new {} }
+
+      it 'raises exception' do
+        expect(error_handler).not_to receive(:call)
+
+        expect { subject.publish(actual_table_name, message, handle_error: false) }
+          .to raise_error(StandardError)
+      end
+    end
+
     context 'https target' do
       let(:url) { 'https://hello.world' }
 

data/lib/active_record_streams/publishers/kinesis_stream.rb

@@ -32,15 +32,16 @@ module ActiveRecordStreams
       ##
       # @param [String] table_name
       # @param [ActiveRecordStreams::Message] message
+      # @param [Boolean] handle_error
 
-      def publish(table_name, message)
+      def publish(table_name, message, handle_error: true)
         return unless (any_table? && allowed_table?(table_name)) ||
                       table_name == @table_name
 
         client.publish(@stream_name, partition_key(table_name),
                        message.json, @overrides)
       rescue StandardError => e
-        raise e unless @error_handler.is_a?(Proc)
+        raise e unless call_error_handler?(handle_error)
 
         @error_handler.call(self, table_name, message, e)
       end
@@ -55,6 +56,10 @@ module ActiveRecordStreams
         !@ignored_tables.include?(table_name)
       end
 
+      def call_error_handler?(handle_error)
+        @error_handler.is_a?(Proc) && handle_error
+      end
+
       def partition_key(table_name)
         "#{table_name}-#{Time.now.utc.strftime(PARTITION_KEY_TIME_FORMAT)}"
       end

data/lib/active_record_streams/publishers/kinesis_stream_spec.rb

@@ -110,5 +110,20 @@ RSpec.describe ActiveRecordStreams::Publishers::KinesisStream do
         subject.publish(actual_table_name, message)
       end
     end
+
+    context 'delivery error error with error handling off' do
+      let(:error_handler) { Proc.new {} }
+
+      it 'raises exception' do
+        expect(kinesis_client).to receive(:publish) do
+          raise StandardError, 'Delivery error'
+        end
+
+        expect(error_handler).not_to receive(:call)
+
+        expect { subject.publish(actual_table_name, message, handle_error: false) }
+          .to raise_error(StandardError)
+      end
+    end
   end
 end

data/lib/active_record_streams/publishers/sns_stream.rb

@@ -29,14 +29,15 @@ module ActiveRecordStreams
       ##
       # @param [String] table_name
       # @param [ActiveRecordStreams::Message] message
+      # @param [Boolean] handle_error
 
-      def publish(table_name, message)
+      def publish(table_name, message, handle_error: true)
         return unless (any_table? && allowed_table?(table_name)) ||
                       table_name == @table_name
 
         client.publish(@topic_arn, message.json, @overrides)
       rescue StandardError => e
-        raise e unless @error_handler.is_a?(Proc)
+        raise e unless call_error_handler?(handle_error)
 
         @error_handler.call(self, table_name, message, e)
       end
@@ -51,6 +52,10 @@ module ActiveRecordStreams
         !@ignored_tables.include?(table_name)
       end
 
+      def call_error_handler?(handle_error)
+        @error_handler.is_a?(Proc) && handle_error
+      end
+
       def client
         @client ||= SnsClient.new
       end

data/lib/active_record_streams/publishers/sns_stream_spec.rb

@@ -106,5 +106,20 @@ RSpec.describe ActiveRecordStreams::Publishers::SnsStream do
         subject.publish(actual_table_name, message)
       end
     end
+
+    context 'delivery error with error handling off' do
+      let(:error_handler) { Proc.new {} }
+
+      it 'raises exception' do
+        expect(sns_client).to receive(:publish) do
+          raise StandardError, 'Delivery error'
+        end
+
+        expect(error_handler).not_to receive(:call)
+
+        expect { subject.publish(actual_table_name, message, handle_error: false) }
+          .to raise_error(StandardError)
+      end
+    end
   end
 end

data/lib/active_record_streams/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveRecordStreams
-  VERSION = '0.1.3'
+  VERSION = '0.1.4'
 end

data/lib/active_record_streams/version_spec.rb

@@ -4,6 +4,6 @@ require_relative 'version'
 
 RSpec.describe ActiveRecordStreams::VERSION do
   it 'has a version number' do
-    expect(subject).to eq('0.1.3')
+    expect(subject).to eq('0.1.4')
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_record_streams
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Advanon Team