image_vise 0.1.6 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2176fbaf15e4bc2d36a698796bfd934f84100070
-  data.tar.gz: 56396a529ce373d32b7a2a27522c8ad82ac65bf9
+  metadata.gz: fe3aff4ba407443d636d9b23b4d7a2a55d74fc8d
+  data.tar.gz: 34a54619e0a29cb45594975dcf8b74b43742bb49
 SHA512:
-  metadata.gz: cb597e00191e3f4428c35f295041ba0b93181f7fb309f3bb2790d93641d1487a3d9f374acc260a20b93b3acc2987decbf0960266cd2518d2b006243e7dc46e4b
-  data.tar.gz: ac54cb79ad6d908e27fb2b0e26dd538a84db39fea929bdf72928587dcc4378764ed4a02fac458e6e0ef28fbb0ca6e3264cecfd740e4bd1d37b0744ba83675d76
+  metadata.gz: 8db53ad9d51d8a367cc316472a54fcb9eaa54e6d0c666fbe16de26ad00c336a8796c390bc343677fee635cf455352414bee43782899a162bb30089d79bc82f72
+  data.tar.gz: 22bc8553e556428e459dc794c123a001dce16f4df527336ecaec8cafa7f9c8f6fd91cad80d20414c0bbc2e03afb28e4c08a507decfcf2d9d4c22348cd91d863d

data/.gitignore

@@ -0,0 +1,8 @@
+scratchpad
+Gemfile.lock
+.bundle
+.env
+*.sqlite3
+*.log
+pkg
+coverage

data/.travis.yml

@@ -0,0 +1,13 @@
+rvm:
+- 2.1
+- 2.2
+- 2.3.0
+- 2.4.1
+sudo: false
+cache: bundler
+
+env:
+  global:
+    - SKIP_INTERACTIVE=yes
+
+script: bundle exec rspec

data/DEVELOPMENT.md

@@ -109,14 +109,3 @@ actually using them. If you are using a library it is a one-time cost, with very
 Also note that image operators are not per definition Imagemagick-specific - it's completely possible to not only use
 a different library for processing them, but even to use a different image processing server complying to the
 same protocol (a signed JSON-encodded waybill of HTTP(S) source-URL + pipeline instructions).
-
-## Using forked child processes for RMagick tasks
-
-You can optionally set the `IMAGE_VISE_ENABLE_FORK` environment variable to `yes` to enable forking. When this
-variable is set, ImageVise will fork a child process and perform the image processing task within that process,
-killing it afterwards and deallocating all the memory. This can be extremely efficient for dealing with potential
-memory bloat issues in ImageMagick/RMagick. However, loading images into RMagick may hang in a forked child. This
-will lead to the child being timeout-terminated, and no image is going to be rendered. This issue is known and
-also platform-dependent (it does not happen on OSX but does happen on Docker within Ubuntu Trusty for instance).
-
-So, this feature _does_ exist but your mileage may vary with regards to it's use.

data/Gemfile

@@ -1,22 +1,4 @@
 source 'https://rubygems.org'
 
-gem 'patron', '~> 0.6'
-gem 'rmagick', '~> 2.15', :require => 'rmagick'
-gem 'exceptional_fork', '~> 1.2'
-gem 'ks'
-gem 'magic_bytes'
-
-group :development do
-  gem 'bundler'
-  gem 'yard'
-  gem 'simplecov'
-  gem 'rack-cache'
-  gem 'strenv'
-  gem 'addressable', :require => %w( addressable/uri )
-  gem 'rack', '~> 1'
-  gem 'rack-test'
-  gem 'foreman'
-  gem 'rspec', '~> 3.2', '< 3.3'
-  gem 'rake', '~> 10'
-  gem "jeweler"
-end
+# Specify your gem's dependencies in image_vise.gemspec
+gemspec

data/Rakefile

@@ -1,29 +1,6 @@
-require 'rspec/core/rake_task'
-require 'jeweler'
-require_relative 'lib/image_vise'
+require "bundler/gem_tasks"
+require "rspec/core/rake_task"
 
