rack-graphql 2.5.1 → 2.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bd612f51ed5029ea31f162ccb63419bbda93203c1c4dc32fd8fe8dc8cb3df296
-  data.tar.gz: b58c4efcf02014ea60969fdd8d4375c19fa1f0bd143f7b6c1496b2e6f977ac05
+  metadata.gz: e93e42e089512fb66fd31d4ee5927904e795b98729b7e61819c85062d1bbe05d
+  data.tar.gz: 01156f4ddac5d400a610aedb10eba7cf7b15f35ae54b2379e5a3cee812128c68
 SHA512:
-  metadata.gz: f8e5816abcb0fc2b487331784dd29b23322a83c788a3f43ac11f02f435d67ed2bc61c9bedc544eb6dabc55376eeaba7d1a2a906f8e4077ad0d488fcce9e24adf
-  data.tar.gz: a7ab2a91badf6bdf080f023beb4e989a2883308896691fa52512fb2034c33ea93e5ed9deb6216cec43918a3e1d0af5e3d51917adbc0c429a924751271129826f
+  metadata.gz: 19716823cf60ebd465c7c9d7927f2b03bc54c0c07e0ffe14105548af9effd090f5ae98df3a50efbf6bd59a54d1823181c6e5669accf5ef291e1a28850d7b4e42
+  data.tar.gz: 60b7edf7e00f84c5f83a7d10f03706660a5dd9b9e95dd3954b2e405c72814c10950c375bbfad2ac2e71043aa7cb79b6f5cfa34fea17949d1638d8e6c5185ab91

data/.bundler-version

@@ -0,0 +1 @@
+2.2.29

data/.dependabot/config.yml

@@ -3,9 +3,14 @@ version: 1
 update_configs:
   - package_manager: "ruby:bundler"
     directory: "/"
-    update_schedule: "monthly"
+    update_schedule: "daily"
     commit_message:
       prefix: "[dependabot]"
     allowed_updates:
       - match:
-          update_type: "all"
+          dependency_type: "all"
+          update_type: "security"
+    automerged_updates:
+      - match:
+          dependency_type: "all"
+          update_type: "security:patch"

data/.github/PULL_REQUEST_TEMPLATE.md

@@ -1,18 +1,6 @@
-## Description
+## Description, motivation and context
 <!--- Describe your changes in detail -->
-
-## Related issue(s)
-<!--- GH issue number -->
-
-## Motivation and Context
 <!--- Why is this change required? What problem does it solve? -->
-<!--- If it fixes an open issue, please link to the issue here. -->
 
-## How Has This Been Tested?
-<!--- Please describe in detail how you tested your changes. -->
-
-## Screenshots (if appropriate):
-<!--- Please add any screenshots of the feature. -->
-
-## Related PRs
-<!--- Please add links to any related PRs (backend, component packages, etc). -->
+## Related issue(s) or PR(s)
+<!--- GH issue number -->

data/.github/workflows/auto-merge.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: automerge
-        uses: "pascalgn/automerge-action@v0.11.0"
+        uses: "pascalgn/automerge-action@v0.12.0"
         env:
           GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
           MERGE_METHOD: squash

data/.github/workflows/ci.yml

@@ -0,0 +1,28 @@
+name: Ruby CI
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby-version: ['2.7', '3.0']
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby
+    # To automatically get bug fixes and new Ruby versions for ruby/setup-ruby,
+    # change this to (see https://github.com/ruby/setup-ruby#versioning):
+    # uses: ruby/setup-ruby@v1
+      uses: ruby/setup-ruby@473e4d8fe5dd94ee328fdfca9f8c9c7afc9dae5e
+      with:
+        ruby-version: ${{ matrix.ruby-version }}
+        bundler-cache: true # runs 'bundle install' and caches installed gems automatically
+
+    - name: Run tests
+      run: bundle exec rake ci

data/CHANGELOG.md

@@ -1,8 +1,20 @@
 # CHANGELOG
 
-## 2.5.1 - 2020-11-18
+## 2.7.0 - 2021-03-24
 
-- respond with http status 400 when UTF null byte passed as part of input
+- Add ability to skip setting up health endpoint on root path with `health_on_root_path` option.
+
+## 2.6.1 - 2021-01-14
+
+- Fix uninitialized `Timeout` error. The issue was fixed in https://github.com/rmosolgo/graphql-ruby/commit/56abba472dbb48a1f8445d41f928bea72b5148e9, but new version has not yet been relased.
+
+## 2.6.0 - 2021-01-14
+
+- Add ability to re-raise exception (`re_raise_exceptions` option)
+
+## 2.5.1 - 2020-11-19
+
+- respond with http status `400` when UTF null byte is passed as a part of the input
 
 ## 2.5.0 - 2020-11-18
 

