delete_message_batch failed is returning nil rather than empty array …

…as before, default to empty array if nil returned (#754)
ruby-shoryuken · Nov 28, 2023 · 7b8cbc7 · 7b8cbc7
1 parent c966255
commit 7b8cbc7
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/shoryuken/queue.rb b/lib/shoryuken/queue.rb
@@ -21,9 +21,10 @@ def visibility_timeout
     end
 
     def delete_messages(options)
-      client.delete_message_batch(
+      failed_messages = client.delete_message_batch(
         options.merge(queue_url: url)
-      ).failed.any? do |failure|
+      ).failed || []
+      failed_messages.any? do |failure|
         logger.error do
           "Could not delete #{failure.id}, code: '#{failure.code}', message: '#{failure.message}', sender_fault: #{failure.sender_fault}"
         end