faraday-http-cache 2.4.0 → 2.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6603be3bbf6a2840ba11f947f7e42f030ab52600912e9a62d16aaf27ee5e6446
-  data.tar.gz: 95b8b2f4b08525406027d47e14341f5f301409a2518332b243dc08d2bde4e693
+  metadata.gz: 6f9734c5eda496aee6fc6d189e27f9e39b43600ffb467976649ad3f7046a8efc
+  data.tar.gz: 1a4d2d137c3ec0a97790cc4b5bcb706f7e0c2af0d0700e0ae03dfc79f5e2bdba
 SHA512:
-  metadata.gz: 4675c865c18641322c71b77881a74d3ca4edbf4fb1d628ac0fe7a6388d6a3c973946f1d7fa7b09d654f1eff61484a8c2d8524c966e4377c1e12854bf8114e13a
-  data.tar.gz: 832d7579eb75d3631e539ca7af996b9d062dbe43582468f98c349b3dbc963dde17cecf57a0202af1916297013143becba1a0fc1f32ddf92ab98490876e3efc84
+  metadata.gz: f93240786d8c3b8012cce14545e2f0f34f1bb12811da4edc549ad145284712b59380dac02f91ed6170d19af7af220e48d905a31a1159129d2b32b45799251df6
+  data.tar.gz: 8b2480bebc178003743ca2e0535966ad617349ef7ead2f7b5907f00d4580ba990332472d2ac393dc6b5bd79b787d8d560e5d772f9bfc1faca76ac6b34a5061d7

data/lib/faraday/http_cache/strategies/base_strategy.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require 'json'
+require 'logger'
 require 'faraday/http_cache/memory_store'
 
 module Faraday

data/lib/faraday/http_cache/strategies/by_vary.rb

@@ -77,7 +77,7 @@ module Faraday
         # @return [String]
         def response_cache_key_for(request, vary)
           method = request.method.to_s
-          headers = vary.split(/[\s,]+/).map { |header| request.headers[header] }
+          headers = vary.split(/[\s,]+/).uniq.sort.map { |header| request.headers[header] }
           Digest::SHA1.hexdigest("by_vary#{@cache_salt}#{method}#{request.url}#{headers.join}")
         end
       end

data/spec/strategies/by_vary_spec.rb

@@ -4,10 +4,11 @@ require 'spec_helper'
 
 describe Faraday::HttpCache::Strategies::ByVary do
   let(:vary_index_cache_key) { '64896419583e8022efeb21d0ece6e266c0e58b59' }
-  let(:cache_key) { '25230d75622fffc4f4de8a6af69e6e3764f7eb6f' }
-  let(:vary) { '' }
+  let(:cache_key) { '978047698d156fe8642a86dbfaacc675917c9a22' }
+  let(:vary) { 'Accept, Accept-Encoding, X-Requested-With' }
+  let(:headers) { {'Accept' => 'text/html', 'Accept-Encoding' => 'gzip, deflate, br' } }
   let(:request) do
-    env = {method: :get, url: 'http://test/index'}
+    env = {method: :get, url: 'http://test/index', headers: headers}
     double(env.merge(serializable_hash: env))
   end
 
@@ -54,11 +55,17 @@ describe Faraday::HttpCache::Strategies::ByVary do
           )
         end
       end
+
+      context 'with reordered and doubled values in the vary' do
+        let(:vary) { 'X-Requested-With,Accept,Accept-Encoding,Accept' }
+
+        it_behaves_like 'A strategy with serialization'
+      end
     end
 
     context 'with the Marshal serializer' do
       let(:vary_index_cache_key) { '6a7cb42440c10ef6edeb1826086a4d90b04103f0' }
-      let(:cache_key) { '45e0efd1a60d29ed69d6c6018dfcb96f58db89e0' }
+      let(:cache_key) { 'c9edbf280da95d4cac5acda8b8109c0aba2a469a' }
       let(:serializer) { Marshal }
       let(:strategy) { described_class.new(store: cache, serializer: Marshal) }
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: faraday-http-cache
 version: !ruby/object:Gem::Version
-  version: 2.4.0
+  version: 2.4.1
 platform: ruby
 authors:
 - Lucas Mazza
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-08 00:00:00.000000000 Z
+date: 2022-08-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday