rackamole 0.2.9 → 0.3.0

data/lib/rackamole.rb

@@ -1,7 +1,7 @@
 module Rackamole
 
   # :stopdoc:
-  VERSION = '0.2.9'
+  VERSION = '0.3.0'
   LIBPATH = ::File.expand_path(::File.dirname(__FILE__)) + ::File::SEPARATOR
   PATH = ::File.dirname(LIBPATH) + ::File::SEPARATOR
   # :startdoc:

data/lib/rackamole/mole.rb

@@ -82,7 +82,7 @@ module Rack
     def call( env )      
       # Bail if application is not moleable
       return @app.call( env ) unless moleable?
-                  
+                
       @stash = env['mole.stash'] if env['mole.stash']      
       @stash = Rackamole::Stash::Collector.new( options[:app_name], options[:environment], options[:expiration] ) unless stash
             
@@ -158,7 +158,7 @@ module Rack
       # send info to configured store
       options[:store].mole( attrs )
       
-      # Check for dups. If we've logged this req before don't log it again...
+      # Check for dups. If we've reported this req before don't report it again...
       unless duplicated?( env, attrs )
         # send email alert ?
         if alertable?( :email, attrs[:type] )
@@ -289,11 +289,15 @@ module Rack
       # Gather up browser and client machine info
       agent_info = Rackamole::Utils::AgentDetect.parse( user_agent )
       %w(browser machine).each { |type| mole?(type.to_sym, info, agent_info[type.to_sym] ) }
-      
+
       # Dump request params
       unless request.params.empty?
         info[:params] = filter_params( request.params, options[:param_excludes] || [] )
       end
+      if route
+        info[:params] ||= {}
+        info[:params].merge!( filter_params( params_from_route( route ), options[:param_excludes] || [] ) )
+      end
             
       # Dump session var
       if session and !session.empty?
@@ -312,6 +316,17 @@ module Rack
       info
     end
     
+    # Parse out rails request params if in rails env
+    def params_from_route( route )
+      params = {}
+      except = [:controller, :action]
+      route.each_pair do |k,v|
+        next if except.include?( k )
+        params[k] = v
+      end
+      params
+    end
+    
     # check exclusion to see if the information should be moled
     def mole?( key, stash, value )
 # puts "Checking #{key} -- #{options[:mole_excludes].inspect}"      

data/spec/rackamole/mole_spec.rb

@@ -283,6 +283,20 @@ describe Rack::Mole do
       res[:controller].should == 'fred'
       res[:action].should     == 'blee'
     end
+    
+    it "should extract request parameters correctly" do
+      rack = Rack::Mole.new( nil, :app_name => "test app" )
+      res = rack.send( :params_from_route, {:controller => "blee", :action => "fred", :bobo => "hello" } )
+      res.should_not be_nil
+      res.should have(1).item
+      res.should == { :bobo => "hello" }
+    end
+    
+    it "should not pick up params if none are specified" do
+      rack = Rack::Mole.new( nil, :app_name => "test app" )
+      res = rack.send( :params_from_route, {:controller => "blee", :action => "fred" } )
+      res.should be_empty
+    end    
   end
   
   # ---------------------------------------------------------------------------      

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: rackamole
 version: !ruby/object:Gem::Version 
-  version: 0.2.9
+  version: 0.3.0
 platform: ruby
 authors: 
 - Fernand Galiana
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-01-31 00:00:00 -07:00
+date: 2010-02-05 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency