graphql-persisted_queries 1.6.1 → 1.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fa710e0a1af1fe00953679781bf408f3c78201b67920671f9af36ff57edefcc0
-  data.tar.gz: cc46697bf61938af0d4359fb9b5bf4898f3bc5108b4f562f94d3003a857064a6
+  metadata.gz: 430993c73790f83784c3e048a8fa0a4032f2fa5809b725265b5600b6ca8b5a01
+  data.tar.gz: 10aaa0422835323129d2e771f9bcfa68c4b8d24563ff7b8fb0f341b29dc92dd4
 SHA512:
-  metadata.gz: 6028cb87dc1e0c7a78f18f29c7afbff2d2aae46fadbec983d6ebbb217016523a8722f27c993570b914b3fec6cebe0d16ad26e7091844a0e60f4bf0d80217cb11
-  data.tar.gz: a0d0dada8b7f34298a69aed43dd988ce077e010866faf33c8d206375ee0cb2f00afed1f0794753983a5b2b01f266888f7988d77a031f33c6f55d2779abd62e35
+  metadata.gz: c8cf0e07efa3c4a2ea92af6014333c3adc08f1471357b3cc31f1d93779e5af3920e7c71b308bff62f34366211fd5912baadd9780cc2978c9b4185582524f9181
+  data.tar.gz: f001860c11580ae570e25875f548a167ec878d87545befb39b7ee13a58385e2d2fda86a0919745030b915fed8bd84b3cc6200462c467b320e823cb43a9785047

data/.github/workflows/rspec.yml

@@ -18,7 +18,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        ruby: [2.6, 2.7, 3.0]
+        ruby: [2.7, '3.0', 3.1]
         gemfile: [
           "gemfiles/graphql_1_12_0.gemfile",
           "gemfiles/graphql_1_12_4.gemfile",

data/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## master
 
+## 1.7.0 (2023-02-02)
+
+- [PR#62](https://github.com/DmitryTsepelev/graphql-ruby-persisted_queries/pull/62)
+Fix double hashing of keys in case of redis_with_local_cache_store adapter ([@mpospelov][])
+- [PR#62](https://github.com/DmitryTsepelev/graphql-ruby-persisted_queries/pull/62) Drop tracers= implementation as it's no longer valid ([@mpospelov][])
+
 ## 1.6.1 (2022-11-17)
 
 - [PR#60](https://github.com/DmitryTsepelev/graphql-ruby-persisted_queries/pull/60)
@@ -112,3 +118,4 @@ Handle situations when prepare_ast happens before instrumentation ([@DmitryTsepe
 [@ogidow]: https://github.com/ogidow
 [@rbviz]: https://github.com/rbviz
 [@louim]: https://github.com/louim
+[@mpospelov]: https://github.com/mpospelov

data/README.md

@@ -6,6 +6,12 @@
 - 🤝**Clients share cached queries** – it's enough to miss cache only once for each unique query
 - 🎅**Works for clients without persisted query support**
 
+Used in production by:
+
+- [Yammer by Microsoft](https://github.com/DmitryTsepelev/graphql-ruby-persisted_queries/issues/20#issuecomment-587945989)
+- Toptal
+- _Want to be here? Let me know_ 🙂
+
 ## Getting started
 
 First of all, install and configure [apollo's persisted queries](https://www.apollographql.com/docs/react/api/link/persisted-queries/) on the front–end side:

data/graphql-persisted_queries.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.required_ruby_version = ">= 2.6"
+  spec.required_ruby_version = ">= 2.7"
 
   spec.add_dependency "graphql", ">= 1.12"
 

data/lib/graphql/persisted_queries/store_adapters/base_store_adapter.rb

@@ -12,7 +12,8 @@ module GraphQL
           @name = :base
         end
 
-        def fetch_query(hash, compiled_query: false)
+        def fetch_query(hash, options = {})
+          compiled_query = options[:compiled_query] || false
           key = build_key(hash, compiled_query)
 
           fetch(key).tap do |result|
@@ -26,8 +27,6 @@ module GraphQL
           trace("save_query", adapter: @name) { save(key, query) }
         end
 
-        protected
-
         def fetch(_hash)
           raise NotImplementedError
         end

data/lib/graphql/persisted_queries/store_adapters/memcached_store_adapter.rb

@@ -17,8 +17,6 @@ module GraphQL
           @name = :memcached
         end
 
-        protected
-
         def fetch(hash)
           @dalli_proc.call { |dalli| dalli.get(key_for(hash)) }
         end

data/lib/graphql/persisted_queries/store_adapters/memory_store_adapter.rb

@@ -10,8 +10,6 @@ module GraphQL
           @name = :memory
         end
 
-        protected
-
         def fetch(hash)
           @storage[hash]
         end

data/lib/graphql/persisted_queries/store_adapters/redis_store_adapter.rb

@@ -17,8 +17,6 @@ module GraphQL
           @name = :redis
         end
 
-        protected
-
         def fetch(hash)
           @redis_proc.call { |redis| redis.get(key_for(hash)) }
         end

data/lib/graphql/persisted_queries/store_adapters/redis_with_local_cache_store_adapter.rb

@@ -22,29 +22,19 @@ module GraphQL
           @name = :redis_with_local_cache
         end
 
-        # We don't need to implement our own traces for this adapter since the
-        # underlying adapters will emit the proper events for us.  However,
-        # since tracers can be defined at any time, we need to pass them through.
-        def tracers=(tracers)
-          @memory_adapter.tracers = tracers
-          @redis_adapter.tracers = tracers
-        end
-
-        protected
-
         def fetch(hash)
-          result = @memory_adapter.fetch_query(hash)
+          result = @memory_adapter.fetch(hash)
           result ||= begin
-            inner_result = @redis_adapter.fetch_query(hash)
-            @memory_adapter.save_query(hash, inner_result) if inner_result
+            inner_result = @redis_adapter.fetch(hash)
+            @memory_adapter.save(hash, inner_result) if inner_result
             inner_result
           end
           result
         end
 
         def save(hash, query)
-          @redis_adapter.save_query(hash, query)
-          @memory_adapter.save_query(hash, query)
+          @redis_adapter.save(hash, query)
+          @memory_adapter.save(hash, query)
         end
 
         private

data/lib/graphql/persisted_queries/version.rb

@@ -2,6 +2,6 @@
 
 module GraphQL
   module PersistedQueries
-    VERSION = "1.6.1"
+    VERSION = "1.7.0"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphql-persisted_queries
 version: !ruby/object:Gem::Version
-  version: 1.6.1
+  version: 1.7.0
 platform: ruby
 authors:
 - DmitryTsepelev
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-11-17 00:00:00.000000000 Z
+date: 2023-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql
@@ -182,14 +182,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.6'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.4.1
 signing_key:
 specification_version: 4
 summary: Persisted queries for graphql-ruby