Issue 70 - Make Rack::Cache multithread friendly

[jtblin]

As described in #70, this will fix the issue with asstes not being loaded correctly when using puma, a multithreaded ruby app server. This changes /lib/rack/cache/context.rb call method from

def call(env)
  if env['rack.run_once']
    call! env
  else
    clone.call! env
  end
end

to

def call(env)
  if env['rack.run_once'] && !env['rack.multithread']
    call! env
  else
    clone.call! env
  end
end

Let me know what you think.

[rtomayko]

Seems sane enough. Why would rack.run_once be set in that case though?

[rtomayko]

Sorry, I missed the description in #70. It still seems really wrong that puma would set run_once true but I guess we'll go with it. The run_once path is mostly there for tests. This should be fine.

[jtblin]

Yes you're correct that puma should not set rack.run_once to true, they will actually change that in 1.0.7 but it doesn't hurt to test for rack.multithread anyways. Thanks for accepting the pull request!

[Gibheer]

Hi,

can you please do a release with that fix? I also hit this bug and search two days for a solution.

thank you

[amacneil]

👍 please publish a gem version with this patch included