Commit 2118ea43 authored by Sean McGivern's avatar Sean McGivern

Merge branch 'update-rack-timeout-take-2' into 'master'

Update `rack-timeout` and make it `require:`'d

See merge request gitlab-org/gitlab!36289
parents 26c00c8e b01f23bd
...@@ -164,6 +164,8 @@ gem 'diff_match_patch', '~> 0.1.0' ...@@ -164,6 +164,8 @@ gem 'diff_match_patch', '~> 0.1.0'
# Application server # Application server
gem 'rack', '~> 2.0.9' gem 'rack', '~> 2.0.9'
# https://github.com/sharpstone/rack-timeout/blob/master/README.md#rails-apps-manually
gem 'rack-timeout', '~> 0.5.1', require: 'rack/timeout/base'
group :unicorn do group :unicorn do
gem 'unicorn', '~> 5.5' gem 'unicorn', '~> 5.5'
...@@ -173,7 +175,6 @@ end ...@@ -173,7 +175,6 @@ end
group :puma do group :puma do
gem 'gitlab-puma', '~> 4.3.3.gitlab.2', require: false gem 'gitlab-puma', '~> 4.3.3.gitlab.2', require: false
gem 'gitlab-puma_worker_killer', '~> 0.1.1.gitlab.1', require: false gem 'gitlab-puma_worker_killer', '~> 0.1.1.gitlab.1', require: false
gem 'rack-timeout', require: false
end end
# State machine # State machine
......
...@@ -817,7 +817,7 @@ GEM ...@@ -817,7 +817,7 @@ GEM
rack rack
rack-test (1.1.0) rack-test (1.1.0)
rack (>= 1.0, < 3) rack (>= 1.0, < 3)
rack-timeout (0.5.1) rack-timeout (0.5.2)
rails (6.0.3.1) rails (6.0.3.1)
actioncable (= 6.0.3.1) actioncable (= 6.0.3.1)
actionmailbox (= 6.0.3.1) actionmailbox (= 6.0.3.1)
...@@ -1350,7 +1350,7 @@ DEPENDENCIES ...@@ -1350,7 +1350,7 @@ DEPENDENCIES
rack-cors (~> 1.0.6) rack-cors (~> 1.0.6)
rack-oauth2 (~> 1.9.3) rack-oauth2 (~> 1.9.3)
rack-proxy (~> 0.6.0) rack-proxy (~> 0.6.0)
rack-timeout rack-timeout (~> 0.5.1)
rails (~> 6.0.3.1) rails (~> 6.0.3.1)
rails-controller-testing rails-controller-testing
rails-i18n (~> 6.0) rails-i18n (~> 6.0)
......
---
title: Update `rack-timeout` to `0.5.2`
merge_request: 36289
author:
type: changed
...@@ -10,8 +10,6 @@ ...@@ -10,8 +10,6 @@
# logged and we should fix the potential timeout issue in the code itself. # logged and we should fix the potential timeout issue in the code itself.
if Gitlab::Runtime.puma? && !Rails.env.test? if Gitlab::Runtime.puma? && !Rails.env.test?
require 'rack/timeout/base'
Rack::Timeout::Logger.level = Logger::ERROR Rack::Timeout::Logger.level = Logger::ERROR
Gitlab::Application.configure do |config| Gitlab::Application.configure do |config|
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment