access_logging 0.0.2 → 0.0.3

data/lib/access_logging/model.rb

@@ -2,46 +2,52 @@ require 'active_support/concern'
 
 module AccessLogging::Model
   extend ActiveSupport::Concern
-  
+
   include DateRangeScopes
-  
+
   REDIS_ATTRIBUTES = [
     :created_at,
-    :ip, :user_type, :user_id, 
+    :ip, :user_type, :user_id,
     :verb, :model_type, :model_id, :description,
     :path
   ]
-  
+
   REDIS_SEPARATOR = "||"
-  
+
   included do
     belongs_to :user, polymorphic: true
     belongs_to :model, polymorphic: true
-    
+
     before_validation :set_user_name_and_email, :set_description
-    
+
     validates_presence_of :ip, :path, :verb,
                           :user_name, :user_email, :user_id, :user_type,
                           :created_at
-    
+
     default_scope order('access_logs.created_at DESC')
   end
-  
+
   module ClassMethods
     def log_request(user, request, opts={})
       log = AccessLog.new
       log.user = user
-      log.path = request.fullpath
       log.ip = request.remote_ip
-      
+
+      if [:get, :delete].include?(request.method_symbol)
+        log.path = request.fullpath
+      else
+        # Don't log params on POST or PUT requests
+        log.path = request.fullpath.split("?")[0]
+      end
+
       log.model = opts[:model]
       log.verb = opts[:verb] || default_verb_for_method(request.request_method)
-      
+
       return if log.model && log.model.new_record?
-      
+
       log.save!
     end
-  
+
     def default_verb_for_method(method)
       case method.to_s.upcase
       when 'GET'
@@ -54,33 +60,33 @@ module AccessLogging::Model
         "deleted"
       end
     end
-  
+
     def build_from_redis_string(str)
       log = AccessLog.new
       parts = str.split(REDIS_SEPARATOR, -1)
-    
+
       REDIS_ATTRIBUTES.each_with_index do |attribute, i|
         log.send "#{attribute}=", parts[i]
       end
-    
+
       log
     end
   end
-  
+
   def created_at
     self[:created_at] ||= Time.zone.now
   end
-  
+
   def redis_string
     REDIS_ATTRIBUTES.map { |attribute| send(attribute) }.join(REDIS_SEPARATOR)
   end
-  
-  
+
+
   private
-  
+
   def set_description
     self.description = "" and return unless model && !model.new_record?
-    
+
     if model.respond_to?(:description)
       self.description = model.description
     elsif model.respond_to?(:name)
@@ -89,7 +95,7 @@ module AccessLogging::Model
       self.description = "#{model.class.name} #{model.id}"
     end
   end
-  
+
   def set_user_name_and_email
     self.user_name = user.try :name
     self.user_email = user.try :email

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: access_logging
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-11-29 00:00:00.000000000Z
+date: 2012-03-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &70220242243000 !ruby/object:Gem::Requirement
+  requirement: &70200272177340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: '3'
   type: :runtime
   prerelease: false
-  version_requirements: *70220242243000
+  version_requirements: *70200272177340
 - !ruby/object:Gem::Dependency
   name: date_range_scopes
-  requirement: &70220242242600 !ruby/object:Gem::Requirement
+  requirement: &70200272172220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70220242242600
+  version_requirements: *70200272172220
 - !ruby/object:Gem::Dependency
   name: sqlite3
-  requirement: &70220242242140 !ruby/object:Gem::Requirement
+  requirement: &70200272170680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,7 +43,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70220242242140
+  version_requirements: *70200272170680
 description: Log access to models through your controllers.
 email: nick.ragaz@gmail.com
 executables: []
@@ -76,7 +76,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.10
+rubygems_version: 1.8.11
 signing_key: 
 specification_version: 3
 summary: Log access to models through your controllers.