-Jeweler::Tasks.new do |gem|
-  gem.version = ImageVise::VERSION
-  gem.name = "image_vise"
-  gem.summary = "Runtime thumbnailing proxy"
-  gem.description = "Image processing via URLs"
-  gem.email = "me@julik.nl"
-  gem.homepage = "https://github.com/WeTransfer/image_vise"
-  gem.authors = ["Julik Tarkhanov"]
-  gem.license = 'MIT'
+RSpec::Core::RakeTask.new(:spec)
 
-  # Do not package invisibles
-  gem.files.exclude ".*"
-  
-  # When running as a gem, do not lock all of our versions
-  # even though the lockfile is in the repo for running standalone
-  gem.files.exclude "Gemfile.lock"
-end
-
-Jeweler::RubygemsDotOrgTasks.new
-
-RSpec::Core::RakeTask.new(:spec) do |t|
-  t.rspec_opts = ["-c", "-f progress", "-r ./spec/spec_helper.rb"]
-  t.pattern = 'spec/**/*_spec.rb'
-end
 task :default => :spec

data/image_vise.gemspec

@@ -1,138 +1,43 @@
-# Generated by jeweler
-# DO NOT EDIT THIS FILE DIRECTLY
-# Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
-# -*- encoding: utf-8 -*-
-# stub: image_vise 0.1.6 ruby lib
+# coding: utf-8
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'image_vise/version'
 
-Gem::Specification.new do |s|
-  s.name = "image_vise"
-  s.version = "0.1.6"
+Gem::Specification.new do |spec|
+  spec.name          = "image_vise"
+  spec.version       = ImageVise::VERSION
+  spec.authors       = ["Julik Tarkhanov"]
+  spec.email         = ["me@julik.nl"]
 
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.require_paths = ["lib"]
-  s.authors = ["Julik Tarkhanov"]
-  s.date = "2016-12-07"
-  s.description = "Image processing via URLs"
-  s.email = "me@julik.nl"
-  s.extra_rdoc_files = [
-    "LICENSE.txt",
-    "README.md"
-  ]
-  s.files = [
-    "DEVELOPMENT.md",
-    "Gemfile",
-    "LICENSE.txt",
-    "README.md",
-    "Rakefile",
-    "SECURITY.md",
-    "examples/config.ru",
-    "examples/custom_image_operator.rb",
-    "examples/error_handline_appsignal.rb",
-    "examples/error_handling_sentry.rb",
-    "image_vise.gemspec",
-    "lib/image_vise.rb",
-    "lib/image_vise/fetchers/fetcher_file.rb",
-    "lib/image_vise/fetchers/fetcher_http.rb",
-    "lib/image_vise/file_response.rb",
-    "lib/image_vise/image_request.rb",
-    "lib/image_vise/operators/auto_orient.rb",
-    "lib/image_vise/operators/crop.rb",
-    "lib/image_vise/operators/ellipse_stencil.rb",
-    "lib/image_vise/operators/fit_crop.rb",
-    "lib/image_vise/operators/geom.rb",
-    "lib/image_vise/operators/sRGB_v4_ICC_preference_displayclass.icc",
-    "lib/image_vise/operators/sharpen.rb",
-    "lib/image_vise/operators/srgb.rb",
-    "lib/image_vise/operators/strip_metadata.rb",
-    "lib/image_vise/pipeline.rb",
-    "lib/image_vise/render_engine.rb",
-    "spec/image_vise/auto_orient_spec.rb",
-    "spec/image_vise/crop_spec.rb",
-    "spec/image_vise/ellipse_stencil_spec.rb",
-    "spec/image_vise/fetcher_file_spec.rb",
-    "spec/image_vise/fetcher_http_spec.rb",
-    "spec/image_vise/file_response_spec.rb",
-    "spec/image_vise/fit_crop_spec.rb",
-    "spec/image_vise/geom_spec.rb",
-    "spec/image_vise/image_request_spec.rb",
-    "spec/image_vise/pipeline_spec.rb",
-    "spec/image_vise/render_engine_spec.rb",
-    "spec/image_vise/sharpen_spec.rb",
-    "spec/image_vise/srgb_spec.rb",
-    "spec/image_vise/strip_metadata_spec.rb",
-    "spec/image_vise_spec.rb",
-    "spec/layers-with-blending.psd",
-    "spec/spec_helper.rb",
-    "spec/test_server.rb",
-    "spec/waterside_magic_hour.jpg",
-    "spec/waterside_magic_hour.psd",
-    "spec/waterside_magic_hour_adobergb.jpg",
-    "spec/waterside_magic_hour_gray.tif",
-    "spec/waterside_magic_hour_transp.png"
-  ]
-  s.homepage = "https://github.com/WeTransfer/image_vise"
-  s.licenses = ["MIT"]
-  s.rubygems_version = "2.4.5.1"
-  s.summary = "Runtime thumbnailing proxy"
+  spec.summary       = "Runtime thumbnailing proxy"
+  spec.description   = "Image processing via URLs"
+  spec.homepage      = "https://github.com/WeTransfer/image_vise"
+  spec.license       = "MIT"
 
-  if s.respond_to? :specification_version then
-    s.specification_version = 4
-
-    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<patron>, ["~> 0.6"])
-      s.add_runtime_dependency(%q<rmagick>, ["~> 2.15"])
-      s.add_runtime_dependency(%q<exceptional_fork>, ["~> 1.2"])
-      s.add_runtime_dependency(%q<ks>, [">= 0"])
-      s.add_runtime_dependency(%q<magic_bytes>, [">= 0"])
-      s.add_development_dependency(%q<bundler>, [">= 0"])
-      s.add_development_dependency(%q<yard>, [">= 0"])
-      s.add_development_dependency(%q<simplecov>, [">= 0"])
-      s.add_development_dependency(%q<rack-cache>, [">= 0"])
-      s.add_development_dependency(%q<strenv>, [">= 0"])
-      s.add_development_dependency(%q<addressable>, [">= 0"])
-      s.add_development_dependency(%q<rack>, ["~> 1"])
-      s.add_development_dependency(%q<rack-test>, [">= 0"])
-      s.add_development_dependency(%q<foreman>, [">= 0"])
-      s.add_development_dependency(%q<rspec>, ["< 3.3", "~> 3.2"])
-      s.add_development_dependency(%q<rake>, ["~> 10"])
-      s.add_development_dependency(%q<jeweler>, [">= 0"])
-    else
-      s.add_dependency(%q<patron>, ["~> 0.6"])
-      s.add_dependency(%q<rmagick>, ["~> 2.15"])
-      s.add_dependency(%q<exceptional_fork>, ["~> 1.2"])
-      s.add_dependency(%q<ks>, [">= 0"])
-      s.add_dependency(%q<magic_bytes>, [">= 0"])
-      s.add_dependency(%q<bundler>, [">= 0"])
-      s.add_dependency(%q<yard>, [">= 0"])
-      s.add_dependency(%q<simplecov>, [">= 0"])
-      s.add_dependency(%q<rack-cache>, [">= 0"])
-      s.add_dependency(%q<strenv>, [">= 0"])
-      s.add_dependency(%q<addressable>, [">= 0"])
-      s.add_dependency(%q<rack>, ["~> 1"])
-      s.add_dependency(%q<rack-test>, [">= 0"])
-      s.add_dependency(%q<foreman>, [">= 0"])
-      s.add_dependency(%q<rspec>, ["< 3.3", "~> 3.2"])
-      s.add_dependency(%q<rake>, ["~> 10"])
-      s.add_dependency(%q<jeweler>, [">= 0"])
-    end
+  # Prevent pushing this gem to RubyGems.org. To allow pushes either set the 'allowed_push_host'
+  # to allow pushing to a single host or delete this section to allow pushing to any host.
+  if spec.respond_to?(:metadata)
+    spec.metadata['allowed_push_host'] = "https://rubygems.org"
   else