data/Gemfile.lock

@@ -1,69 +1,71 @@
 PATH
   remote: .
   specs:
-    rack-graphql (2.5.1)
-      graphql (~> 1.11)
+    rack-graphql (2.7.1)
+      graphql (~> 1.12)
       oj
       rack (~> 2.2)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    ast (2.4.1)
-    awesome_print (1.8.0)
+    ast (2.4.2)
+    awesome_print (1.9.2)
     coderay (1.1.3)
     diff-lcs (1.4.4)
-    graphql (1.11.6)
+    graphql (1.12.17)
     method_source (1.0.0)
-    oj (3.10.16)
-    parallel (1.20.0)
-    parser (2.7.2.0)
+    oj (3.13.9)
+    parallel (1.21.0)
+    parser (3.0.2.0)
       ast (~> 2.4.1)
-    pry (0.13.1)
+    pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
     rack (2.2.3)
     rack-test (1.1.0)
       rack (>= 1.0, < 3)
     rainbow (3.0.0)
-    rake (13.0.1)
-    regexp_parser (1.8.2)
-    rexml (3.2.4)
+    rake (13.0.6)
+    regexp_parser (2.1.1)
+    rexml (3.2.5)
     rspec (3.10.0)
       rspec-core (~> 3.10.0)
       rspec-expectations (~> 3.10.0)
       rspec-mocks (~> 3.10.0)
-    rspec-core (3.10.0)
+    rspec-core (3.10.1)
       rspec-support (~> 3.10.0)
-    rspec-expectations (3.10.0)
+    rspec-expectations (3.10.1)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.10.0)
-    rspec-mocks (3.10.0)
+    rspec-mocks (3.10.2)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.10.0)
-    rspec-support (3.10.0)
-    rubocop (1.3.1)
+    rspec-support (3.10.2)
+    rubocop (1.22.1)
       parallel (~> 1.10)
-      parser (>= 2.7.1.5)
+      parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
-      regexp_parser (>= 1.8)
+      regexp_parser (>= 1.8, < 3.0)
       rexml
-      rubocop-ast (>= 1.1.1)
+      rubocop-ast (>= 1.12.0, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 2.0)
-    rubocop-ast (1.1.1)
-      parser (>= 2.7.1.5)
-    rubocop-performance (1.9.0)
-      rubocop (>= 0.90.0, < 2.0)
-      rubocop-ast (>= 0.4.0)
-    rubocop-rspec (2.0.0)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.12.0)
+      parser (>= 3.0.1.1)
+    rubocop-rake (0.6.0)
       rubocop (~> 1.0)
-      rubocop-ast (>= 1.1.0)
-    ruby-progressbar (1.10.1)
-    unicode-display_width (1.7.0)
+    rubocop-rspec (2.5.0)
+      rubocop (~> 1.19)
+    ruby-progressbar (1.11.0)
+    unicode-display_width (2.1.0)
 
 PLATFORMS
-  ruby
+  aarch64-linux-musl
+  x86_64-darwin-18
+  x86_64-darwin-19
+  x86_64-darwin-20
+  x86_64-linux
 
 DEPENDENCIES
   awesome_print
@@ -74,8 +76,8 @@ DEPENDENCIES
   rake
   rspec
   rubocop
-  rubocop-performance
+  rubocop-rake
   rubocop-rspec
 
 BUNDLED WITH
-   2.1.4
+   2.2.29

data/README.md

