dragonfly-cache 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 69db42bcdd7613f8f7738bec974e989b3863d508b24757fff67b4e003b6b5a04
-  data.tar.gz: 5b48ec8599fb6aed358fb281d6f4c137aa75c427cc22554557cf61f99e0ee9f1
+  metadata.gz: d9c2bd6a296538d40627ac2d72d59b22899b939f058df6733c536cb8ccb31154
+  data.tar.gz: c42cb333f27257b8b2522433dc1a303768eb8eca01c829793e7e30ac1e51aa23
 SHA512:
-  metadata.gz: 789d49580b89f059a1086466fc22975e71bcc8cd435467efe4000dc3b1da7609187442f1514687b42df1d5ba1bd5f6543b034a6b5cb046f36a9ff3bcb1b6092c
-  data.tar.gz: cf0a7cd0b9d29a7470ea057e085c2f92cb495bf74cb87cff5d6151776d51db06f1d8b44058a7d8919eba2b0cd261ba363a4c407ac396924dfa242c102b57aacd
+  metadata.gz: 405297344133ee35cb4547dd1d7499020cb77f751e3e81e23cf2378a77e6e6d5cb83e91fc11e60d105cd131bf54d67364bf197ab244299e700ba8f41ea4e7b75
+  data.tar.gz: fbbf31316483fd3721015225e25145967ad0dc2a929b68e042d5a0a064ca23958eba53c1328aace339dce28aa7b926e430c44e978722541f12b6743f96fa1fa4

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    dragonfly-cache (0.1.2)
+    dragonfly-cache (0.1.3)
       dragonfly (~> 1.1.5)
       i18n
 
@@ -42,7 +42,7 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.8.0)
     rspec-support (3.8.0)
-    rubocop (0.58.2)
+    rubocop (0.59.0)
       jaro_winkler (~> 1.5.1)
       parallel (~> 1.10)
       parser (>= 2.5, != 2.5.1.1)

data/README.md