-    s.add_dependency(%q<patron>, ["~> 0.6"])
-    s.add_dependency(%q<rmagick>, ["~> 2.15"])
-    s.add_dependency(%q<exceptional_fork>, ["~> 1.2"])
-    s.add_dependency(%q<ks>, [">= 0"])
-    s.add_dependency(%q<magic_bytes>, [">= 0"])
-    s.add_dependency(%q<bundler>, [">= 0"])
-    s.add_dependency(%q<yard>, [">= 0"])
-    s.add_dependency(%q<simplecov>, [">= 0"])
-    s.add_dependency(%q<rack-cache>, [">= 0"])
-    s.add_dependency(%q<strenv>, [">= 0"])
-    s.add_dependency(%q<addressable>, [">= 0"])
-    s.add_dependency(%q<rack>, ["~> 1"])
-    s.add_dependency(%q<rack-test>, [">= 0"])
-    s.add_dependency(%q<foreman>, [">= 0"])
-    s.add_dependency(%q<rspec>, ["< 3.3", "~> 3.2"])
-    s.add_dependency(%q<rake>, ["~> 10"])
-    s.add_dependency(%q<jeweler>, [">= 0"])
+    raise "RubyGems 2.0 or newer is required to protect against public gem pushes."
   end
-end
 
+  spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  spec.bindir        = "exe"
+  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.require_paths = ["lib"]
+
+  spec.add_dependency 'patron', '~> 0.6'
+  spec.add_dependency 'rmagick', '~> 2.15'
+  spec.add_dependency 'ks'
+  spec.add_dependency 'magic_bytes', '~> 1'
+  spec.add_dependency 'rack', '~> 1'
+  
+  spec.add_development_dependency "bundler", "~> 1.7"
+  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "rack-test"
+  spec.add_development_dependency "rspec", "~> 3.0"
+  spec.add_development_dependency "addressable"
+  spec.add_development_dependency "strenv"
+  spec.add_development_dependency "simplecov"
+end

data/lib/image_vise/file_response.rb

@@ -5,7 +5,7 @@ class ImageVise::FileResponse
   def initialize(file)
     @file = file
   end
-  
+
   def each
     @file.flush # Make sure all the writes have been synchronized
     # We can easily open another file descriptor
@@ -15,7 +15,7 @@ class ImageVise::FileResponse
       end
     end
   end
-  
+
   def close
     ImageVise.close_and_unlink(@file)
   end

data/lib/image_vise/image_request.rb

@@ -1,3 +1,5 @@
+require 'openssl'
+
 class ImageVise::ImageRequest < Ks.strict(:src_url, :pipeline)
   class InvalidRequest < ArgumentError; end
   class SignatureError < InvalidRequest; end

data/lib/image_vise/operators/background_fill.rb

@@ -0,0 +1,18 @@
+# Applies a background fill color.
+# Can handle most 'word' colors and hex color codes but not RGB values.
+#
+# The corresponding Pipeline method is `background_fill`.
+class ImageVise::BackgroundFill < Ks.strict(:color)
+  def initialize(*)
+    super
+    self.color = color.to_s
+    raise ArgumentError, "the :color parameter must be present and not empty" if self.color.empty?
+  end
+
+  def apply!(image)
+    image.border!(0, 0, color)
+    image.alpha(Magick::DeactivateAlphaChannel)
+  end
+
+  ImageVise.add_operator 'background_fill', self
+end

data/lib/image_vise/operators/ellipse_stencil.rb

@@ -9,7 +9,8 @@
 # The corresponding Pipeline method is `ellipse_stencil`.
 class ImageVise::EllipseStencil
   C_black = 'black'.freeze
-  private_constant :C_black
+  C_white = 'white'.freeze
+  private_constant :C_white, :C_black
 
   def apply!(magick_image)
     width, height = magick_image.columns, magick_image.rows
@@ -25,9 +26,8 @@ class ImageVise::EllipseStencil
     # must be taken not to overmult or overdivide.
     #
     # To begin,generate a black and white image for the stencil
-    circle_img = Magick::Image.new(width, height)
-    draw_circle(circle_img, width, height)
-    mask = circle_img.negate
+    mask = Magick::Image.new(width, height)
+    draw_circle(mask, width, height)
     
     # At this stage the mask contains a B/W image of the circle, black outside, white inside.
     # Retain the alpha of the original in a separate image