@@ -1,5 +1,5 @@
 [![Gem Version](https://badge.fury.io/rb/rack-graphql.svg)](https://rubygems.org/gems/rack-graphql)
-[![Build Status](https://travis-ci.org/RenoFi/rack-graphql.svg?branch=master)](https://travis-ci.org/RenoFi/rack-graphql)
+[![Build Status](https://github.com/RenoFi/rack-graphql/actions/workflows/ci.yml/badge.svg)](https://github.com/RenoFi/rack-graphql/actions/workflows/ci.yml?query=branch%3Amain)
 
 # rack-graphql
 
@@ -23,14 +23,16 @@ Add following to your `config.ru` file:
 
 ```ruby
 run RackGraphql::Application.call(
-  schema: YourGraqphqlSchema,                      # required
-  app_name: 'your-service-name',                   # optional, used for health endpoint content
-  context_handler: YourGraphqlContextHandler,      # optional, empty `proc` by default
-  log_exception_backtrace: !A9n.env.production?,   # optional, `false` default
+  schema: YourGraqphqlSchema,                       # required
+  app_name: 'your-service-name',                    # optional, used for health endpoint content
+  context_handler: YourGraphqlContextHandler,       # optional, empty `proc` by default
+  log_exception_backtrace: !A9n.env.production?,    # optional, `false` default
   # `true` when `RACK_GRAPHQL_LOG_EXCEPTION_BACKTRACE` env var is set to `'1'` or `'true'`
-  health_route: true,                              # optional, true by default
-  logger: A9n.logger,                              # optional, not set by default
+  health_route: true,                               # optional, true by default
+  health_on_root_path: health_route,                # optional, health_route value by default (mind map '/' is covering '/any/path-123') 
+  logger: A9n.logger,                               # optional, not set by default
   error_status_code_map: { IamTeapotError => 418 }, # optional
+  re_raise_exceptions: true,                        # optional, false by default
 )
 ```
 
@@ -81,6 +83,38 @@ RackGraphql catches all errors and respond with 500 code. By default it adds exc
 RackGraphql.log_exception_backtrace = false
 ```
 
+### Error tracking/reporting
+
+To respect the graphql spec, all errors need to be returned as json and `rack-graphql` catches all exceptions and does NOT re-raise them. You can change this behavior via `re_raise_exceptions` argument.
+Because of this, using error tracking middleware (`use Sentry::Rack::CaptureExceptions`, `use Raven::Rack`) does not take any effect for graphql requests.
+
+To use Sentry or other reporting tool for graphql queries, you should handle it on graphql schema level:
+
+```ruby
+class MySchema < GraphQL::Schema
+  rescue_from StandardError do |e, obj, args, ctx, field|
+    extra = {
+      args: args,
+      field: field.inspect,
+      context: ctx
+    }
+    Sentry.capture_exception(e, extra: extra)
+    # re-raise to be handled by rack middleware
+    raise
+    # or return execution error
+    ::GraphQL::ExecutionError.new(
+      exception.class.to_s,
+      options: { "http_status" => 500 },
+      extensions: {
+        "code" => exception.class.to_s,
+        "http_status" => 500,
+        "details" => exception.inspect
+      }
+    )
+  end
+end
+```
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/RenoFi/rack-graphql. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [Contributor Covenant](http://contributor-covenant.org) code of conduct.

data/Rakefile

@@ -5,4 +5,5 @@ require 'rubocop/rake_task'
 RSpec::Core::RakeTask.new(:spec)
 RuboCop::RakeTask.new
 
+task ci: %i[spec rubocop]
 task default: %i[spec rubocop:auto_correct]

data/lib/rack_graphql/application.rb

@@ -5,9 +5,11 @@ module RackGraphql
       app_name: 'rack-graphql-service',
       logger: nil,
       context_handler: nil,
+      re_raise_exceptions: false,
       log_exception_backtrace: RackGraphql.log_exception_backtrace,
       health_route: true,
       health_response_builder: RackGraphql::HealthResponseBuilder,
+      health_on_root_path: health_route,
       error_status_code_map: {}
     )
 
@@ -17,6 +19,7 @@ module RackGraphql
             app_name: app_name,
             schema: schema,
             context_handler: context_handler,
+            re_raise_exceptions: re_raise_exceptions,
             logger: logger,
             log_exception_backtrace: log_exception_backtrace,
             error_status_code_map: error_status_code_map,
@@ -31,7 +34,9 @@ module RackGraphql
           map '/healthz' do
             run ->(env) { health_response_builder.new(app_name: app_name, env: env).build }
           end
+        end
 
+        if health_on_root_path
           map '/' do
             run ->(env) { health_response_builder.new(app_name: app_name, env: env).build }
           end

data/lib/rack_graphql/middleware.rb

@@ -2,6 +2,7 @@ module RackGraphql
   class Middleware
     DEFAULT_STATUS_CODE = 200
     DEFAULT_ERROR_STATUS_CODE = 500
+    NULL_BYTE = '\u0000'.freeze
 
     def initialize(
       schema:,
@@ -9,6 +10,7 @@ module RackGraphql
       context_handler: nil,
       logger: nil,
       log_exception_backtrace: RackGraphql.log_exception_backtrace,
+      re_raise_exceptions: false,
       error_status_code_map: {}
     )
 
@@ -17,6 +19,7 @@ module RackGraphql
       @context_handler = context_handler || ->(_) {}
       @logger = logger
       @log_exception_backtrace = log_exception_backtrace
+      @re_raise_exceptions = re_raise_exceptions
       @error_status_code_map = error_status_code_map
     end
 
@@ -51,9 +54,14 @@ module RackGraphql
       ]
     rescue StandardError, LoadError, SyntaxError => e
       # To respect the graphql spec, all errors need to be returned as json.
-      # It needs to take Rack::ShowExceptions role of catching all exceptions raised by the app.
+      # By default exceptions are not re-raised,
+      # so they cannot be caught by error tracking rack middlewares.
+      # You can change this behavior via `re_raise_exceptions` argument.
       exception_string = dump_exception(e)
       log(exception_string)
+
+      raise e if re_raise_exceptions
+
       env[Rack::RACK_ERRORS].puts(exception_string)
       env[Rack::RACK_ERRORS].flush
       [
@@ -67,7 +75,8 @@ module RackGraphql
 
     private
 
-    attr_reader :schema, :app_name, :logger, :context_handler, :log_exception_backtrace, :error_status_code_map
+    attr_reader :schema, :app_name, :logger, :context_handler,
+      :log_exception_backtrace, :error_status_code_map, :re_raise_exceptions
 
     def post_request?(env)
       env['REQUEST_METHOD'] == 'POST'
@@ -75,7 +84,7 @@ module RackGraphql
 
     def post_data(env)
       payload = env['rack.input'].read.to_s
-      return nil if payload.index('\u0000')
+      return nil if payload.index(NULL_BYTE)
 
       ::Oj.load(payload)
     rescue Oj::ParseError

data/lib/rack_graphql/version.rb

@@ -1,3 +1,3 @@
 module RackGraphql
-  VERSION = '2.5.1'.freeze
+  VERSION = '2.7.1'.freeze
 end

data/lib/rack_graphql.rb

@@ -2,6 +2,10 @@ require 'oj'
 require 'rack'
 require 'graphql'
 
+# Timeout needs to be manually required before following fix is released:
+# https://github.com/rmosolgo/graphql-ruby/commit/56abba472dbb48a1f8445d41f928bea72b5148e9
+require 'timeout'
+
 require 'rack_graphql/version'
 require 'rack_graphql/exceptions'
 require 'rack_graphql/health_response_builder'

data/rack-graphql.gemspec

@@ -25,7 +25,7 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = '>= 2.6'
 
-  spec.add_dependency 'graphql', '~> 1.11'
+  spec.add_dependency 'graphql', '~> 1.12'
   spec.add_dependency 'oj'
   spec.add_dependency 'rack', '~> 2.2'
 
@@ -36,6 +36,6 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec'
   spec.add_development_dependency 'rubocop'
-  spec.add_development_dependency 'rubocop-performance'
+  spec.add_development_dependency 'rubocop-rake'
   spec.add_development_dependency 'rubocop-rspec'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rack-graphql
 version: !ruby/object:Gem::Version
-  version: 2.5.1
+  version: 2.7.1
 platform: ruby
 authors:
 - Krzysztof Knapik
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-11-19 00:00:00.000000000 Z
+date: 2021-10-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.11'
+        version: '1.12'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.11'
+        version: '1.12'
 - !ruby/object:Gem::Dependency
   name: oj
   requirement: !ruby/object:Gem::Requirement
@@ -152,7 +152,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop-performance
+  name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -187,6 +187,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".bundler-version"
 - ".dependabot/config.yml"
 - ".github/ISSUE_TEMPLATE/bug_report.md"
 - ".github/ISSUE_TEMPLATE/config.yml"
@@ -194,9 +195,9 @@ files:
 - ".github/PULL_REQUEST_TEMPLATE.md"
 - ".github/workflows/auto-approve.yml"
 - ".github/workflows/auto-merge.yml"
+- ".github/workflows/ci.yml"
 - ".gitignore"
 - ".rspec"
-- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - Gemfile.lock
@@ -233,7 +234,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.2.23
 signing_key:
 specification_version: 4
 summary: Rack middleware implementing graphql endpoint.

data/.travis.yml

@@ -1,11 +0,0 @@
----
-sudo: false
-language: ruby
-cache: bundler
-rvm:
-  - 2.6
-  - 2.7
-before_install: gem install bundler
-script:
-  - bundle exec rake rubocop
-  - bundle exec rake spec