Caller context key strategy totally borks if Garner::Cache::Context#garner is monkey-patched

[fancyremarker]

Let's say I do the following

# config/initializers/garner.rb

module Garner
  module Mixins
    module Rack

      alias_method :_garner, :garner
      def garner(&block)
        response, headers = _garner.key({ foo: "bar" }, &block)
      end
    end
  end
end

Now the calling line number will always be config/initializers/garner.rb:9, instead of where the method was originally called from.

Can we be smarter about this?

[dblock]

Possibly by examining class names on the stack instead of file names?

[monfresh]

Is this related to issue #54 ? Is that monkey patch not safe to use in production? It seems to be working fine for me.

[fancyremarker]

@monfresh This is unrelated; the patch from #54 is safe since it patches fetch, not garner.