@@ -44,7 +44,7 @@ class ImageVise::EllipseStencil
     # And perform the operation (set gray(RGB) of mask as the A of magick_image)
     magick_image.composite!(mask, Magick::CenterGravity, Magick::CopyOpacityCompositeOp)
   ensure
-    [mask, only_alpha, circle_img].each do |maybe_image|
+    [mask, only_alpha].each do |maybe_image|
       ImageVise.destroy(maybe_image)
     end
   end
@@ -58,6 +58,8 @@ class ImageVise::EllipseStencil
 
     gc = Magick::Draw.new
     gc.fill C_black
+    gc.rectangle(0, 0, width, height)
+    gc.fill C_white
     gc.ellipse(center_x, center_y, radius_width, radius_height, deg_start=0, deg_end=360)
     gc.draw(into_image)
   ensure

data/lib/image_vise/operators/force_jpg_out.rb

@@ -0,0 +1,17 @@
+# Forces the output format to be JPEG and specifies the quality factor to use when saving
+#
+# The corresponding Pipeline method is `force_jpg_out`.
+class ImageVise::ForceJPGOut < Ks.strict(:quality)
+  def initialize(quality:)
+    unless (0..100).cover?(quality)
+      raise ArgumentError, "the :quality setting must be within 0..100, but was %d" % quality
+    end
+    self.quality = quality
+  end
+
+  def apply!(_, metadata)
+    metadata[:writer] = ImageVise::JPGWriter.new(quality: quality)
+  end
+
+  ImageVise.add_operator 'force_jpg_out', self
+end

data/lib/image_vise/pipeline.rb

@@ -44,10 +44,20 @@ class ImageVise::Pipeline
     end
   end
 
-  def apply!(magick_image)
-    @ops.each{|e| e.apply!(magick_image) }
+  def apply!(magick_image, image_metadata)
+    @ops.each do |operator|
+      apply_operator_passing_metadata(magick_image, operator, image_metadata)
+    end
   end
-  
+
+  def apply_operator_passing_metadata(magick_image, operator, image_metadata)
+    if operator.method(:apply!).arity == 1
+      operator.apply!(magick_image)
+    else
+      operator.apply!(magick_image, image_metadata)
+    end
+  end
+
   def each(&b)
     @ops.each(&b)
   end

data/lib/image_vise/render_engine.rb

@@ -1,11 +1,11 @@
-class ImageVise::RenderEngine
+  class ImageVise::RenderEngine
   class UnsupportedInputFormat < StandardError; end
   class EmptyRender < StandardError; end
 
   DEFAULT_HEADERS = {
     'Allow' => "GET"
   }.freeze
-  
+
   # Headers for error responses that denote an invalid or
   # an unsatisfiable request
   JSON_ERROR_HEADERS_REQUEST = DEFAULT_HEADERS.merge({
@@ -19,7 +19,7 @@ class ImageVise::RenderEngine
     'Content-Type' => 'application/json',
     'Cache-Control' => 'public, max-age=5'
   }).freeze
-  
+
   # "public" of course. Add max-age so that there is _some_
   # revalidation after a time (otherwise some proxies treat it
   # as "must-revalidate" always), and "no-transform" so that
@@ -27,7 +27,7 @@ class ImageVise::RenderEngine
   # with Rack::Cache and leads Chrome to throw up on content
   # decoding for example).
   IMAGE_CACHE_CONTROL = 'public, no-transform, max-age=2592000'
-  
+
   # How long is a render (the ImageMagick/write part) is allowed to
   # take before we kill it
   RENDER_TIMEOUT_SECONDS = 10
@@ -35,16 +35,9 @@ class ImageVise::RenderEngine
   # Which input files we permit (based on extensions stored in MagicBytes)
   PERMITTED_SOURCE_FILE_EXTENSIONS = %w( gif png jpg psd tif)
 
-  # Which output files are permitted (regardless of the input format
-  # the processed images will be converted to one of these types)
-  PERMITTED_OUTPUT_FILE_EXTENSIONS = %W( gif png jpg)
-
   # How long should we wait when fetching the image from the external host
   EXTERNAL_IMAGE_FETCH_TIMEOUT_SECONDS = 4
