rack-mini-profiler 0.1.11.pre → 0.1.16

data/CHANGELOG

@@ -47,3 +47,32 @@
 10-August-2012 - Sam 
 
   * Added basic prepared statement profiling for postgres 
+
+20-August-2012 - Sam 
+  
+  * 1.12.pre 
+  * Cap X-MiniProfiler-Ids at 10, otherwise the header can get killed
+
+3-September-2012 - Sam
+
+  * 1.13.pre
+  * pg gem prepared statements were not being logged correctly 
+  * added setting config.backtrace_ignores = [] - an array of regexes that match on caller lines that get ignored
+  * added setting config.backtrace_includes = [] - an array of regexes that get included in the trace by default 
+  * cleaned up the way client settings are stored
+  * made pp=full-backtrace "sticky"
+  * added pp=normal-backtrace to clear the "sticky" state
+  * change "pp=sample" to work with "caller" no need for stack trace gem
+
+4-September-2012 - Sam 
+
+  * 1.15.pre
+  * fixed annoying bug where client settings were not sticking 
+  * fixed long standing issue with Rack::ConditionalGet stopping MiniProfiler from working properly
+
+5-September-2012 - Sam 
+
+  * 1.16
+  * fixed long standing problem specs (issue with memory store)
+  * fixed issue where profiler would be dumped when you got a 404 in production (and any time rails is bypassed)
+  * implemented stacktrace properly

data/README.md

@@ -87,7 +87,33 @@ In a Rails app, this can be done conveniently in an initializer such as config/i
 
 ## Rails 2.X support
 
