graphql-relay-walker 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4afa127af28fd4cffeea4fcc562e50275f397dd9aaead073d406d615a82d3a93
-  data.tar.gz: 9e26b00ba9b1b2d4f81454b79621a55368263894bb2ba164da98708707967934
+  metadata.gz: d351dcfe5896d75e7086bab93406475e6af14d9f5901330a03dfaae0c1f225a0
+  data.tar.gz: a4f68b19ecb09f2e07168f7b3ad2f58fd71600acf169054249026a18c42f068d
 SHA512:
-  metadata.gz: a44890c61009ccd8cf9cc1e700f150107105bc3f03da0b8703948e682af1224cda18b9f71a941cf12e94c6f0a315649f7dd0c2c892c21cec5e081c9149722c16
-  data.tar.gz: 83dff8d60ef1a583e1c1a92db5d91c348b6bbb9034c3f3f86be8ca35f97f3e618d920586af1762715909b8efbcd15742d049dcb2f351f34bcb5b36089ae50d54
+  metadata.gz: b42954a51f5607972833980833fc45d7a5e4eb512b91c91aba116fa53af3cd9eed7a89b179d2a42ac01642ad17a20cc368b107c854359ae6da97a0fcc82cab4a
+  data.tar.gz: 0e1c06e2284e48fa72f7f3452c133e450cbdd6aebb020cdd05c5afffeb26d48f40f77808bd92b60340d2f863d440edb0b9c653100e320ec1c9331e75edbf5fae

data/graphql-relay-walker.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name        = 'graphql-relay-walker'
-  s.version     = '0.2.0'
+  s.version     = '0.3.0'
   s.licenses    = ['MIT']
   s.summary     = 'A tool for traversing your GraphQL schema to proactively detect potential data access vulnerabilities.'
   s.authors     = ['Ben Toews']
@@ -9,9 +9,9 @@ Gem::Specification.new do |s|
   s.files += Dir.glob('lib/**/*.rb')
   s.homepage = 'https://github.com/github/graphql-relay-walker'
 
-  s.add_dependency 'graphql', '>= 1.5.6'
+  s.add_dependency 'graphql', '~> 1.8'
 
-  s.add_development_dependency 'graphql-client', '~> 0.2'
+  s.add_development_dependency 'graphql-client', '~> 0.15'
   s.add_development_dependency 'rake',           '~> 11.3'
   s.add_development_dependency 'rspec',          '~> 3.5'
 end

data/lib/graphql/relay/walker/query_builder.rb

@@ -70,7 +70,10 @@ module GraphQL::Relay::Walker
     def inline_fragment_ast(type, with_children: true)
       selections = []
       if with_children
-        type.all_fields.each do |field|
+        # Class-based types return all fields in `.fields`
+        all_fields = type.respond_to?(:all_fields) ? type.all_fields : type.fields.values
+        all_fields = all_fields.sort_by(&:graphql_name)
+        all_fields.each do |field|
           field_type = field.type.unwrap
           if node_field?(field) && include?(field_type)
             selections << node_field_ast(field)
@@ -88,7 +91,7 @@ module GraphQL::Relay::Walker
         nil
       else
         GraphQL::Language::Nodes::InlineFragment.new(
-          type: make_type_name_node(type.name),
+          type: make_type_name_node(type.graphql_name),
           selections: selections,
         )
       end
@@ -114,12 +117,12 @@ module GraphQL::Relay::Walker
       if !required_args_are_present
         nil
       else
-        f_alias = field.name == 'id' ? nil : random_alias
+        f_alias = field.graphql_name == 'id' ? nil : random_alias
         f_args = arguments.map do |name, value|
           GraphQL::Language::Nodes::Argument.new(name: name, value: value)
         end
 
-        GraphQL::Language::Nodes::Field.new(name: field.name, alias: f_alias, arguments: f_args)
+        GraphQL::Language::Nodes::Field.new(name: field.graphql_name, alias: f_alias, arguments: f_args)
       end
     end
 
@@ -213,11 +216,12 @@ module GraphQL::Relay::Walker
     # `node` field that is a relay node. Returns false otherwise.
     def connection_field?(field)
       type = field.type.unwrap
-
-      if edges_field = type.get_field('edges')
-        edges = edges_field.type.unwrap
-        if node_field = edges.get_field('node')
-          return node_field?(node_field)
+      if type.kind.fields?
+        if edges_field = type.get_field('edges')
+          edges = edges_field.type.unwrap
+          if node_field = edges.get_field('node')
+            return node_field?(node_field)
+          end
         end
       end
 
@@ -268,7 +272,7 @@ module GraphQL::Relay::Walker
     end
 
     def valid_input?(type, input)
-      type.valid_isolated_input?(input)
+      type.valid_input?(input, GraphQL::Query::NullContext)
     end
 
     def make_type_name_node(type_name)

metadata

@@ -1,43 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: graphql-relay-walker
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Ben Toews
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-06-10 00:00:00.000000000 Z
+date: 2019-10-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.5.6
+        version: '1.8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.5.6
+        version: '1.8'
 - !ruby/object:Gem::Dependency
   name: graphql-client
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: '0.15'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: '0.15'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement