pwwka 0.7.0.RC1 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 86fe2875f9d9221237358a25d30bcecee224554a
-  data.tar.gz: c1cad94e3a3d146b18480354297b5214544ecca8
+  metadata.gz: 5d748e0097f28d4bac944b11074a16298e986063
+  data.tar.gz: e8bbbf37f1e6d71c95b3ce448dfb09f6e8eaa78b
 SHA512:
-  metadata.gz: b56b412dc2ce73c3dec1ca1ca1f20432bbea8565699aff56e006fa00df9605e627a58db4a6aca9db248890d5cf70a2e69e5bbe4fb093f066d65a2c706584f4ea
-  data.tar.gz: 154f1b930db5502349df6b0c8b15723ce4f726c95b4d742db5b277078c097afb529b17639634a9b31b079497bf1b4dfe171e55b9e903ea924d55a9746f83947d
+  metadata.gz: b1074d5e3420223ca2d3ea219119f288adce4b464d15dd6312ac7ae8880ad757b335b2f8d78762649d532f42a022eda478fc9980be7e30508f9df7cdf25a9d5e
+  data.tar.gz: 87a47d11cf6e95683393162cc60ca3986a802394a4cf6ff583883ee7d7794f68ef15cf6e63e6d97408a035b55cb44fdb7a92629a05c352f3e9bb5602f00d8812

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    pwwka (0.7.0.RC1)
+    pwwka (0.7.0)
       activemodel
       activesupport
       bunny
@@ -10,20 +10,23 @@ PATH
 GEM
   remote: https://www.rubygems.org/
   specs:
-    activemodel (5.0.0.1)
-      activesupport (= 5.0.0.1)
-    activesupport (5.0.0.1)
-      concurrent-ruby (~> 1.0, >= 1.0.2)
+    activemodel (4.2.6)
+      activesupport (= 4.2.6)
+      builder (~> 3.1)
+    activesupport (4.2.6)
       i18n (~> 0.7)
+      json (~> 1.7, >= 1.7.7)
       minitest (~> 5.1)
+      thread_safe (~> 0.3, >= 0.3.4)
       tzinfo (~> 1.1)
     amq-protocol (2.0.1)
-    bunny (2.6.1)
+    builder (3.2.2)
+    bunny (2.3.1)
       amq-protocol (>= 2.0.1)
-    concurrent-ruby (1.0.2)
     diff-lcs (1.2.5)
     i18n (0.7.0)
-    minitest (5.10.1)
+    json (1.8.3)
+    minitest (5.8.4)
     mono_logger (1.1.0)
     multi_json (1.11.2)
     rack (1.6.4)
@@ -73,4 +76,4 @@ DEPENDENCIES
   rspec
 
 BUNDLED WITH
-   1.13.6
+   1.12.5

data/README.md

@@ -98,12 +98,12 @@ send_message!(payload, routing_key)
 ```
 
 ### Error Handling
-
 This method accepts several strategies for handling errors, pass in using the `on_error` parameter:
 
   * `:raise`: Log the error and raise the exception received from Bunny. (default strategy)
   * `:ignore`: Log the error and return false.
-  * `:resque`: Log the error and return false. Also, enqueue a job with Resque to send the message. See `send_message_async` below. **Note, this doesn't guarantee the message will actually be sent—it just guarantees an attempt is made to queue a Resque job [which could fail]**
+  * `:resque`: Log the error and return false. Also, enqueue a job with Resque 
+    to send the message. See `send_message_async` below.
 
 ### Delayed Messages
 You might want to delay sending a message (for example, if you have just created a database 
@@ -217,32 +217,6 @@ class ClientIndexMessageHandler
 
 end
 ```
-#### Handling Errors
-
-By default, if your handler raises an uncaught exception, the message will be Nacked, **but not requeued**.  This means
-it's dropped on the floor and likely won't have been completely processed.
-
-You can configure `requeue_on_error` in the configuration to change this behavior:
-
-```ruby
-require 'pwwka'
-Pwwka.configure do |config|
-
-  # ...
-
-  config.requeue_on_error = true
-end
-```
-
-This will requeue the message **exactly once**.  It uses the headers to check if the message has been retried.  If it
-hasn't, and your handler raises an exception, it will be placed back on the queue.  The second time your handler
-processes it, there is a header indicating it's been retried, so if a failure happens again, the message **is not
-requeued**.
-
-Because requeuing puts the message at the head of the queue, a hard failure will result in an infinite loop, which will
-lead to filling up your queue. Nevertheless, this should address intermittent failures.
-
-**It is recommended that you set this option**.  It's off for backwards compatibility.
 
 #### Handling Messages with Resque
 