@@ -43,3 +43,7 @@ Configured as this, cached files will be stored in `/public/media-cache/:sha/:na
 ## How does it work?
 
 `dragonfly-cache` use a method similar to [the one described in the Dragonfly documentation to process files on-the-fly and serve them remotely](http://markevans.github.io/dragonfly/cache#processing-on-the-fly-and-serving-remotely) but use only `define_url`. 
+
+## Credits
+
+The dragonfly photography used in specs is a work from [André Karwath](https://commons.wikimedia.org/wiki/User:Aka) and is licensed under the [Creative Commons Attribution-Share Alike 2.5 Generic](https://creativecommons.org/licenses/by-sa/2.5/deed.en) license.

data/lib/dragonfly/cache/config.rb

@@ -15,6 +15,13 @@ module Dragonfly
         rewrite_url_format!
       end
 
+      def base_dir
+        @base_dir ||= begin
+          path_format = File.join(servers_options[:server_root], servers_options[:url_format])
+          path_format.split('/').take_while { |p| p != ':shaish' }.join('/')
+        end
+      end
+
       protected
 
       def validate!

data/lib/dragonfly/cache/manager.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
-require 'dragonfly/cache/storage'
+require 'dragonfly/cache/mapper/yaml'
+require 'dragonfly/cache/storage/local'
 
 module Dragonfly
   module Cache
@@ -10,79 +11,65 @@ module Dragonfly
       MIN_SHA_SIZE = 2
       MAX_SHA_SIZE = 16 # Length of SHA identifier generated by Dragonfly
 
-      attr_reader :plugin, :storage, :sha_size
+      attr_reader :plugin, :storage, :map
 
-      delegate %i[servers_options] => :plugin
-      delegate %i[write writen? current_max_sha_size base_dir] => :storage
+      delegate %i[config] => :plugin
 
       def initialize(plugin)
-        @plugin   = plugin
-        @storage  = Dragonfly::Cache::Storage.new(self)
-        initialize!
+        @plugin = plugin
+        @map = Dragonfly::Cache::Mapper::Yaml.new(config.servers_options)
+        @storage = Dragonfly::Cache::Storage::Local.new(config.servers_options)
       end
 
-      def valid_uri?(job, uri)
-        return true unless wrong_key?(job, uri) || wrong_value?(job, uri)
-        increase_sha_size!
-        false
-      end
-
-      def store(job, uri)
-        write(job, uri) unless writen?(job, uri)
-        map(job, uri)   unless mapped?(job, uri)
-      rescue Dragonfly::Cache::Error => e
-        Dragonfly.warn(e.message)
-      end
+      def cache(job)
+        return @map[job.sha] if @map.key?(job.sha)
 
-      protected
-
-      def wrong_key?(job, uri)
-        @cache_map.value?(uri.path) && @cache_map.key(uri.path) != job.sha
+        store(job, yield)
       end
 
-      def wrong_value?(job, uri)
-        @cache_map.key?(job.sha) && @cache_map[job.sha] != uri.path
+      def valid?(job, uri)
+        valid = (@map.key?(job.sha) && @map[job.sha] == uri) || !@map.key?(job.sha)
+        increase_sha_size! unless valid
+        valid
       end
 
-      def initialize!
-        detect_sha_size
-        load_map
-      rescue ::StandardError => e
-        raise Dragonfly::Cache::Error, e.message
+      def job_options(job)
+        {
+          shaish: shaish(job),
+          normalized_name: normalized_name(job)
+        }
       end
 
-      def detect_sha_size
-        @sha_size = [current_max_sha_size, MIN_SHA_SIZE].compact.max
-        @sha_size = [@sha_size, MAX_SHA_SIZE].min
-      end
-
-      def increase_sha_size!
-        raise Error, "Can't build longer :sha identifier" if @sha_size == MAX_SHA_SIZE
-        @sha_size += 1
+      protected
 
-        @cache_map.clear
-        save_map
+      def shaish(job)
+        job.sha[0..(sha_size - 1)]
       end
 
-      def map_path
-        @map_path ||= File.join(base_dir, 'map.yml')
+      def normalized_name(job)
+        basename = job.basename || job.signature
+        sanitized = basename.gsub(/[[:space:][:punct:][:cntrl:]]/, ' ').squeeze(' ').strip
+        transliterated = I18n.transliterate(sanitized, replacement: ' ').squeeze(' ').strip
+        downcased = (transliterated.empty? ? job.signature : transliterated).downcase.tr(' ', '-')
+        [downcased, job.ext].compact.join('.')
       end
 
-      def load_map
-        @cache_map = File.size?(map_path) ? YAML.load_file(map_path) : {}
+      def sha_size
+        @sha_size ||= [[storage.sha_size, MIN_SHA_SIZE].compact.max, MAX_SHA_SIZE].min
       end
 
-      def save_map
-        File.open(map_path, 'wb') { |f| YAML.dump(@cache_map, f) }
-      end
+      def increase_sha_size!
+        raise Error, "Can't build longer :sha identifier" if @sha_size == MAX_SHA_SIZE
 
-      def map(job, uri)
-        @cache_map[job.sha] = uri.path
-        save_map
+        @sha_size += 1
       end
 
-      def mapped?(job, _uri)
-        @cache_map.key?(job.sha)
+      def store(job, url)
+        storage.store(url, job)
+        map.store(job.sha, url)
+        url
+      rescue Dragonfly::Cache::Error => e
+        Dragonfly.warn(e.message)
       end
     end
   end

data/lib/dragonfly/cache/mapper/yaml.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require 'yaml'
+
+module Dragonfly
+  module Cache
+    module Mapper
+      class Yaml
+        extend Forwardable
+
+        attr_reader :root, :internal
+        delegate %i[[] key? value? keys values] => :internal
+
+        def initialize(config)
+          @root = config[:server_root]
+          @internal = {}
+        end
+
+        def store(key, value)
+          @internal[key] = value
+          save!
+        end
+
+        alias []= store
+
+        protected
+
+        def path
+          @path ||= File.join(root, 'map.yml')
+        end
+
+        def load!
+          @internal = File.size?(path) ? YAML.load_file(path) : {}
+        end
+
+        def save!
+          File.open(path, 'wb') { |f| YAML.dump(@internal, f) }
+        end
+      end
+    end
+  end
+end

data/lib/dragonfly/cache/plugin.rb

@@ -17,12 +17,11 @@ module Dragonfly
 
       attr_reader :config, :manager
 
-      delegate %i[servers_options] => :config
-      delegate %i[sha_size valid_uri? store] => :manager
+      delegate %i[cache valid? job_options] => :manager
 
       def call(app, cache_servers_options = {})
-        @config   = Dragonfly::Cache::Config.new(cache_servers_options)
-        @manager  = Dragonfly::Cache::Manager.new(self)
+        @config = Dragonfly::Cache::Config.new(cache_servers_options)
+        @manager = Dragonfly::Cache::Manager.new(self)
 
         app.define_url do |app, job, opts|
           url_for(app, job, opts)
@@ -32,41 +31,24 @@ module Dragonfly
         end
       end
 
-      protected
-
       def url_for(app, job, opts)
-        uri = find_valid_url_for(app, job, opts)
-        # File are stored on url building instead of in a before_serve block to allow use of assets host
-        store(job, uri)
-        uri.to_s
+        cache(job) { build_url_for(app, job, opts) }
       end
 
-      def find_valid_url_for(app, job, opts)
+      protected
+
+      def build_url_for(app, job, opts)
         loop do
-          url = server_for(app).url_for(job, options_for(job).merge(opts))
-          uri = URI.parse(url)
-          uri.query = nil
-          uri.fragment = nil
-          return uri if valid_uri?(job, uri)
+          url = server_for(app).url_for(job, job_options(job).merge(opts))
+          path = URI.parse(url).path
+          return path if valid?(job, path)
         end
       end
 
-      def options_for(job)
-        basename = job.basename || job.signature
-        sanitized = basename.gsub(/[[:space:][:punct:][:cntrl:]]/, ' ').squeeze(' ').strip
-        transliterated = I18n.transliterate(sanitized, replacement: ' ').squeeze(' ').strip
-        downcased = (transliterated.empty? ? job.signature : transliterated).downcase.tr(' ', '-')
-
-        {
-          shaish: job.sha[0..(sha_size - 1)],
-          normalized_name: [downcased, job.ext].join('.')
-        }
-      end
-
       def server_for(app)
         @@servers[app.name] ||= begin
           server = app.server.dup
-          servers_options.each do |name, value|
+          config.servers_options.each do |name, value|
             server.send("#{name}=", value) if server.respond_to?("#{name}=")
           end
           server

data/lib/dragonfly/cache/storage/local.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Dragonfly
+  module Cache
+    module Storage
+      class Local
+        attr_reader :root, :format
+
+        def initialize(config)
+          @root = config[:server_root]
+          @format = config[:url_format]
+        end
+
+        def store(url, job)
+          job.to_file(path(url), mode: 0o644)
+        end
+
+        def sha_size
+          longest = Dir["#{base_dir}/*"].select { |p| File.directory?(p) }.max { |a, b| a <=> b }
+          (longest ? File.basename(longest).size : nil)
+        end
+
+        protected
+
+        def path(url)
+          File.join(root, url)
+        end
+
+        def base_dir
+          @base_dir ||= begin
+            path_format = File.join(root, format)
+            path_format.split('/').take_while { |p| p != ':shaish' }.join('/')
+          end
+        end
+      end
+    end
+  end
+end

data/lib/dragonfly/cache/version.rb

@@ -2,6 +2,6 @@
 
 module Dragonfly
   module Cache
-    VERSION = '0.1.2'
+    VERSION = '0.1.3'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dragonfly-cache
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Gaël-Ian Havard
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-09-07 00:00:00.000000000 Z
+date: 2018-09-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dragonfly
@@ -113,8 +113,9 @@ files:
 - lib/dragonfly/cache.rb
 - lib/dragonfly/cache/config.rb
 - lib/dragonfly/cache/manager.rb
+- lib/dragonfly/cache/mapper/yaml.rb
 - lib/dragonfly/cache/plugin.rb
-- lib/dragonfly/cache/storage.rb
+- lib/dragonfly/cache/storage/local.rb
 - lib/dragonfly/cache/version.rb
 homepage: https://github.com/notus-sh/dragonfly-cache
 licenses:

data/lib/dragonfly/cache/storage.rb

@@ -1,68 +0,0 @@
-# frozen_string_literal: true
-
-module Dragonfly
-  module Cache
-    class Storage
-      extend Forwardable
-
-      attr_reader :manager
-
-      delegate %i[servers_options] => :manager
-
-      def initialize(manager)
-        @manager = manager
-        check_directory!(base_dir)
-      end
-
-      def current_max_sha_size
-        longest = Dir["#{base_dir}/*"].select { |p| File.directory?(p) }.max { |a, b| a <=> b }
-        (longest ? File.basename(longest).size : nil)
-      end
-
-      def write(job, uri)
-        path = cache_path(uri)
-        check_directory!(File.dirname(path))
-        with_umask(0o022) do
-          job.to_file(path, mode: 0o644, mkdirs: false)
-        end
-      end
-
-      def writen?(_job, uri)
-        File.exist?(cache_path(uri))
-      end
-
-      def base_dir
-        @base_dir ||= begin
-          path_format = File.join(servers_options[:server_root], servers_options[:url_format])
-          path_format.split('/').take_while { |p| p != ':shaish' }.join('/')
-        end
-      end
-
-      protected
-
-      def check_directory!(directory)
-        return if File.exist?(directory) && File.directory?(directory) && File.writable?(directory)
-        with_umask(0o022) do
-          File.unlink(directory) if File.exist?(directory) && !File.directory?(directory)
-          FileUtils.mkdir_p(directory, mode: 0o755) unless File.exist?(directory)
-          File.chmod(0o755, directory) unless File.writable?(directory)
-        end
-      rescue ::StandardError => e
-        raise Dragonfly::Cache::Error, e.message
-      end
-
-      def with_umask(umask)
-        original_umask = File.umask(umask)
-        begin
-          yield
-        ensure
-          File.umask(original_umask)
-        end
-      end
-
-      def cache_path(uri)
-        File.join(servers_options[:server_root], uri.path)
-      end
-    end
-  end
-end