rack_resize 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 89ff24cb927e939e377fa9de0e4af8d736e026d2536a5721a94c8884c745de16
-  data.tar.gz: 13e4ca927f5ef1f058a528ded9158cd324e2eaf0e34c2797b21e1791028a31fc
+  metadata.gz: 48043ad5846a54c36958ce1d2b4648de14b01c30db46ec8dcdb2b601628118f2
+  data.tar.gz: 2ccd0c5703c104cb952bd4e299fc6e1be49ef2e08849f2a9637d661adca13a22
 SHA512:
-  metadata.gz: 58f951cd5354359a5a6261a739ded0e7136d346a91572c4bc8ba856530feb57644b1825959acc4ecdc79f91a15ab93e1e38c58d3578d82bd4c803a70ac015fdc
-  data.tar.gz: 462e57a07904d0e4c845b40611a2d7bc601e4c2fb207b8b7f2cccac9de934d6471bc222550e770ba19e0e58235d977d86b4ed8df0771761ac4b09a884495f106
+  metadata.gz: 9fa7b2853a0701bf476f9730d1aae31ffc9edb28f53d032e8d51acb3c7e82fdd07e64e2225d4195507bfde69956b44bc48152d9d4b1c4e2fa840fe140d592518
+  data.tar.gz: e0b76b4d16323c0a87bae2d8c61fa2d44d0a72d1d27c6d8872bd091084ab2a5f9055216a48af9724af5ccff8dba6cf782acf8f7a38ca0216039318a87268c479

data/.github/workflows/test.yml

@@ -0,0 +1,29 @@
+name: Test
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: ['3.3', '3.4', '4.0']
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install package
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y libvips42 imagemagick libimlib2 libimlib2-dev
+
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+
+      - run: bundle exec rake test

data/.gitignore

@@ -1,3 +1,4 @@
 .idea/
 Gemfile.lock
 .DS_Store
+*.gem

data/.ruby-version

@@ -0,0 +1 @@
+4.0.5

data/Gemfile

@@ -2,6 +2,13 @@ source 'https://rubygems.org'
 
 gemspec
 
+gem 'image_processing'
 gem 'mini_magick'
 gem 'ruby-vips'
 gem 'rszr'
+
+gem 'railties'
+
+gem 'minitest'
+gem 'minitest-reporters'
+gem 'rake'

data/README.md

