faktory_worker_ruby 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 804ff26f314376744d731cc7bae95efdfda57c9d835ebc32fed9efbfcd5c813d
-  data.tar.gz: 70758dc5254f96908ff969342a30e8ccdef261e1a813b1aeb30993b7df9c4e4e
+  metadata.gz: 2a183f21103562c224add2460da73a145a0c12c32a167525fd18dd02fe334241
+  data.tar.gz: b7c76d7001ba133f685427ffcc00bcdd75d026747106fac705b741e57f3d7b94
 SHA512:
-  metadata.gz: 10024bc0283cd638063ba48d54887d9f8b65aaf059a726edf40a9fe0e40dd870eced9713746e2a60db397e4057881132a939e7296277c93916b4b40de8b7ab44
-  data.tar.gz: da7660a768934e05240f5b6e228bc6a7f6dff31557127f0391f29a9c8d8c3a1bfc307235dda97f233a2d051832f9f9ff166bf9511283cfc45bd23f432e2cfc55
+  metadata.gz: 6bae416fa9d2675c586a8a40ccce0005f665eb2ab2bc19e6a5e12405cafee67aef22984d9e84f6db96012143eee87307d81e089fad5fd81d6dd278489fe62437
+  data.tar.gz: 42806775422a7542a1a7c7e912bc373dfe8c0d2b1d47d8cccb3ce8d91a0c1a95f8a835cb99cc1b9da2262d17ac3f50e7447ecefcf4eff6ba3ef0d9672e8b1f06

data/Changes.md

@@ -1,5 +1,9 @@
 # Changes
 
+## 1.0.3
+
+- Fix corruption in `custom` hash elements [#55]
+
 ## 1.0.2
 
 - Fix "batch not open" errors

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    faktory_worker_ruby (1.0.1)
+    faktory_worker_ruby (1.0.2)
       connection_pool (~> 2.2, >= 2.2.2)
 
 GEM

data/lib/faktory/job.rb

@@ -88,6 +88,11 @@ module Faktory
         pool = Thread.current[:faktory_via_pool] || item["pool"] || Faktory.server_pool
         item.delete("pool")
 
+        # the payload hash is shallow copied by `merge` calls BUT we don't deep clone
+        # the 'custom' child hash which can be problematic if we mutate it within middleware.
+        # Proactively dup it first.
+        item["custom"] = item["custom"].dup if item["custom"]
+
         Faktory.client_middleware.invoke(item, pool) do
           pool.with do |c|
             c.push(item)
@@ -99,7 +104,7 @@ module Faktory
     module ClassMethods
 
       def set(options)
-        Setter.new(options.merge!('jobtype'.freeze => self))
+        Setter.new(options.merge('jobtype'.freeze => self))
       end
 
       def perform_async(*args)

data/lib/faktory/mutate.rb

@@ -1,5 +1,11 @@
 require 'faktory/client'
 
+##
+#
+# Faktory's MUTATE API allows you to scan the sorted sets
+# within Redis (retries, scheduled, dead) and take action
+# (delete, enqueue, kill) on entries.
+#
 # require 'faktory/mutate'
 # cl = Faktory::Client.new
 # cl.discard(Faktory::RETRIES) do |filter|

data/lib/faktory/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Faktory
-  VERSION = "1.0.2"
+  VERSION = "1.0.3"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: faktory_worker_ruby
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Mike Perham
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-04 00:00:00.000000000 Z
+date: 2021-02-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: connection_pool