data/lib/pwwka/channel_connector.rb

@@ -27,18 +27,6 @@ module Pwwka
     end
 
     def delayed_queue
-      # This works by hacking the dead letter exchange concept with a timeout.
-      # We set up a delayed exchange that has a delayed queue.  This queue, configured below,
-      # sets its dead letter exchange to be the main exchange (topic_exchange above).
-      #
-      # This means that when a message send to the delayed queue is either nack'ed with no retry OR
-      # it's TTL expires, it will be sent to the configured dead letter exchange, which is the main topic_exchange.
-      #
-      # Since nothing is actually consuming messages on the delayed queue, the only way messages can be removed and
-      # sent back to the main exchange is if their TTL expires.  As you can see in Pwwka::Transmitter#send_delayed_message!
-      # we set an expiration on the message and send it to the delayed exchange.  This means that the delay time is the TTL,
-      # so the messages sits in the delayed queue until its TTL/delay expires, and then it's sent onto the 
-      # main exchange for everyone to consume.  Thus creating a delay.
       raise_if_delayed_not_allowed
       @delayed_queue ||= begin
         queue = channel.queue("pwwka_delayed_#{Pwwka.environment}", durable: true,
@@ -47,7 +35,7 @@ module Pwwka
         })
         queue.bind(delayed_exchange)
         queue
-      end
+      end 
     end
     alias :create_delayed_queue :delayed_queue
 

data/lib/pwwka/configuration.rb

@@ -10,7 +10,6 @@ module Pwwka
     attr_accessor :logger
     attr_accessor :options
     attr_accessor :send_message_resque_backoff_strategy
-    attr_accessor :requeue_on_error
 
     def initialize
       @rabbit_mq_host        = nil
@@ -21,7 +20,6 @@ module Pwwka
       @send_message_resque_backoff_strategy = [5,                  #intermittent glitch?
                                                60,                 # quick interruption
                                                600, 600, 600] # longer-term outage?
-      @requeue_on_error = false
     end
 
     def payload_logging

data/lib/pwwka/receiver.rb

@@ -29,13 +29,9 @@ module Pwwka
             receiver.ack(delivery_info.delivery_tag)
             logf "Processed Message on %{queue_name} -> %{payload}, %{routing_key}", queue_name: queue_name, payload: payload, routing_key: delivery_info.routing_key
           rescue => e
-            if Pwwka.configuration.requeue_on_error && !delivery_info.redelivered
-              logf "Retrying an Error Processing Message on %{queue_name} -> %{payload}, %{routing_key}: %{exception}: %{backtrace}", queue_name: queue_name, payload: payload, routing_key: delivery_info.routing_key, exception: e, backtrace: e.backtrace.join(';'), at: :error
-              receiver.nack_requeue(delivery_info.delivery_tag)
-            else
-              logf "Error Processing Message on %{queue_name} -> %{payload}, %{routing_key}: %{exception}: %{backtrace}", queue_name: queue_name, payload: payload, routing_key: delivery_info.routing_key, exception: e, backtrace: e.backtrace.join(';'), at: :error
-              receiver.nack(delivery_info.delivery_tag)
-            end
+            logf "Error Processing Message on %{queue_name} -> %{payload}, %{routing_key}: %{exception}: %{backtrace}", queue_name: queue_name, payload: payload, routing_key: delivery_info.routing_key, exception: e, backtrace: e.backtrace.join(';'), at: :error
+            # no requeue
+            receiver.nack(delivery_info.delivery_tag)
           end
         end
       rescue Interrupt => _
@@ -48,7 +44,7 @@ module Pwwka
 
     def topic_queue
       @topic_queue ||= begin
-        queue = channel.queue(queue_name, durable: true, arguments: {})
+        queue = channel.queue(queue_name, durable: true)
         queue.bind(topic_exchange, routing_key: routing_key)
         queue
       end
@@ -67,14 +63,14 @@ module Pwwka
     end
 
     def drop_queue
-      topic_queue.purge
+      topic_queue.purge  
       topic_queue.delete
     end
 
     def test_teardown
       drop_queue
       topic_exchange.delete
-      channel_connector.connection_close
+      channel_connector.connection_close 
     end
 
   end

data/lib/pwwka/test_handler.rb

@@ -30,7 +30,7 @@ module Pwwka
                        end
     end
 
