async-http-cache 0.1.3 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 835bedef1b242fc89e416ac99d2a410896262c18eaa985e6eea715b01e9a9f37
-  data.tar.gz: 36a5a86d671ab98e8531926a2d3ea56340820c729426d6d8dd55c6145f6b0158
+  metadata.gz: 17173f66af03c869ff56b8dd197565e230d10899eee2d3da2f18685ca670e892
+  data.tar.gz: 1d6b1a438c596bdca021820f69dd86f02bcca593ffd771ab7912feb8f2f530a4
 SHA512:
-  metadata.gz: 3b006e0e15afc6c7f506d7f6ff1fc97f4036690786ce72741f28ee1570e4eed238afc719ac62a8880b1ce2d9acee6542e9129b2cb21cfc30a38ae005d78f62f1
-  data.tar.gz: 02ae7fb372c4952541c061c91476352684fb2be0f9cd1c4d56f9d9a7b4dfa459d371a03b2b44cdbc12bc46b1c8d808ae5c13a62072b60c80f911048859735fe4
+  metadata.gz: 43ab1d479efa3d86814c73241ba78281ea3ce7fa125fde7393556ba1529192608da09979dc6e895e81621531beaf132195fb993d72638a35c38489e4f6436f37
+  data.tar.gz: 79e4aaf2b574eab58253a92e5dc55b0f526733ad3335f035f3b07b91a3f1e54bc4e039e047d1c31da4484e41c132607888edc4c44ef8734bdb31c4ee87a86d8a

data/.github/workflows/development.yml

@@ -11,7 +11,6 @@ jobs:
           - macos
         
         ruby:
-          - 2.4
           - 2.5
           - 2.6
           - 2.7

data/Gemfile

@@ -2,3 +2,7 @@ source "https://rubygems.org"
 
 # Specify your gem's dependencies in async-http-cache.gemspec
 gemspec
+
+# gem "async-http", path: "../async-http"
+# gem "protocol-http", path: "../protocol-http"
+# gem "protocol-http1", path: "../protocol-http1"

data/async-http-cache.gemspec

@@ -21,13 +21,12 @@ Gem::Specification.new do |spec|
 	
 	spec.require_paths = ["lib"]
 	
-	spec.add_dependency "async-http"
-	spec.add_dependency "protocol-http", "~> 0.14"
+	spec.add_dependency "async-http", "~> 0.56"
 	
 	spec.add_development_dependency "async-rspec", "~> 1.10"
 	
 	spec.add_development_dependency "covered"
 	spec.add_development_dependency "bundler"
 	spec.add_development_dependency "rspec"
-	spec.add_development_dependency "rake"
+	spec.add_development_dependency "bake-bundler"
 end

data/lib/async/http/cache/body.rb

@@ -21,33 +21,49 @@
 # THE SOFTWARE.
 
 require 'protocol/http/body/rewindable'
-require 'protocol/http/body/streamable'
+require 'protocol/http/body/completable'
+require 'protocol/http/body/digestable'
 
 module Async
 	module HTTP
 		module Cache
 			module Body
-				def self.wrap(message, &block)
-					if body = message.body
-						# Create a rewindable body wrapping the message body:
-						rewindable = ::Protocol::HTTP::Body::Rewindable.new(body)
-						
-						# Set the message body to the rewindable body:
-						message.body = rewindable
-						
-						# Wrap the message with the callback:
-						::Protocol::HTTP::Body::Streamable.wrap(message) do |error|
-							if error
-								Async.logger.error(self) {error}
-							else
-								yield message, rewindable.buffered
+				TRAILER = 'trailer'
+				ETAG = 'etag'
+				
+				def self.wrap(response, &block)
+					if body = response.body
+						if body.empty?
+							# A body that is empty? at the outset, is immutable. This generally only applies to HEAD requests.
+							yield response, body
+						else
+							# Insert a rewindable body so that we can cache the response body:
+							rewindable = ::Protocol::HTTP::Body::Rewindable.wrap(response)
+							
+							unless response.headers.include?(ETAG)
+								# Compute a digest and add it to the response headers:
+								::Protocol::HTTP::Body::Digestable.wrap(response) do |wrapper|
+									response.headers.add(ETAG, wrapper.etag)
+								end
+								
+								# Ensure the etag is listed as a trailer:
+								response.headers.add(TRAILER, ETAG)
+							end
+							
+							# Wrap the response with the callback:
+							::Protocol::HTTP::Body::Completable.wrap(response) do |error|
+								if error
+									Async.logger.error(self) {error}
+								else
+									yield response, rewindable.buffered
+								end
 							end
 						end
 					else