@@ -1,6 +1,108 @@
+[![Gem Version](https://badge.fury.io/rb/rack_resize.svg)](https://badge.fury.io/rb/rack_resize)
 
-RESIZE IMAGE mini_magick samples/big.jpg - {:width=>"150"} - 0.949s
-RESIZE IMAGE sips samples/big.jpg - {:width=>"150"} - 0.121s
-RESIZE IMAGE vips samples/big.jpg - {:width=>"150"} - 0.116s
+Designed to be local simulation of CDN's resize feature
 
-s.add_runtime_dependency "image_processing", ["> 1.0", "< 3.0"]
+How to use with Rails
+
+```ruby
+group :development do
+  gem "rack_resize", require: "rack_resize/rails_autoload"
+end
+```
+
+How to use with Rack
+
+```ruby
+use RackResize::RackApp, processor: :imlib2, assets_folder: "samples"
+use Rack::Static, urls: [""], root: "samples", index: "index.html" # optional
+```
+
+### URL Formats:
+
+Cloudflare format: (can be used with helpers from `carrierwave-cloudflare` gem)
+```
+/cdn-cgi/image/width=426,format=auto/assets/pets/dog.jpg
+```
+Fastly and bunny.net: (tbd)
+```
+/assets/pets/dog.jpg?width=300
+```
+
+### Configuration:
+
+```ruby
+RackResize.configure do |config|
+  config.assets_folders     = { "assets" => Rails.root.join('app', 'assets', 'images') }
+  config.processor          = :sips / :vips / :mini_magick / :imlib2
+  config.default_quality    = 95
+  config.save_resized       = false
+  config.cache_folder       = Rails.root.join('tmp', 'rack_resize_cache') # used if save_resized enabled
+  config.http_cache_max_age = 86400
+end
+```
+
+### Supported Processing Backends:
+
+<table>
+<tr>
+<th>libaray</th>
+<th>dependency</th>
+<th>config</th>
+</tr>
+<tr>
+<td>mini_magick</td>
+<td>
+
+```ruby
+gem "image_processing"
+gem  "mini_magick"
+```
+
+</td>
+<td>
+
+`processor: :mini_magick`
+
+</td>
+</tr>
+<tr>
+<td>vips</td>
+<td>
+
+```ruby
+gem "image_processing"
+gem  "ruby-vips"
+```
+
+</td>
+<td>
+
+`processor: :vips`
+
+</td>
+</tr>
+<tr>
+<td>sips (MacOS only)</td>
+<td>none</td>
+<td>
+
+`processor: :sips`
+
+</td>
+</tr>
+<tr>
+<td>imlib2</td>
+<td>
+
+```ruby
+gem  "rszr"
+```
+
+</td>
+<td>
+
+`processor: :imlib2`
+
+</td>
+</tr>
+</table>

data/Rakefile

@@ -0,0 +1,9 @@
+require 'rake/testtask'
+
+Rake::TestTask.new(:test) do |t|
+  t.libs << 'test'
+  t.pattern = 'test/**/*_test.rb'
+  t.verbose = true
+end
+
+task default: :test

data/config.ru

@@ -1,7 +1,13 @@
+#
+# usage:
+#   gem install rackup
+#   rackup
+#
+
 require_relative "lib/rack_resize"
 require "rack/static"
 
-use RackResize::RackApp, processor: :imlib2, assets_folder: "samples" # save_resized: false,
+use RackResize::RackApp, processor: :imlib2, assets_folder: "samples"
 
 use Rack::Static, urls: [""], root: "samples", index: "index.html"
 

data/lib/rack_resize/configuration.rb

@@ -3,21 +3,33 @@ require 'pathname'
 class RackResize::Configuration
   PROCESSORS = %i[sips vips mini_magick imlib2].freeze
 
-  attr_reader :processor
-  attr_accessor :save_resized, :default_quality, :cache_folder, :assets_folder
+  attr_reader :processor, :assets_folders
+  attr_accessor :save_resized, :default_quality, :cache_folder, :http_cache_max_age, :max_dimension, :logger
 
   alias_method :save_resized?, :save_resized
 
   def initialize(options = {})
-    @processor       = options[:processor] || RUBY_PLATFORM.include?('darwin') ? :sips : :mini_magick
-    @save_resized    = options.key?(:save_resized) ? options[:save_resized] : true
-    @default_quality = options[:default_quality] || 95
-    @cache_folder    = options[:cache_folder]
-    @assets_folder   = options[:assets_folder] ? Pathname.new(options[:assets_folder]) : nil
+    @processor          = options[:processor] || RUBY_PLATFORM.include?('darwin') ? :sips : :mini_magick
+    @save_resized       = options.key?(:save_resized) ? options[:save_resized] : false
+    @default_quality    = options[:default_quality] || 95
+    @cache_folder       = options[:cache_folder]
+    @http_cache_max_age = options[:http_cache_max_age] || 86400 # 1 day
+    @max_dimension      = options[:max_dimension] || 4000
+    @logger             = options[:logger]
 
-    if defined?(Rails)
+    self.assets_folders = options[:assets_folders] if options[:assets_folders]
+
+    if defined?(Rails) && Rails.respond_to?(:root) && Rails.root
       @cache_folder ||= Rails.root.join('tmp', 'rack_resize_cache')
-      @assets_folder ||= Rails.root.join('app', 'assets', 'images')
+      unless options.key?(:assets_folders)
+        self.assets_folders = {
+          "assets"  => Rails.root.join("app/assets/images"),
+          "uploads" => Rails.root.join("public/uploads"),
+        }
+      end
+      unless options.key?(:save_resized)
+        @save_resized = true
+      end
     end
 
     if options[:processor]
@@ -25,6 +37,17 @@ class RackResize::Configuration
     end
   end
 
+  def assets_folders=(values)
+    if values.is_a?(Array)
+      @assets_folders = values.map { |v| [format_asset_path(v), Pathname.new(v)] }.to_h
+    elsif values.is_a?(Hash)
+      @assets_folders = values.transform_values { |v| v.is_a?(Pathname) ? v : Pathname.new(v) }
+                              .transform_keys {|k| format_asset_path(k) }
+    else
+      raise ArgumentError, "RackResize::Configuration.assets_folders can be either hash or array, received #{values.class}"
+    end
+  end
+
   def processor=(value)
     value = value.to_sym
     unless PROCESSORS.include?(value)
@@ -36,7 +59,7 @@ class RackResize::Configuration
 
   def processor_instance
     @processor_instance ||= case @processor
-                            when :mini_magick then RackResize::MiniMagick.new
+                            when :mini_magick then RackResize::Processors::MiniMagick.new
                             when :vips        then RackResize::Processors::Vips.new
                             when :sips        then RackResize::Processors::Sips.new
                             when :imlib2      then RackResize::Processors::Imlib2.new
@@ -44,4 +67,10 @@ class RackResize::Configuration
                               raise "RackResize - Unknow image processor #{@processor.inspect}"
                             end
   end
+
+  private
+
+  def format_asset_path(path)
+    path.start_with?('/') ? path.to_s : "/#{path}"
+  end
 end

data/lib/rack_resize/input_parsers/cloudflare.rb

@@ -1,4 +1,22 @@
-# frozen_string_literal: true
+module RackResize::InputParsers::Cloudflare
+  extend self
 
-class Cloudflare
+  # process string like this
+  # /cdn-cgi/image/width=426,format=auto/assets/templates/vancouver-27c47f55.jpg
+
+  def parse_input(fullpath, cf_path_prefix:)
+    unless fullpath.start_with?(cf_path_prefix)
+      return { route_matched: false, req_params: nil, asset_path: nil }
+    end
+
+    fullpath = fullpath.delete_prefix("/cdn-cgi").delete_prefix("/image").delete_prefix("/")
+    file_path_match = fullpath.match(%r{(?<params>[^\/]+)(?<file>\/.+?)(-[\da-f]{8})?(?<ext>\.\w{2,})$})
+
+    return { route_matched: true, req_params: nil, asset_path: nil } unless file_path_match
+
+    req_params = file_path_match[:params].split(",").map {|s| s.split("=") }.to_h.transform_keys(&:to_sym)
+    asset_path = "#{file_path_match[:file]}#{file_path_match[:ext]}"
+
+    { route_matched: true, req_params:, asset_path: }
+  end
 end

data/lib/rack_resize/processing.rb

@@ -1,3 +1,5 @@
+require 'digest'
+
 class RackResize::Processing
   attr_reader :config
 
@@ -24,8 +26,11 @@ class RackResize::Processing
 
   def process_file(source_file:, req_params:, target_file: nil)
     dpr = req_params[:dpr]&.to_f || 1.0
-    target_width = (req_params[:w]&.to_i || req_params[:width]&.to_i)&.*(dpr)
-    target_height = (req_params[:h]&.to_i || req_params[:height]&.to_i)&.*(dpr)
+    dpr = [[dpr, 0.1].max, 10.0].min
+
+    max = config.max_dimension.to_f
+    target_width  = (req_params[:w]&.to_i || req_params[:width]&.to_i)&.*(dpr)&.clamp(1, max)
+    target_height = (req_params[:h]&.to_i || req_params[:height]&.to_i)&.*(dpr)&.clamp(1, max)
 
     start_time = Time.now
     begin
@@ -37,13 +42,8 @@ class RackResize::Processing
   end
 
   def logger
-    @logger ||= begin
-                  if defined?(Rails) && Rails.logger
-                    Rails.logger
-                  else
-                    require 'logger'
-                    Logger.new(STDOUT)
-                  end
-                end
+    config.logger ||
+      (defined?(Rails) && Rails.respond_to?(:logger) && Rails.logger) ||
+      (require('logger') && Logger.new(STDOUT))
   end
-end
+end

data/lib/rack_resize/processors/imlib2.rb

@@ -11,7 +11,6 @@ class RackResize::Processors::Imlib2
     image.resize!(target_width || :auto, target_height || :auto) if target_width || target_height
 
     if target_file
-      image.call(destination: target_file)
       image.save(target_file)
     else
       image.save_data(format: source_file.to_s =~ /\.png$/ ? :png : :jpeg)

data/lib/rack_resize/processors/mini_magick.rb

@@ -45,9 +45,3 @@ class RackResize::Processors::MiniMagick
     image&.destroy!
   end
 end
-
-# magick /Users/pavel/Work/resume-io/app/assets/images/templates/vancouver.jpg -auto-orient -resize 426.0x> -quality 95 /Users/pavel/Work/resume-io/tmp/cdn_cgi_cache/60f2727cce350874040b321219f05766.jpg
-
-
-# magick mogrify -resize 426.0x> /var/folders/n9/b__lxh7j1_xcwg4k_8j2_39m0000gp/T/mini_magick20260709-58478-qowydk.jpg
-# magick mogrify -quality 95 /var/folders/n9/b__lxh7j1_xcwg4k_8j2_39m0000gp/T/mini_magick20260709-58478-qowydk.jpg

data/lib/rack_resize/processors/sips.rb

@@ -1,37 +1,41 @@
-require "shellwords"
+require "open3"
 
 class RackResize::Processors::Sips
 
   def resize(source_file:, target_width:, target_height:, target_file: nil)
-    # args = ["sips", "--deleteColorManagementProperties", "--optimizeColorForSharing"]
-    args = ["sips", "--deleteColorManagementProperties", "--debug"]
-    args += ["-s formatOptions", RackResize.config.default_quality]
-    args << "--resampleWidth" << target_width.to_i if target_width
-    args << "--resampleHeight" << target_height.to_i if target_height
-
-    if target_file
-      args << "-o" << Shellwords.escape(target_file)
+    args = %w[sips --deleteColorManagementProperties]
+    args += ["-s", "formatOptions", RackResize.config.default_quality.to_s]
+
+    if target_width && target_height
+      info, status = Open3.capture2("sips", "-g", "pixelWidth", "-g", "pixelHeight", source_file.to_s)
+      raise "sips failed to read dimensions for #{source_file}" unless status.success?
+      src_w = info[/pixelWidth: (\d+)/, 1]&.to_f
+      src_h = info[/pixelHeight: (\d+)/, 1]&.to_f
+      if src_w && src_h && (target_width.to_f / src_w) <= (target_height.to_f / src_h)
+        args += ["--resampleWidth", target_width.to_i.to_s]
+      else
+        args += ["--resampleHeight", target_height.to_i.to_s]
+      end
     else
-      tmp_file = Tempfile.new(["result", File.extname(source_file)])
-      args << "-o" << tmp_file.path
+      args += ["--resampleWidth",  target_width.to_i.to_s]  if target_width
+      args += ["--resampleHeight", target_height.to_i.to_s] if target_height
     end
 
-    args << Shellwords.escape(source_file)
-
-    pp [:args, args, args.join(" ")]
-
-    result = `#{args.join(" ")}`
-
-    puts "sips command result: #{result}"
-
-    unless target_file
-      begin
-        return File.open(tmp_file.path, 'rb', &:read)
-      ensure
-        tmp_file.unlink
-      end
+    if target_file
+      _, status = Open3.capture2(*args, "-o", target_file.to_s, source_file.to_s)
+      raise "sips failed (exit #{status.exitstatus}) for #{source_file}" unless status.success?
+      return nil
     end
 
-    nil
+    tmp = Tempfile.new(["result", File.extname(source_file)])
+    tmp_path = tmp.path || raise("tempfile has no path")
+    begin
+      _, status = Open3.capture2(*args, "-o", tmp_path, source_file.to_s)
+      raise "sips failed (exit #{status.exitstatus}) for #{source_file}" unless status.success?
+      File.binread(tmp_path)
+    ensure
+      tmp.close
+      tmp.unlink
+    end
   end
 end

data/lib/rack_resize/processors/vips.rb

@@ -11,8 +11,6 @@ class RackResize::Processors::Vips
     image = image.resize_to_limit(target_width, target_height) if target_width || target_height
     image = image.saver(quality: RackResize.config.default_quality)
 
-    # image.call(destination: target_file)
-
     if target_file
       image.call(destination: target_file)
     else

data/lib/rack_resize/rack_app.rb

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 require 'rack'
 
 class RackResize::RackApp
@@ -13,40 +11,45 @@ class RackResize::RackApp
 
     @config = options == {} ? RackResize.config : RackResize::Configuration.new(options)
     @processing = RackResize::Processing.new(config: @config)
-    @path_prefix = options[:path_prefix] || "/cdn-cgi/image"
+    @cf_path_prefix = options[:cf_path_prefix] || "/cdn-cgi/image"
   end
 
   def call(env)
-    # process string like this
-    # /cdn-cgi/image/width=426,format=auto/assets/templates/vancouver-27c47f55.jpg
-
     request = Rack::Request.new(env)
     fullpath = request.path_info
 
-    unless fullpath.start_with?(@path_prefix)
-      return @app.call(env)
-    end
+    RackResize::InputParsers::Cloudflare.parse_input(fullpath, cf_path_prefix: @cf_path_prefix) =>
+      {route_matched:, req_params:, asset_path:}
 
-    fullpath = fullpath.delete_prefix("/cdn-cgi").delete_prefix("/image").delete_prefix("/")
-    file_path_match = fullpath.match(%r{(?<params>[^\/]+)(?<file>\/.+?)(-[\da-f]{8})?(?<ext>\.\w{2,})$})
+    return @app.call(env) unless route_matched
+    return error_resp("can't parse file path") unless asset_path
 
-    return error_resp("can't parse file path") unless file_path_match
+    return error_resp("no assets folders configured") if config.assets_folders.nil? || config.assets_folders.empty?
 
-    req_params = file_path_match[:params].split(",").map {|s| s.split("=") }.to_h.transform_keys(&:to_sym)
-    asset_path = "#{file_path_match[:file]}#{file_path_match[:ext]}"
-
-    if defined?(Rails)
-      asset_path = asset_path.delete_prefix("/assets/")
-    else
-      asset_path = asset_path.delete_prefix("/#{config.assets_folder}/")
+    has_matched = false
+    asset_file = nil
+    config.assets_folders.each do |prefix, folder|
+      if asset_path.start_with?(prefix)
+        asset_file = folder.join(asset_path.delete_prefix(prefix + (prefix.end_with?("/") ? "" : "/")))
+        if asset_file.expand_path.to_s.start_with?(folder.to_s)
+          has_matched = true
+        end
+        break
+      end
     end
 
-    asset_file = config.assets_folder.join(asset_path.sub(%r{^/assets/}, ''))
-
-    return error_resp(".. is not allowed in image path") if asset_path.include?("..")
-
-    return error_resp("invalid file path") unless asset_file.to_s.start_with?(config.assets_folder.to_s)
-    return error_resp("file not exists on a server") unless asset_file.exist?
+    if asset_path.include?("..")
+      @processing.logger.info("RackResize::RackApp - File path has invalid byte sequence #{asset_path}")
+      return error_resp(".. is not allowed in image path")
+    end
+    unless has_matched
+      @processing.logger.info("RackResize::RackApp - requested file #{asset_path} not match any of configured assets folder #{config.assets_folders.keys}")
+      return error_resp("invalid file path")
+    end
+    unless asset_file.exist?
+      @processing.logger.info("RackResize::RackApp - File path fond #{asset_path} => #{asset_file}")
+      return error_resp("file not exists on a server")
+    end
 
     file_content = @processing.process!(source_file: asset_file, req_params:)
     return send_file(asset_file:, file_content:)
@@ -58,7 +61,6 @@ class RackResize::RackApp
 
   def send_file(asset_file:, file_content: nil)
     content_type = Rack::Mime.mime_type(File.extname(asset_file), "application/octet-stream")
-    # file = file_path ? File.open(file_path, "rb") : StringIO.new(file_content)
 
     [
       200,
@@ -66,7 +68,7 @@ class RackResize::RackApp
         "content-type"        => content_type,
         "content-length"      => file_content.size.to_s,
         "content-disposition" => "inline",
-        "cache-control"       => "max-age=86400" # 1 day
+        "cache-control"       => "max-age=#{config.http_cache_max_age}" # 1 day
       },
       file_content
     ]

data/lib/rack_resize/rails_autoload.rb

@@ -1,7 +1,7 @@
 require "rack_resize"
 require "rails"
 
-module MyGem
+module RackResize
   class Railtie < Rails::Railtie
     initializer "rack_resize.auto_register_itself" do |app|
       app.config.middleware.insert_before 0, RackResize::RackApp

data/lib/rack_resize.rb

@@ -7,10 +7,14 @@ module RackResize
   module Processors
     autoload :Sips,      "#{__dir__}/rack_resize/processors/sips"
     autoload :Vips,      "#{__dir__}/rack_resize/processors/vips"
-    autoload :MiniMagic, "#{__dir__}/rack_resize/processors/mini_magic"
+    autoload :MiniMagick, "#{__dir__}/rack_resize/processors/mini_magick"
     autoload :Imlib2,    "#{__dir__}/rack_resize/processors/imlib2"
   end
 
+  module InputParsers
+    autoload :Cloudflare, "#{__dir__}/rack_resize/input_parsers/cloudflare"
+  end
+
   class << self
     def configuration
       @configuration ||= Configuration.new

data/rack_resize.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name        = "rack_resize"
-  s.version     = "0.1.0"
+  s.version     = "0.1.2"
   s.author      = ["Pavel Evstigneev"]
   s.email       = ["pavel.evst@gmail.com"]
   s.homepage    = "https://github.com/paxa/rack_resize"
@@ -8,7 +8,7 @@ Gem::Specification.new do |s|
   s.license     = 'MIT'
   s.required_ruby_version = ['>= 3.0']
 
-  s.files       = `git ls-files`.split("\n")
+  s.files       = `git ls-files`.split("\n").delete_if{|f| f.start_with?('samples', 'test') }
   s.test_files  = []
 
   s.require_paths = ["lib"]

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: rack_resize
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
 platform: ruby
 authors:
 - Pavel Evstigneev
 bindir: bin
 cert_chain: []
-date: 2026-07-15 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -35,9 +35,12 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/test.yml"
 - ".gitignore"
+- ".ruby-version"
 - Gemfile
 - README.md
+- Rakefile
 - config.ru
 - lib/rack_resize.rb
 - lib/rack_resize/configuration.rb
@@ -68,7 +71,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.5
+rubygems_version: 4.0.10
 specification_version: 4
 summary: Image resizing on a fly
 test_files: []