-  
-  # The default file type for images with alpha
-  PNG_FILE_TYPE = MagicBytes::FileType.new('png','image/png').freeze
-  
+
   def bail(status, *errors_array)
     headers = if (300...500).cover?(status)
       JSON_ERROR_HEADERS_REQUEST.dup
@@ -54,7 +47,7 @@ class ImageVise::RenderEngine
     response = [status.to_i, headers, [JSON.pretty_generate({errors: errors_array})]]
     throw :__bail, response
   end
-  
+
   # The main entry point for the Rack app. Wraps a call to {#handle_request} in a `catch{}` block
   # so that any method can abort the request by calling {#bail}
   #
@@ -63,7 +56,7 @@ class ImageVise::RenderEngine
   def call(env)
     catch(:__bail) { handle_request(env) }
   end
-  
+
   # Hadles the Rack request. If one of the steps calls {#bail} the `:__bail` symbol will be
   # thrown and the execution will abort. Any errors will cause either an error response in
   # JSON format or an Exception will be raised (depending on the return value of `raise_exceptions?`)
@@ -80,9 +73,9 @@ class ImageVise::RenderEngine
     req = parse_env_into_request(env)
     bail(405, 'Only GET supported') unless req.get?
     params = extract_params_from_request(req)
-    
+
     image_request = ImageVise::ImageRequest.from_params(qs_params: params, secrets: ImageVise.secret_keys)
-    render_destination_file, render_file_type, etag = process_image_request(image_request) 
+    render_destination_file, render_file_type, etag = process_image_request(image_request)
     image_rack_response(render_destination_file, render_file_type, etag)
   rescue *permanent_failures => e
     handle_request_error(e)
@@ -97,7 +90,7 @@ class ImageVise::RenderEngine
       raise_exception_or_error_response(e, 500)
     end
   end
-  
+
   # Parses the Rack environment into a Rack::Reqest. The following methods
   # are going to be called on it: `#get?` and `#params`. You can use this
   # method to override path-to-parameter translation for example.
@@ -115,7 +108,7 @@ class ImageVise::RenderEngine
   def extract_params_from_request(rack_request)
     # Prevent cache bypass DOS attacks by only permitting :sig and :q
     bail(400, 'Query strings are not supported') if rack_request.params.any?
-    
+
     # Extract the tail (signature) and the front (the Base64-encoded request).
     # Slashes within :q are masked by ImageRequest already, so we don't have
     # to worry about them.
@@ -143,11 +136,11 @@ class ImageVise::RenderEngine
     # Compute an ETag which describes this image transform + image source location.
     # Assume the image URL contents does _never_ change.
     etag = image_request.cache_etag
-    
+
     # Download/copy the original into a Tempfile
     fetcher = ImageVise.fetcher_for(source_image_uri.scheme)
     source_file = fetcher.fetch_uri_to_tempfile(source_image_uri)
-    
+
     # Make sure we do not try to process something...questionable
     source_file_type = detect_file_type(source_file)
     unless source_file_type_permitted?(source_file_type)
@@ -156,15 +149,8 @@ class ImageVise::RenderEngine
 
     render_destination_file = Tempfile.new('imagevise-render').tap{|f| f.binmode }
 
-    # Perform the processing
-    if enable_forking?
-      require 'exceptional_fork'
-      ExceptionalFork.fork_and_wait do
-        apply_pipeline(source_file.path, pipeline, source_file_type, render_destination_file.path)
-      end
-    else
-      apply_pipeline(source_file.path, pipeline, source_file_type, render_destination_file.path)
-    end
+    # Do the actual imaging stuff
+    apply_pipeline(source_file.path, pipeline, source_file_type, render_destination_file.path)
 
     # Catch this one early
     render_destination_file.rewind
@@ -175,7 +161,7 @@ class ImageVise::RenderEngine
   ensure
     ImageVise.close_and_unlink(source_file)
   end
-  
+
   # Returns a Rack response triplet. Accepts the return value of
   # `process_image_request` unsplatted, and returns a triplet that
   # can be returned as a Rack response. The Rack response will contain
