mini_magick 5.3.1 → 5.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: edcf3d52dfa447f650be4854b37f3352979bb33c52726fe0b48174b9f66d8259
-  data.tar.gz: 979b3307e4fc554f29d2f0293e1267d968fff48a9f6948c750efae4d5dfd813e
+  metadata.gz: 4e0a993e69c42a76477e2002d0e2aeaa51feefe85e39f1fd41bacae32daa98a1
+  data.tar.gz: 30b93efd2b00c30f3f959ba24ac2995d6d57f5e72f40a025f44cb9b92492d91f
 SHA512:
-  metadata.gz: 3d53b6c00fafe8db9142358a3d56daa971e305a3a8a2161a19b8f7893fbaffd6c37bc06de1f2e5f9bd026ae8dd936102af274c118ea70f7107b0645b4fac9e27
-  data.tar.gz: 4a94a35509be48ae56ba3113778a49ca592569bc0ee711856c50d89f3b4cf277ccf207ff616b20b3e56c305fabb88a2534542ff79937fdf8e2e77e3caf541f01
+  metadata.gz: 1cff1e1f6dabb0bec2bc3a346212f0add2d2fd83c6e25c7d48f1a9c9f6ce0ab042cce699127f64152054e7dbd63efb038c8dec4c1a5d3508b1af25e59672dc50
+  data.tar.gz: f09c780d5e5eae7fb276ab137495ec89aba893ba49299ade03a3d6084075f2563f970c76543696790f3e500f9796b79a43c5b8cd85b2fc6dcf11fcda13adf184

data/lib/mini_magick/shell.rb

@@ -31,7 +31,33 @@ module MiniMagick
       env["MAGICK_TIME_LIMIT"] = timeout.to_s if timeout
 
       stdout, stderr, status = log(command.join(" ")) do
-        Open3.capture3(env, *command, stdin_data: stdin, unsetenv_others: MiniMagick.restricted_env)
+        # We would ideally use Open3.capture3, but it doesn't allow us to
+        # terminate the command after timing out. We can't rely solely on
+        # ImageMagick's own $MAGICK_TIME_LIMIT for this, because it's only
+        # checked periodically inside ImageMagick's processing loops, so it
+        # can fire too late (or not at all) depending on the operation.
+        Open3.popen3(env, *command, unsetenv_others: MiniMagick.restricted_env) do |stdin_io, stdout_io, stderr_io, wait_thread|
+          stdin_io.binmode
+          stdout_io.binmode
+          stderr_io.binmode
+
+          stdout_reader = Thread.new { stdout_io.read }
+          stderr_reader = Thread.new { stderr_io.read }
+
+          begin
+            stdin_io.write(stdin)
+          rescue Errno::EPIPE
+          end
+          stdin_io.close
+
+          if timeout && !wait_thread.join(timeout)
+            Process.kill("TERM", wait_thread.pid) rescue nil
+            wait_thread.join
+            fail MiniMagick::TimeoutError, "`#{command.join(" ")}` has timed out"
+          end
+
+          [stdout_reader.value, stderr_reader.value, wait_thread.value]
+        end
       end
 
       [stdout, stderr, status&.exitstatus]

data/lib/mini_magick/tool.rb

@@ -27,7 +27,7 @@ module MiniMagick
     #   instance of the tool, if block is given, returns the output of the
     #   command.
     #
-    def self.new(name, **options)
+    def self.new(*args, **options)
       instance = super
 
       if block_given?

data/lib/mini_magick/version.rb

@@ -9,7 +9,7 @@ module MiniMagick
   module VERSION
     MAJOR = 5
     MINOR = 3
-    TINY  = 1
+    TINY  = 2
     PRE   = nil
 
     STRING = [MAJOR, MINOR, TINY, PRE].compact.join('.')

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mini_magick
 version: !ruby/object:Gem::Version
-  version: 5.3.1
+  version: 5.3.2
 platform: ruby
 authors:
 - Corey Johnson
@@ -99,7 +99,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements:
 - You must have ImageMagick installed
-rubygems_version: 3.6.7
+rubygems_version: 4.0.13
 specification_version: 4
 summary: Manipulate images with minimal use of memory via ImageMagick
 test_files: []