response_bank 1.3.0 → 1.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7d25739f9bd846e1cd3364a422cc90590b32fcb5f7e56cd307af0e83f0d5ddac
-  data.tar.gz: 14c1a7f96abcc45025cee9f998342fb6185779774dc72986ebb1a5bbcb96c413
+  metadata.gz: 87b964dde00b2a9aa0b319430185a6cfefa8d40323b634556cf1298c8e2ee1a9
+  data.tar.gz: 16b9d49921f037bec53031f145e44815a820de2ec93992a6d6c4926cb22b2f95
 SHA512:
-  metadata.gz: b6120bc7a707fa5ddd1cd3a143e40681e375864b075fe0e74b1128cc4ee460690400a70c47599772addba8d2bee56b02ce14a820c5e4af376f0eaa154f1fe88c
-  data.tar.gz: 5ae1545f2c40d7abbf06b8ba43c8d353cb171fc97ce667fa280c5f991c37f7e514c7673898a99242e446f9975ef22f4cb91eaa053165664a9c309fa4d6b39197
+  metadata.gz: 1f95e92a493db39dd4cb3dc43dd5582d19e64a9857f3304420152ae235088f1d1101cc480d112c9f49346c08e168eaaf881e351061f0ac17d6676a452ca4dc06
+  data.tar.gz: b77dfa70dc32d45153c2177e9e5c4762a881d7bd998479b8a209121b903ef9d3f323335ebba7aeb6bbfab381fd5fbddc2f861ddf19145e606348d8b5b00a60e7

data/lib/response_bank/middleware.rb

@@ -22,8 +22,7 @@ module ResponseBank
 
       if env['cacheable.cache']
         if [200, 404, 301, 304].include?(status)
-          headers['ETag'] = env['cacheable.key']
-
+          headers['ETag'] = %{"#{env['cacheable.key']}"}
         end
 
         if [200, 404, 301].include?(status) && env['cacheable.miss']

data/lib/response_bank/response_cache_handler.rb

@@ -145,11 +145,14 @@ module ResponseBank
         # regen
         @headers.merge!(headers)
 
-        # if a cache key hit and client doesn't match encoding, return the raw body
-        if !@env['HTTP_ACCEPT_ENCODING'].to_s.include?(@headers['Content-Encoding'])
-          ResponseBank.log("uncompressing payload for client as client doesn't require encoding")
-          body = ResponseBank.decompress(body, @headers['Content-Encoding'])
-          @headers.delete('Content-Encoding')
+        if @headers['Content-Encoding']
+          if !@env['HTTP_ACCEPT_ENCODING'].to_s.include?(@headers['Content-Encoding'])
+            ResponseBank.log("uncompressing payload for client as client doesn't require encoding")
+            body = ResponseBank.decompress(body, @headers['Content-Encoding'])
+            @headers.delete('Content-Encoding')
+          end
+        else
+          ResponseBank.log("Cache hit, but missing content-encoding in the cache value headers, maybe because of 301 or 404 response or empty body string")
         end
 
         [status, @headers, [body]]

data/lib/response_bank/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module ResponseBank
-  VERSION = "1.3.0"
+  VERSION = "1.3.1"
 end

data/lib/response_bank.rb

@@ -64,7 +64,7 @@ module ResponseBank
         key = %{#{key}:#{hash_value_str(data[:version])}} if data[:version]
 
         # add the encoding to only the cache key but don't expose this detail in the entity_tag
-        key = %{#{key}:#{hash_value_str(data[:encoding])}} if data[:encoding] && data[:encoding] != "gzip"
+        key = %{#{key}:#{hash_value_str(data[:encoding])}} if data[:encoding]
 
         key
       when Array

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: response_bank
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
 platform: ruby
 authors:
 - Tobias Lütke
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-04-04 00:00:00.000000000 Z
+date: 2023-04-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: msgpack