@@ -204,13 +190,13 @@ class ImageVise::RenderEngine
   # @param exception[Exception] the error that has to be captured
   # @param status_code[Fixnum] the HTTP status code
   def raise_exception_or_error_response(exception, status_code)
-    if raise_exceptions? 
+    if raise_exceptions?
       raise exception
     else
       bail status_code, exception.message
     end
   end
-  
+
   # Detects the file type of the given File and returns
   # a MagicBytes::FileType object that contains the extension and
   # the MIME type.
@@ -230,15 +216,6 @@ class ImageVise::RenderEngine
     PERMITTED_SOURCE_FILE_EXTENSIONS.include?(magic_bytes_file_info.ext)
   end
 
-  # Tells whether the given file type may be returned
-  # as the result of the render
-  #
-  # @param magic_bytes_file_info[MagicBytes::FileType] the filetype
-  # @return [Boolean]
-  def output_file_type_permitted?(magic_bytes_file_info)
-    PERMITTED_OUTPUT_FILE_EXTENSIONS.include?(magic_bytes_file_info.ext)
-  end
-
   # Lists exceptions that should lead to the request being flagged
   # as invalid (4xx as opposed to 5xx for a generic server error).
   # Decent clients should _not_ retry those requests.
@@ -249,7 +226,7 @@ class ImageVise::RenderEngine
       ImageVise::ImageRequest::InvalidRequest
     ]
   end
-  
+
   # Is meant to be overridden by subclasses,
   # will be called at the start of each request to set up the error handling
   # library (Appsignal, Honeybadger, Sentry...)
@@ -278,7 +255,7 @@ class ImageVise::RenderEngine
   # @return [void]
   def handle_generic_error(exception)
   end
-  
+
   # Tells whether the engine must raise the exceptions further up the Rack stack,
   # or they should be suppressed and a JSON response must be returned.
   #
@@ -286,14 +263,7 @@ class ImageVise::RenderEngine
   def raise_exceptions?
     false
   end
-  
-  # Tells whether image processing in a forked subproces should be turned on
-  #
-  # @return [Boolean]
-  def enable_forking?
-    ENV['IMAGE_VISE_ENABLE_FORK'] == 'yes'
-  end
-  
+
   # Applies the given {ImageVise::Pipeline} to the image, and writes the render to
   # the given path.
   #
@@ -303,21 +273,23 @@ class ImageVise::RenderEngine
   # @return [void]
   def apply_pipeline(source_file_path, pipeline, source_file_type, render_to_path)
     render_file_type = source_file_type
-    
+
     # Load the first frame of the animated GIF _or_ the blended compatibility layer from Photoshop
     image_list = Magick::Image.read(source_file_path)
-    magick_image = image_list.first
-
-    # Apply the pipeline
-    pipeline.apply!(magick_image)
-
-    # If processing the image has created an alpha channel, use PNG always.
-    # Otherwise, keep the original format for as far as the supported formats list goes.
-    render_file_type = PNG_FILE_TYPE if magick_image.alpha?
-    render_file_type = PNG_FILE_TYPE unless output_file_type_permitted?(render_file_type)
-    
-    magick_image.format = render_file_type.ext
-    magick_image.write(render_to_path)
+    magick_image = image_list.first # Picks up the "precomp" PSD layer in compatibility mode, or the first frame of a GIF
+
+    # If any operators want to stash some data for downstream use we use this Hash
+    metadata = {}
+
+    # Apply the pipeline (all the image operators)
+    pipeline.apply!(magick_image, metadata)
+
+    # Write out the file honoring the possible injected metadata. One of the metadata
+    # elements (that an operator might want to alter) is the :writer, we forcibly #fetch
+    # it so that we get a KeyError if some operator has deleted it without providing a replacement.
+    # If no operators touched the writer we are going to use the automatic format selection
+    writer = metadata.fetch(:writer, ImageVise::AutoWriter.new)
+    writer.write_image!(magick_image, metadata, render_to_path)
   ensure
     # destroy all the loaded images explicitly
     (image_list || []).map {|img| ImageVise.destroy(img) }

data/lib/image_vise/version.rb

@@ -0,0 +1,3 @@
+class ImageVise
+  VERSION = '0.2.0'
+end