-    # Get the message on the queue as TestHandler::Message
+    # Get the message on the queue as TestHandler::Message.
     def pop_message
       delivery_info, properties, payload = test_queue.pop
       Message.new(delivery_info,
@@ -70,7 +70,7 @@ module Pwwka
         channel_connector.delayed_exchange.delete
       end
 
-      channel_connector.connection_close
+      channel_connector.connection_close 
     end
 
     # Simple class to hold a popped message.

data/lib/pwwka/version.rb

@@ -1,3 +1,3 @@
 module Pwwka
-  VERSION = '0.7.0.RC1'
+  VERSION = '0.7.0'
 end

data/spec/receiver_spec.rb

@@ -25,7 +25,6 @@ describe Pwwka::Receiver do
 
     after(:each) do
       Pwwka.configuration.logger = @original_logger
-      Pwwka.configuration.requeue_on_error = false
       @receiver.test_teardown rescue nil
     end
 
@@ -54,47 +53,6 @@ describe Pwwka::Receiver do
       expect(logger).to have_received(:error).with(/Error Processing Message.*#{Regexp.escape(payload.to_s)}.*#{Regexp.escape(exception.backtrace.join(';'))}/)
     end
 
-    context "when we're configured to requeue on error" do
-      before do
-        Pwwka.configuration.requeue_on_error = true
-      end
-      it "should nack_requeue the sent message if it hasn't been retried before" do
-        exception = begin
-                      raise "blow up"
-                    rescue => ex
-                      ex
-                    end
-        expect(HandyHandler).to receive(:handle!).and_raise(ex)
-        expect(@receiver).not_to receive(:ack)
-        expect(@receiver).to receive(:nack_requeue).with(instance_of(Fixnum))
-        Pwwka::Transmitter.send_message!(payload, routing_key)
-        @receiver.test_teardown # force the message to be processed and exception handled
-        expect(logger).to have_received(:info).with(/START Transmitting.*#{Regexp.escape(payload.to_s)}/)
-        expect(logger).to have_received(:info).with(/END Transmitting.*#{Regexp.escape(payload.to_s)}/)
-        expect(logger).to have_received(:info).with(/AFTER Transmitting.*#{Regexp.escape(payload.to_s)}/)
-        expect(logger).to have_received(:error).with(/Error Processing Message.*#{Regexp.escape(payload.to_s)}.*#{Regexp.escape(exception.backtrace.join(';'))}/)
-      end
-
-      it "should nack the sent message if it HAS been retried before" do
-        # Super cheesy, but I don't see another way to access this
-        allow_any_instance_of(Bunny::DeliveryInfo).to receive(:redelivered).and_return(true)
-        exception = begin
-                      raise "blow up"
-                    rescue => ex
-                      ex
-                    end
-        expect(HandyHandler).to receive(:handle!).and_raise(ex)
-        expect(@receiver).not_to receive(:ack)
-        expect(@receiver).to receive(:nack).with(instance_of(Fixnum))
-        Pwwka::Transmitter.send_message!(payload, routing_key)
-        @receiver.test_teardown # force the message to be processed and exception handled
-        expect(logger).to have_received(:info).with(/START Transmitting.*#{Regexp.escape(payload.to_s)}/)
-        expect(logger).to have_received(:info).with(/END Transmitting.*#{Regexp.escape(payload.to_s)}/)
-        expect(logger).to have_received(:info).with(/AFTER Transmitting.*#{Regexp.escape(payload.to_s)}/)
-        expect(logger).to have_received(:error).with(/Error Processing Message.*#{Regexp.escape(payload.to_s)}.*#{Regexp.escape(exception.backtrace.join(';'))}/)
-      end
-    end
-
   end
 
   describe "instance methods and ::new" do

data/spec/spec_helper.rb

@@ -10,7 +10,7 @@ RSpec.configure do |config|
     c.syntax = :expect
   end
 
-  config.before(:all) do
+  config.before(:each) do
     Pwwka.configure do |c|
       c.topic_exchange_name     = "topics-test"
       c.logger                  = MonoLogger.new("/dev/null")

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pwwka
 version: !ruby/object:Gem::Version
-  version: 0.7.0.RC1
+  version: 0.7.0
 platform: ruby
 authors:
 - Stitch Fix Engineering
@@ -15,7 +15,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-15 00:00:00.000000000 Z
+date: 2016-09-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bunny
@@ -183,12 +183,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.6
 signing_key: 
 specification_version: 4
 summary: Send and receive messages via RabbitMQ