기본적으로 rack-timeout을 사용할 때 시간 초과 요청을 하는 scope도 후속 요청에 적용할 수 있습니다

13576 단어 RailsRubyracktech
기본적으로 rack-timeout을 사용하는 사람은 clustered modeterm_on_timeout: true를 사용하고 Puma도 사용하고 있습니다.term_on_timeout: false(기본값) 상태에서rack-timeout을 사용하면 코드에 User.count라고 쓰여 있든 요청 전User.where(name: "foo")이든 계속 사용되며User.where(name: "foo").count될 수 있습니다.
다음 코드는 재현 코드입니다.
require "bundler/inline"

gemfile do
  source "https://rubygems.org"

  gem "rails", "6.1.3.1"
  gem "sqlite3"
  gem "rack-timeout", require: "rack/timeout/base"
end

require "rack/test"
require "active_record/railtie"
require "action_controller/railtie"

ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: ":memory:")
ActiveRecord::Base.logger = Logger.new(File::NULL)
ActiveRecord::Base.logger.level = Logger::ERROR

ActiveRecord::Schema.define do
  create_table :users, force: true do |t|
    t.string :name
  end
end

class User < ActiveRecord::Base
  scope :sleep_a_while, -> { all.tap { sleep(rand / 1000000.0) } }
end

User.create!(name: "foo")
User.create!(name: "bar")

class TestApp < Rails::Application
  config.root = __dir__
  config.hosts << "example.org"
  config.session_store :cookie_store, key: "cookie_store_key"
  secrets.secret_key_base = "secret_key_base"

  config.logger = ActiveRecord::Base.logger
  Rails.logger  = config.logger

  config.middleware.insert_before Rack::Runtime, Rack::Timeout, service_timeout: 0.1

  config.exceptions_app = ->(*) { [500, {}, ['']] }

  routes.draw do
    get "/" => "test#index"
    get "/timeout" => "test#timeout"
  end
end

class TestController < ActionController::Base
  include Rails.application.routes.url_helpers

  def index
    render plain: User.count
  end

  def timeout
    loop do
      User.where(name: "foo").sleep_a_while
    end
  end
end

require "minitest/autorun"

class BugTest < Minitest::Test
  include Rack::Test::Methods

  def test_index
    get "/"

    # Make sure User.count is 2
    assert_equal "2", last_response.body

    until last_response.body == "1"
      get "/timeout"

      get "/"
    end

    get "/"
    # User.count must be always 2 but it's now 1
    assert_equal "2", last_response.body
  end

  private
    def app
      Rails.application
    end
end
이것은 Timeout 시 ensure[1]를 실행하지 않을 수 있기 때문이다.
또한, Timeout 시 발생하는 문제에 대해 Rails는 고려하지 않는 것 같다[2].
또한rack-timeout은 역사상 예외에 불과하기 때문에 기본값을 바꾸지 않았다[3].
각주
https://qiita.com/mtsmfm/items/c967e0624bbafb25f2a3 ↩︎
https://github.com/rails/rails/pull/17607#issuecomment-70538060 ↩︎
https://github.com/sharpstone/rack-timeout/issues/169#issuecomment-806077437 ↩︎

좋은 웹페이지 즐겨찾기