-MiniProfiler uses [railtie](https://github.com/SamSaffron/MiniProfiler/blob/master/Ruby/lib/mini_profiler_rails/railtie.rb) to bootstrap itself. This will not be called in a Rails 2 app. You are going to need to hook it up manually. (TODO: document this - pull request please)
+To get MiniProfiler working with Rails 2.3.X you need to do the initialization manually as well as monkey patch away an incompatibility between activesupport and json_pure.
+
+Add the following code to your environment.rb (or just in a specific environment such as development.rb) for initialization and configuration of MiniProfiler.
+
+```ruby
+# configure and initialize MiniProfiler
+require 'rack-mini-profiler'
+c = ::Rack::MiniProfiler.config
+c.pre_authorize_cb = lambda { |env|
+  Rails.env.development? || Rails.env.production?
+}
+tmp = Rails.root.to_s + "/tmp/miniprofiler"
+FileUtils.mkdir_p(tmp) unless File.exists?(tmp)
+c.storage_options = {:path => tmp}
+c.storage = ::Rack::MiniProfiler::FileStore
+config.middleware.use(::Rack::MiniProfiler)
+::Rack::MiniProfiler.profile_method(ActionController::Base, :process) {|action| "Executing action: #{action}"}
+::Rack::MiniProfiler.profile_method(ActionView::Template, :render) {|x,y| "Rendering: #{@virtual_path}"}
+
+# monkey patch away an activesupport and json_pure incompatability
+# http://pivotallabs.com/users/alex/blog/articles/1332-monkey-patch-of-the-day-activesupport-vs-json-pure-vs-ruby-1-8
+if JSON.const_defined?(:Pure)
+  class JSON::Pure::Generator::State
+    include ActiveSupport::CoreExtensions::Hash::Except
+  end
+end
+```
 
 ## Available Options
 

data/lib/html/includes.css

@@ -58,7 +58,7 @@
 .profiler-results .profiler-popup .profiler-timings th,.profiler-results .profiler-popup .profiler-timings td{padding-left:6px;padding-right:6px;}
 .profiler-results .profiler-popup .profiler-timings th{font-size:95%;padding-bottom:3px;}
 .profiler-results .profiler-popup .profiler-timings .profiler-label{max-width:275px;}
-.profiler-results .profiler-queries{display:none;z-index:2147483643;position:absolute;overflow-y:auto;overflow-x:hidden;background-color:#fff;}.profiler-results .profiler-queries th{font-size:17px;}
+.profiler-results .profiler-queries{display:none;z-index:2147483643;position:absolute;overflow-y:auto;overflow-x:auto;background-color:#fff;}.profiler-results .profiler-queries th{font-size:17px;}
 .profiler-results.profiler-min .profiler-result{display:none;}
 .profiler-results.profiler-min .profiler-controls span{display:none;}
 .profiler-results.profiler-min .profiler-controls .profiler-min-max{border-right:none;padding:0px;margin:0px;}

data/lib/html/includes.js

@@ -5,7 +5,8 @@ var MiniProfiler = (function ($) {
         container,
         controls,
         fetchedIds = [],
-        fetchingIds = []  // so we never pull down a profiler twice
+        fetchingIds = [], // so we never pull down a profiler twice
+        ajaxStartTime
         ;
 
     var hasLocalStorage = function () {
@@ -73,15 +74,15 @@ var MiniProfiler = (function ($) {
             clientPerformance = null;
             clientProbes = null;
 
-            if (id == options.currentId) {
-
-                if (window.mPt) {
-                    clientProbes = mPt.t;
-                    for (j = 0; j < clientProbes.length; j++) {
-                        clientProbes[j].d = clientProbes[j].d.getTime();
-                    }
-                    mPt.t = [];
+            if (window.mPt) {
+                clientProbes = mPt.results();
+                for (j = 0; j < clientProbes.length; j++) {
+                    clientProbes[j].d = clientProbes[j].d.getTime();
                 }
+                mPt.flush();
+            }
+            
+            if (id == options.currentId) {
 
                 clientPerformance = getClientPerformance();
 
@@ -113,6 +114,9 @@ var MiniProfiler = (function ($) {
 
                     }
                 }
+            } else if (ajaxStartTime != null && clientProbes && clientProbes.length > 0) {
+                clientPerformance = { timing: { navigationStart: ajaxStartTime.getTime() } };
+                ajaxStartTime = null;
             }
 
             if ($.inArray(id, fetchedIds) < 0 && $.inArray(id, fetchingIds) < 0) {
@@ -449,6 +453,9 @@ var MiniProfiler = (function ($) {
             jQuery(document).ajaxComplete(jQueryAjaxComplete);
         }
 
+        if (jQuery(document).ajaxStart)
+            jQuery(document).ajaxStart(function () { ajaxStartTime = new Date(); });
+        
         // fetch results after ASP Ajax calls
         if (typeof (Sys) != 'undefined' && typeof (Sys.WebForms) != 'undefined' && typeof (Sys.WebForms.PageRequestManager) != 'undefined') {
             // Get the instance of PageRequestManager.

data/lib/html/includes.less

@@ -359,7 +359,7 @@
         z-index:@zindex + 3;
         position:absolute;
         overflow-y:auto;
-        overflow-x:hidden;
+        overflow-x:auto;
         background-color:#fff;
 
         th {

data/lib/html/includes.tmpl

@@ -138,7 +138,7 @@
     </td>
 
   {{if HasSqlTimings}}
-    <td class="profiler-duration {{if HasDuplicateSqlTimings}}profiler-warning{{/if}}" title="{{if HasDuplicateSqlTimings}}duplicate queries detected - {{/if}}${ExecutedReaders} reader, ${ExecutedScalars} scalar, ${ExecutedNonQueries} non-query statements executed">
+    <td class="profiler-duration {{if HasDuplicateSqlTimings}}profiler-warning{{/if}}" title="{{if HasDuplicateSqlTimings}}duplicate queries detected - {{/if}}{{if ExecutedReaders > 0 || ExecutedScalars > 0 || ExecutedNonQueries > 0}}${ExecutedReaders} reader, ${ExecutedScalars} scalar, ${ExecutedNonQueries} non-query statements executed{{/if}}">
       <a class="profiler-queries-show">
         {{if HasDuplicateSqlTimings}}<span class="profiler-nuclear">!</span>{{/if}}
         ${SqlTimings.length} <span class="profiler-unit">sql</span>
@@ -192,4 +192,4 @@
     </td>
   </tr>
 
-</script>
+</script>

data/lib/mini_profiler/client_settings.rb

@@ -0,0 +1,65 @@
+module Rack
+  class MiniProfiler
+    class ClientSettings
+      
+      COOKIE_NAME = "__profilin"
+
+      BACKTRACE_DEFAULT = nil
+      BACKTRACE_FULL = 1 
+      BACKTRACE_NONE = 2
+
+      attr_accessor :disable_profiling
+      attr_accessor :backtrace_level
+
+      
+      def initialize(env)
+        request = ::Rack::Request.new(env)
+        @cookie = request.cookies[COOKIE_NAME] 
+        if @cookie
+          @cookie.split(",").map{|pair| pair.split("=")}.each do |k,v|
+            @orig_disable_profiling = @disable_profiling = (v=='t') if k == "dp"
+            @backtrace_level = v.to_i if k == "bt"
+          end
+        end
+
+        @backtrace_level = nil if !@backtrace_level.nil? && (@backtrace_level == 0 || @backtrace_level > BACKTRACE_NONE) 
+        @orig_backtrace_level = @backtrace_level
+
+      end
+
+      def write!(headers)
+        if @orig_disable_profiling != @disable_profiling || @orig_backtrace_level != @backtrace_level || @cookie.nil?
+          settings = {"p" =>  "t" }
+          settings["dp"] = "t" if @disable_profiling 
+          settings["bt"] = @backtrace_level if @backtrace_level
+          settings_string = settings.map{|k,v| "#{k}=#{v}"}.join(",")
+          Rack::Utils.set_cookie_header!(headers, COOKIE_NAME, :value => settings_string, :path => '/')
+        end
+      end
+
+      def discard_cookie!(headers)
+        Rack::Utils.delete_cookie_header!(headers, COOKIE_NAME, :path => '/')
+      end
+
+      def has_cookie?
+        !@cookie.nil?
+      end
+
+      def disable_profiling? 
+        @disable_profiling
+      end
+
+      def backtrace_full? 
+        @backtrace_level == BACKTRACE_FULL
+      end
+
+      def backtrace_default? 
+        @backtrace_level == BACKTRACE_DEFAULT
+      end
+      
+      def backtrace_none? 
+        @backtrace_level == BACKTRACE_NONE
+      end
+    end
+  end
+end

data/lib/mini_profiler/config.rb

@@ -13,7 +13,7 @@ module Rack
     end
 
     attr_accessor :auto_inject, :base_url_path, :pre_authorize_cb, :position,
-        :backtrace_remove, :backtrace_filter, :skip_schema_queries, 
+        :backtrace_remove, :backtrace_includes, :backtrace_ignores, :skip_schema_queries, 
         :storage, :user_provider, :storage_instance, :storage_options, :skip_paths, :authorization_mode, :use_existing_jquery
       
       def self.default

data/lib/mini_profiler/profiler.rb

@@ -13,12 +13,13 @@ require 'mini_profiler/storage/file_store'
 require 'mini_profiler/config'
 require 'mini_profiler/profiling_methods'
 require 'mini_profiler/context'
+require 'mini_profiler/client_settings'
 
 module Rack
 
 	class MiniProfiler
 
-		VERSION = '104'.freeze
+		VERSION = '107'.freeze
 
     class << self 
       
@@ -56,35 +57,6 @@ module Rack
         self.current.discard = true if current
       end
 
-      # user has the mini profiler cookie, only used when config.authorization_mode == :whitelist
-      def has_profiling_cookie?(env)
-        env['HTTP_COOKIE'] && env['HTTP_COOKIE'].include?("__profilin=stylin")
-      end
-
-      # remove the mini profiler cookie, only used when config.authorization_mode == :whitelist
-      def remove_profiling_cookie(headers)
-        Rack::Utils.set_cookie_header!(headers, '__profilin', :value => 'notstylin', :path => '/')
-      end
-
-      def set_profiling_cookie(headers)
-        Rack::Utils.set_cookie_header!(headers, '__profilin', :value => 'stylin', :path => '/')
-      end
-
-      # user has the mini profiler cookie, only used when config.authorization_mode == :whitelist
-      def has_disable_profiling_cookie?(env)
-        env['HTTP_COOKIE'] && env['HTTP_COOKIE'].include?("__profilin_disable=stylin")
-      end
-
-      # remove the mini profiler cookie, only used when config.authorization_mode == :whitelist
-      def remove_disable_profiling_cookie(headers)
-        #something is odd with delete_cookie_header
-        Rack::Utils.set_cookie_header!(headers, '__profilin_disable', :value => 'notstylin', :path => '/')
-      end
-
-      def set_disable_profiling_cookie(headers)
-        Rack::Utils.set_cookie_header!(headers, '__profilin_disable', :value => 'stylin', :path => '/')
-      end
-
       def create_current(env={}, options={})
         # profiling the request
         self.current = Context.new
@@ -195,6 +167,9 @@ module Rack
 
 
 		def call(env)
+
+      client_settings = ClientSettings.new(env)
+
       status = headers = body = nil
       query_string = env['QUERY_STRING']
       path = env['PATH_INFO']
@@ -203,12 +178,12 @@ module Rack
                 (@config.skip_paths && @config.skip_paths.any?{ |p| path[0,p.length] == p}) ||
                 query_string =~ /pp=skip/ 
       
-      has_profiling_cookie = MiniProfiler.has_profiling_cookie?(env)
+      has_profiling_cookie = client_settings.has_cookie?
     
       if skip_it || (@config.authorization_mode == :whitelist && !has_profiling_cookie)
         status,headers,body = @app.call(env)
         if !skip_it && @config.authorization_mode == :whitelist && !has_profiling_cookie && MiniProfiler.request_authorized? 
-          MiniProfiler.set_profiling_cookie(headers) 
+          client_settings.write!(headers) 
         end
         return [status,headers,body]
       end
@@ -216,7 +191,7 @@ module Rack
       # handle all /mini-profiler requests here
 			return serve_html(env) if path.start_with? @config.base_url_path
 
-      has_disable_cookie = MiniProfiler.has_disable_profiling_cookie?(env)
+      has_disable_cookie = client_settings.disable_profiling?
       # manual session disable / enable
       if query_string =~ /pp=disable/ || has_disable_cookie
         skip_it = true
@@ -228,38 +203,52 @@ module Rack
 
       if skip_it
         status,headers,body = @app.call(env)
-        MiniProfiler.set_disable_profiling_cookie(headers) unless has_disable_cookie
+        client_settings.disable_profiling = true
+        client_settings.write!(headers)
         return [status,headers,body]
       end
 
       MiniProfiler.create_current(env, @config)
       MiniProfiler.deauthorize_request if @config.authorization_mode == :whitelist
-      if query_string =~ /pp=no-backtrace/
+      if query_string =~ /pp=normal-backtrace/
+        client_settings.backtrace_level = ClientSettings::BACKTRACE_DEFAULT
+      elsif query_string =~ /pp=no-backtrace/
         current.skip_backtrace = true
-      elsif query_string =~ /pp=full-backtrace/
+        client_settings.backtrace_level = ClientSettings::BACKTRACE_NONE
+      elsif query_string =~ /pp=full-backtrace/ || client_settings.backtrace_full?
         current.full_backtrace = true
+        client_settings.backtrace_level = ClientSettings::BACKTRACE_FULL
+      elsif client_settings.backtrace_none?
+        current.skip_backtrace = true
       end
 
       done_sampling = false
       quit_sampler = false
       backtraces = nil
-      missing_stacktrace = false
+      stacktrace_installed = true
       if query_string =~ /pp=sample/
+        skip_frames = 0
         backtraces = []
         t = Thread.current
+        
+        begin 
+          require 'stacktrace'
+          skip_frames = stacktrace.length
+        rescue
+          stacktrace_installed = false
+        end
+
         Thread.new {
           begin
-            require 'stacktrace' rescue nil
-            if !t.respond_to? :stacktrace
-              missing_stacktrace = true
-              quit_sampler = true 
-              return
-            end
             i = 10000 # for sanity never grab more than 10k samples 
             while i > 0
               break if done_sampling
               i -= 1
-              backtraces << t.stacktrace
+              if stacktrace_installed
+                backtraces << t.stacktrace(0,-(1+skip_frames), StackFrame::Flags::METHOD | StackFrame::Flags::KLASS)
+              else
+                backtraces << t.backtrace 
+              end
               sleep 0.001
             end
           ensure
@@ -271,10 +260,14 @@ module Rack
 			status, headers, body = nil
       start = Time.now 
       begin 
+
+        # Strip all the caching headers so we don't get 304s back 
+        #  This solves a very annoying bug where rack mini profiler never shows up
+        env['HTTP_IF_MODIFIED_SINCE'] = nil
+        env['HTTP_IF_NONE_MATCH'] = nil
+
         status,headers,body = @app.call(env)
-        if has_disable_cookie
-          MiniProfiler.remove_disable_profiling_cookie(headers)
-        end
+        client_settings.write!(headers)
       ensure
         if backtraces 
           done_sampling = true
@@ -284,10 +277,14 @@ module Rack
 
       skip_it = current.discard
       if (config.authorization_mode == :whitelist && !MiniProfiler.request_authorized?)
-        MiniProfiler.remove_profiling_cookie(headers)
+        # this is non-obvious, don't kill the profiling cookie on errors or short requests
+        # this ensures that stuff that never reaches the rails stack does not kill profiling
+        if status == 200 && ((Time.now - start) > 0.1) 
+          client_settings.discard_cookie!(headers)
+        end
         skip_it = true
       end
-      
+
       return [status,headers,body] if skip_it
 
       # we must do this here, otherwise current[:discard] is not being properly treated
@@ -298,7 +295,7 @@ module Rack
 
       if query_string =~ /pp=help/
         body.close if body.respond_to? :close
-        return help
+        return help(nil, client_settings)
       end
       
       page_struct = current.page_struct
@@ -306,7 +303,6 @@ module Rack
 
       if backtraces
         body.close if body.respond_to? :close
-        return help(:stacktrace) if missing_stacktrace
         return analyze(backtraces, page_struct)
       end
       
@@ -317,6 +313,8 @@ module Rack
 			
       # inject headers, script
 			if status == 200
+
+        client_settings.write!(headers)
         
         # mini profiler is meddling with stuff, we can not cache cause we will get incorrect data
         # Rack::ETag has already inserted some nonesense in the chain
@@ -346,6 +344,7 @@ module Rack
 				end
 			end
 
+      client_settings.write!(headers)
 			[status, headers, body]
     ensure
       # Make sure this always happens
@@ -373,16 +372,17 @@ module Rack
       [200, headers, [body]]
     end
 
-    def help(category = nil)
+    def help(category = nil, client_settings)
       headers = {'Content-Type' => 'text/plain'}
       body = "Append the following to your query string:
 
   pp=help : display this screen
   pp=env : display the rack environment
   pp=skip : skip mini profiler for this request
-  pp=no-backtrace : don't collect stack traces from all the SQL executed
-  pp=full-backtrace : enable full backtrace for SQL executed
-  pp=sample : sample stack traces and return a report isolating heavy usage (requires the stacktrace gem)
+  pp=no-backtrace #{"(*) " if client_settings.backtrace_none?}: don't collect stack traces from all the SQL executed (sticky, use pp=normal-backtrace to enable)
+  pp=normal-backtrace #{"(*) " if client_settings.backtrace_default?}: collect stack traces from all the SQL executed and filter normally
+  pp=full-backtrace #{"(*) " if client_settings.backtrace_full?}: enable full backtraces for SQL executed (use pp=normal-backtrace to disable) 
+  pp=sample : sample stack traces and return a report isolating heavy usage (experimental works best with the stacktrace gem)
   pp=disable : disable profiling for this session 
   pp=enable : enable profiling for this session (if previously disabled)
 "
@@ -390,6 +390,7 @@ module Rack
         body = "pp=stacktrace requires the stacktrace gem - add gem 'stacktrace' to your Gemfile"
       end
     
+      client_settings.write!(headers)
       [200, headers, [body]]
     end
 
@@ -403,13 +404,13 @@ module Rack
         fulldump << "\n\n"
         distinct = {}
         trace.each do |frame|
-          name = "#{frame.klass} #{frame.method}"
-          unless distinct[name]
-            distinct[name] = true
-            seen[name] ||= 0
-            seen[name] += 1
+          frame = "#{frame.klass}::#{frame.method}" unless String === frame
+          unless distinct[frame]
+            distinct[frame] = true
+            seen[frame] ||= 0
+            seen[frame] += 1
           end
-          fulldump << name << "\n"
+          fulldump << frame << "\n"
         end
       end
 
@@ -427,7 +428,8 @@ module Rack
     end
 
     def ids_json(env)
-      ids = [current.page_struct["Id"]] + (@storage.get_unviewed_ids(user(env)) || [])
+      # cap at 10 ids, otherwise there is a chance you can blow the header
+      ids = [current.page_struct["Id"]] + (@storage.get_unviewed_ids(user(env)) || [])[0..8]
       ::JSON.generate(ids.uniq)
     end
 

data/lib/mini_profiler/profiling_methods.rb

@@ -9,7 +9,7 @@ module Rack
 		  end
 
       # perform a profiling step on given block
-      def step(name)
+      def step(name, opts = nil)
         if current
           parent_timer = current.current_timer
           result = nil
@@ -20,15 +20,17 @@ module Rack
             current_timer.record_time
             current.current_timer = parent_timer
           end
-          result
         else
           yield if block_given?
         end
       end
 
       def unprofile_method(klass, method)
-        with_profiling = (method.to_s + "_with_mini_profiler").intern
-        without_profiling = (method.to_s + "_without_mini_profiler").intern
+
+        clean = clean_method_name(method)
+
+        with_profiling = ("#{clean}_with_mini_profiler").intern
+        without_profiling = ("#{clean}_without_mini_profiler").intern
         
         if klass.send :method_defined?, with_profiling
           klass.send :alias_method, method, without_profiling
@@ -39,8 +41,10 @@ module Rack
 
       def profile_method(klass, method, &blk)
         default_name = klass.to_s + " " + method.to_s
-        with_profiling = (method.to_s + "_with_mini_profiler").intern
-        without_profiling = (method.to_s + "_without_mini_profiler").intern
+        clean = clean_method_name(method)
+
+        with_profiling =  ("#{clean}_with_mini_profiler").intern
+        without_profiling = ("#{clean}_without_mini_profiler").intern
         
         if klass.send :method_defined?, with_profiling
           return # dont double profile
@@ -68,6 +72,13 @@ module Rack
         end
         klass.send :alias_method, method, with_profiling
       end
+
+      private
+
+      def clean_method_name(method)
+        method.to_s.gsub(/[\?\!]/, "")
+      end
+
     end
   end
 end

data/lib/mini_profiler/sql_timer_struct.rb

@@ -14,7 +14,17 @@ module Rack
            # Clean up the stack trace if there are options to do so
           Kernel.caller.each do |ln|
             ln.gsub!(Rack::MiniProfiler.config.backtrace_remove, '') if Rack::MiniProfiler.config.backtrace_remove and !full_backtrace
-            if full_backtrace or Rack::MiniProfiler.config.backtrace_filter.nil? or ln =~ Rack::MiniProfiler.config.backtrace_filter
+            if    full_backtrace or 
+                  (
+                    (
+                      Rack::MiniProfiler.config.backtrace_includes.nil? or 
+                      Rack::MiniProfiler.config.backtrace_includes.all?{|regex| ln =~ regex}
+                    ) and
+                    (
+                      Rack::MiniProfiler.config.backtrace_ignores.nil? or 
+                      Rack::MiniProfiler.config.backtrace_ignores.all?{|regex| !(ln =~ regex)}
+                    )
+                  ) 
               stack_trace << ln << "\n" 
             end
           end

data/lib/mini_profiler/storage/file_store.rb

@@ -13,7 +13,6 @@ module Rack
             data = ::File.open(path(key),"rb") {|f| f.read}
             return Marshal.load data
           rescue => e
-            p e
             return nil
           end
         end

data/lib/mini_profiler/storage/memory_store.rb

@@ -55,10 +55,9 @@ module Rack
       def cleanup_cache
         expire_older_than = ((Time.now.to_f - MiniProfiler::MemoryStore::EXPIRE_TIMER_CACHE) * 1000).to_i
         @timer_struct_lock.synchronize {
-          @timer_struct_cache.delete_if { |k, v| v['Root']['StartMilliseconds'] < expire_older_than }
+          @timer_struct_cache.delete_if { |k, v| v['Started'] < expire_older_than }
         }
       end
-    
     end
   end
 end

data/lib/mini_profiler_rails/railtie.rb

@@ -31,7 +31,7 @@ module MiniProfilerRails
 
       # Quiet the SQL stack traces
       c.backtrace_remove = Rails.root.to_s + "/"
-      c.backtrace_filter =  /^\/?(app|config|lib|test)/
+      c.backtrace_includes =  [/^\/?(app|config|lib|test)/]
       c.skip_schema_queries =  Rails.env != 'production'
 
       # Install the Middleware

data/lib/patches/sql_patches.rb

@@ -96,17 +96,18 @@ if SqlPatches.class_exists? "PG::Result"
     alias_method :prepare_without_profiling, :prepare
 
     def prepare(*args,&blk)
-      current = ::Rack::MiniProfiler.current
-      return prepare_without_profiling(*args,&blk) unless current
-
+      # we have no choice but to do this here, 
+      # if we do the check for profiling first, our cache may miss critical stuff  
+      
       @prepare_map ||= {}
       @prepare_map[args[0]] = args[1]
-
       # dont leak more than 10k ever
-      @prepare_map = {} if @prepare_map.length > 10000
+      @prepare_map = {} if @prepare_map.length > 1000
 
-      prepare_without_profiling(*args,&blk) 
+      current = ::Rack::MiniProfiler.current
+      return prepare_without_profiling(*args,&blk) unless current
 
+      prepare_without_profiling(*args,&blk) 
     end
 
     def exec(*args,&blk)

data/rack-mini-profiler.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
 	s.name = "rack-mini-profiler"
-	s.version = "0.1.11.pre"
+	s.version = "0.1.16"
 	s.summary = "Profiles loading speed for rack applications."
 	s.authors = ["Aleks Totic","Sam Saffron", "Robin Ward"]
 	s.description = "Page loading speed displayed on every page. Optimize while you develop, performance is a feature."

metadata

@@ -1,8 +1,8 @@
 --- !ruby/object:Gem::Specification
 name: rack-mini-profiler
 version: !ruby/object:Gem::Version
-  version: 0.1.11.pre
-  prerelease: 7
+  version: 0.1.16
+  prerelease: 
 platform: ruby
 authors:
 - Aleks Totic
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-08-10 00:00:00.000000000 Z
+date: 2012-09-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -91,6 +91,7 @@ files:
 - lib/mini_profiler/request_timer_struct.rb
 - lib/mini_profiler/timer_struct.rb
 - lib/mini_profiler/page_timer_struct.rb
+- lib/mini_profiler/client_settings.rb
 - lib/mini_profiler/context.rb
 - lib/mini_profiler/config.rb
 - lib/mini_profiler/profiling_methods.rb
@@ -130,13 +131,16 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -126757873
+      hash: 1044164429
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
-  - - ! '>'
+  - - ! '>='
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
+      segments:
+      - 0
+      hash: 1044164429
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24