rack-steady_etag 0.1.1 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bbaf5489b11a9fe181a1f92469ebaebafaa41bb11d911f8fb0f0ee0589fb7f15
-  data.tar.gz: ff4685ce1f6215217bfadeb9cd3d02fc750b84b9aa74b1d2deb6186c8ed7da89
+  metadata.gz: 4e9c4ec38a103ceb321da1a48a78a09872e7d47875fbd0fa1e15814116f79e64
+  data.tar.gz: cb4abb1d12ac1c2a06427f7e6f63de34c4d8b99c56248287470f2c4aa3be655a
 SHA512:
-  metadata.gz: 4004ed7d3b297299c3bfb343c15053548957afe60835804e087a71cf6b94558ef3dfbdb41918d54034b8fd5a464142fe137ef9a535f5f28a3ddcc9e3330c95c2
-  data.tar.gz: ec21028482dc4c0da1a4534d0c322ab012a21b988df7ffda07764f7cb66063c0606b4656707149307ac94ea8c301c27e20fce4e7d71ac88093f75a1c36f7d3de
+  metadata.gz: ab95c964994e61c962903a4a395be6715bc2a8d69e1cb66f0eb19371c4691972ff8d2cd76d8740129715652d515f0f0704238afbae5e30be9527065e0b127d59
+  data.tar.gz: f10b144400e01f9587e51f7f20330cb8a3d14381c4183ecfa84f9180d301b01986eeb1df3b1f498115eabab2161763106e8dde26c96e3ce9fc031eb94c56016e

data/CHANGELOG.md

@@ -5,14 +5,15 @@ This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html
 
 ## Unreleased
 
-### Breaking changes
+## 0.2.0 - 2022-05-12
 
-### Compatible changes
+- Be more compatible with Rack 2.2.2:
+  - Always set a `Cache-Control` header, even for responses that we don't try to digest.
+- Strip patterns for responses with `Cache-Control: public`
+- Requires Rack 2.x (we want to break with Rack 3)
 
 ## 0.1.1 - 2022-05-16
 
-### Compatible changes
-
 - Activate rubygems MFA
 
 ## 0.1.0 - 2021-12-01

data/Gemfile.lock

@@ -1,25 +1,24 @@
 PATH
   remote: .
   specs:
-    rack-steady_etag (0.1.1)
+    rack-steady_etag (0.2.0)
       activesupport (>= 3.2)
-      rack
+      rack (~> 2.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (6.1.4.1)
+    activesupport (7.0.1)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
       tzinfo (~> 2.0)
-      zeitwerk (~> 2.3)
     byebug (11.1.3)
     concurrent-ruby (1.1.9)
     diff-lcs (1.4.4)
     i18n (1.8.11)
       concurrent-ruby (~> 1.0)
-    minitest (5.14.4)
+    minitest (5.15.0)
     rack (2.2.3)
     rake (13.0.6)
     rspec (3.10.0)
@@ -37,7 +36,6 @@ GEM
     rspec-support (3.10.3)
     tzinfo (2.0.4)
       concurrent-ruby (~> 1.0)
-    zeitwerk (2.5.1)
 
 PLATFORMS
   x86_64-linux

data/README.md

@@ -39,7 +39,7 @@ Transformations are only applied for the `ETag` hash. The response body will not
 - No `ETag` is generated when the response already has an `Last-Modified` header.
 
 
-## Installation
+## Installation in Rails
 
 Add this line to your application's Gemfile:
 
@@ -56,9 +56,11 @@ bundle install
 In your `config/application.rb`:
 
 ```ruby
-config.middleware.swap Rack::ETag, Rack::SteadyETag
+config.middleware.swap Rack::ETag, Rack::SteadyETag, 'no-cache'
 ```
 
+The `'no-cache'` argument is the default `Cache-Control` for responses that cannot be digested. While it may feel surprising that the middleware changes the `Cache-Control` header in such a case, the [Rails default middleware stack](https://github.com/rails/rails/blob/d96609505511a76c618dc3adfa3ca4679317d008/railties/lib/rails/application/default_middleware_stack.rb#L81) configures the same behavior.
+
 
 ## Development
 

data/lib/rack/steady_etag/version.rb

@@ -2,6 +2,6 @@
 
 module Rack
   class SteadyEtag
-    VERSION = "0.1.1"
+    VERSION = "0.2.0"
   end
 end

data/lib/rack/steady_etag.rb

@@ -6,8 +6,8 @@ require_relative "steady_etag/version"
 
 module Rack
 
-  # Based on Rack::Etag
-  # https://github.com/rack/rack/blob/master/lib/rack/etag.rb
+  # Based on Rack::Etag from rack 2.2.2
+  # https://github.com/rack/rack/blob/v2.2.2/lib/rack/etag.rb
   #
   # Automatically sets the ETag header on all String bodies.
   #
@@ -15,6 +15,8 @@ module Rack
   # a sendfile body (body.responds_to :to_path) is given (since such cases
   # should be handled by apache/nginx).
   class SteadyETag
+
+    # Yes, Rack::ETag sets a default Cache-Control for responses that it can digest.
     DEFAULT_CACHE_CONTROL = "max-age=0, private, must-revalidate"
 
     IGNORE_PATTERNS = [
@@ -24,11 +26,14 @@ module Rack
       lambda { |string| string.gsub(/(<script\b[^>]*)\bnonce=(["'])[^"']+\2+/i, '\1') }
     ]
 
-    def initialize(app, no_digest_cache_control: nil, digest_cache_control: DEFAULT_CACHE_CONTROL, ignore_patterns: IGNORE_PATTERNS.dup)
+    def initialize(app, no_digest_cache_control = nil, digest_cache_control = DEFAULT_CACHE_CONTROL)
       @app = app
+
       @digest_cache_control = digest_cache_control
+
+      # Rails sets a default `Cache-Control: no-cache` for responses that
+      # we cannot digest.
       @no_digest_cache_control = no_digest_cache_control
-      @ignore_patterns = ignore_patterns
     end
 
     def call(env)
@@ -45,6 +50,18 @@ module Rack
         headers[ETAG] = %(W/"#{digest}") if digest
       end
 
+      # It would make more sense to only set a Cache-Control for responses that we process.
+      # However, the original Rack::ETag sets Cache-Control: @no_digest_cache_control
+      # for all responses, even responses that we don't otherwise modify.
+      # Hence if we move this code into the `if` above we would remove Rails' default
+      # Cache-Control headers for non-digestable responses, which would be a considerable
+      # change in behavior.
+      if digest
+        set_cache_control_with_digest(headers)
+      else
+        set_cache_control_without_digest(headers)
+      end
+
       [status, headers, body]
     end
 
@@ -63,6 +80,9 @@ module Rack
     end
 
     def etag_body?(body)
+      # Rack main branch checks for `:to_ary` here to exclude streaming responses,
+      # but that had other issues for me in testing. Maybe recheck when there is a
+      # new Rack release after 2.2.2.
       !body.respond_to?(:to_path)
     end
 
@@ -70,10 +90,6 @@ module Rack
       headers.key?(ETAG) || headers.key?('Last-Modified')
     end
 
-    def cache_control_private?(headers)
-      headers[CACHE_CONTROL] && headers[CACHE_CONTROL] =~ /\bprivate\b/
-    end
-
     def digest_body(body, headers, session)
       parts = []
       digest = nil
@@ -82,11 +98,7 @@ module Rack
         parts << part
 
         if part.present?
-          set_cache_control_with_digest(headers)
-
-          if cache_control_private?(headers)
-            part = strip_ignore_patterns(part)
-          end
+          part = strip_ignore_patterns(part)
 
           unless digest
             digest = Digest::SHA256.new
@@ -102,15 +114,13 @@ module Rack
 
       if digest
         digest = digest.hexdigest.byteslice(0,32)
-      else
-        set_cache_control_without_digest(headers)
       end
 
       [digest, parts]
     end
 
     def strip_ignore_patterns(html)
-      @ignore_patterns.each do |ignore_pattern|
+      IGNORE_PATTERNS.each do |ignore_pattern|
         if ignore_pattern.respond_to?(:call)
           html = ignore_pattern.call(html)
         else

data/rack-steady_etag.gemspec

@@ -31,10 +31,10 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  # Uncomment to register a new dependency of your gem
-  spec.add_dependency "rack"
-  spec.add_dependency 'activesupport', '>= 3.2'
+  # I expect Rack 3 to have a new ETag middleware to no longer buffer
+  # streaming responses: https://github.com/rack/rack/issues/1619
+  # Once Rack 3 is out we should release a new version of this gem.
+  spec.add_dependency "rack", '~>2.0'
 
-  # For more information and examples about making a new gem, checkout our
-  # guide at: https://bundler.io/guides/creating_gem.html
+  spec.add_dependency 'activesupport', '>= 3.2'
 end

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: rack-steady_etag
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.2.0
 platform: ruby
 authors:
 - Henning Koch
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-03-16 00:00:00.000000000 Z
+date: 2022-05-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
@@ -83,7 +83,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.2.6
 signing_key: 
 specification_version: 4
 summary: Rack Middleware that produces the same ETag for responses that only differ