-						yield message, nil
+						yield response, nil
 					end
 					
-					return message
+					return response
 				end
 			end
 		end

data/lib/async/http/cache/general.rb

@@ -31,6 +31,7 @@ module Async
 		module Cache
 			class General < ::Protocol::HTTP::Middleware
 				CACHE_CONTROL  = 'cache-control'
+				
 				CONTENT_TYPE = 'content-type'
 				AUTHORIZATION = 'authorization'
 				COOKIE = 'cookie'
@@ -91,9 +92,17 @@ module Async
 						return response
 					end
 					
-					return Body.wrap(response) do |message, body|
+					if request.head? and body = response.body
+						unless body.empty?
+							Async.logger.warn(self) {"HEAD request resulted in non-empty body!"}
+							
+							return response
+						end
+					end
+					
+					return Body.wrap(response) do |response, body|
 						Async.logger.debug(self) {"Updating cache for #{key}..."}
-						@store.insert(key, request, Response.new(message, body))
+						@store.insert(key, request, Response.new(response, body))
 					end
 				end
 				
@@ -107,7 +116,7 @@ module Async
 							Async.logger.debug(self) {"Cache hit for #{key}..."}
 							@count += 1
 							
-							# Create a dup of the response:
+							# Return the cached response:
 							return response
 						end
 					end

data/lib/async/http/cache/response.rb

@@ -29,6 +29,9 @@ module Async
 			class Response < ::Protocol::HTTP::Response
 				CACHE_CONTROL = 'cache-control'
 				SET_COOKIE = 'set-cookie'
+				ETAG = 'etag'
+				
+				X_CACHE = 'x-cache'
 				
 				def initialize(response, body)
 					@generated_at = Async::Clock.now
@@ -36,16 +39,23 @@ module Async
 					super(
 						response.version,
 						response.status,
-						response.headers.dup,
+						response.headers.flatten,
 						body,
 						response.protocol
 					)
 					
 					@max_age = @headers[CACHE_CONTROL]&.max_age
+					@etag = nil
+					
+					@headers.set(X_CACHE, 'hit')
 				end
 				
 				attr :generated_at
 				
+				def etag
+					@etag ||= @headers[ETAG]
+				end
+				
 				def cachable?
 					if cache_control = @headers[CACHE_CONTROL]
 						if cache_control.private? || !cache_control.public?

data/lib/async/http/cache/store/memory.rb

@@ -64,6 +64,9 @@ module Async
 					
 					attr :index
 					
+					IF_NONE_MATCH = 'if-none-match'
+					NOT_MODIFIED = ::Protocol::HTTP::Response[304]
+					
 					def lookup(key, request)
 						if response = @index[key]
 							if response.expired?
@@ -74,6 +77,12 @@ module Async
 								return nil
 							end
 							
+							if etags = request.headers[IF_NONE_MATCH]
+								if etags.include?(response.etag)
+									return NOT_MODIFIED
+								end
+							end
+							
 							@hit += 1
 							
 							return response.dup
@@ -86,7 +95,8 @@ module Async
 					
 					def insert(key, request, response)
 						if @index.size < @limit
-							if response.body.length < 1024*64
+							length = response.body&.length
+							if length.nil? or length < 1024*64
 								@index[key] = response
 							end
 						end

data/lib/async/http/cache/version.rb

@@ -23,7 +23,7 @@
 module Async
 	module HTTP
 		module Cache
-			VERSION = "0.1.3"
+			VERSION = "0.4.0"
 		end
 	end
 end

metadata

@@ -1,43 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: async-http-cache
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.4.0
 platform: ruby
 authors:
 - Samuel Williams
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-03-21 00:00:00.000000000 Z
+date: 2021-04-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: async-http
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: protocol-http
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '0.56'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '0.56'
 - !ruby/object:Gem::Dependency
   name: async-rspec
   requirement: !ruby/object:Gem::Requirement
@@ -95,7 +81,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: bake-bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -108,7 +94,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
+description:
 email:
 - samuel.williams@oriontransfer.co.nz
 executables: []
@@ -120,7 +106,6 @@ files:
 - ".rspec"
 - Gemfile
 - README.md
-- Rakefile
 - async-http-cache.gemspec
 - lib/async/http/cache.rb
 - lib/async/http/cache/body.rb
@@ -134,7 +119,7 @@ homepage: https://github.com/socketry/async-http-cache
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -150,7 +135,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Standard-compliant cache for async-http.
 test_files: []

data/Rakefile

@@ -1,6 +0,0 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
-
-RSpec::Core::RakeTask.new(:spec)
-
-task :default => :spec