pg_audit_log 0.5.6 → 0.6.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 8f7c84ef450326cc585d0ffdf7963a07ba9e2341
+  data.tar.gz: a61e2f9ab3687d5e2afb14b6542adf56767c01af
+SHA512:
+  metadata.gz: ce57db6aee49eb73c8b0e739a6be62281f361b79ddf2290108ceb7374d86b345c0342b81e527714b71b1ed0dda6dd1b6a047b23c643e91d2efd7221a31551ea1
+  data.tar.gz: a2e21fefdf61ea1ec1865d020c3c4f38382e8c4f1a664c2e28b5305e198656ae522fd82f528c410c00f03a6d6f069d4ea7fd1289beff0d8490c9157a960e89f2

data/.travis.yml

@@ -0,0 +1,28 @@
+language: ruby
+
+addons:
+  postgresql: '9.3'
+
+before_script:
+  - psql -c 'CREATE DATABASE pg_audit_log_test;' -U postgres
+
+rvm:
+  - 1.9.3
+  - 2.0.0
+  - 2.1.1
+  #- jruby-19mode
+  - rbx-2.2
+
+env:
+  #- RAILS_BRANCH="master"
+  #- RAILS_BRANCH="4-1-stable"
+  #- RAILS_BRANCH="4-0-stable"
+  - RAILS_VERSION="~> 4.1.0"
+  - RAILS_VERSION="~> 4.0.0"
+  - RAILS_VERSION="~> 3.2.0"
+
+matrix:
+  allow_failures:
+    - env: RAILS_BRANCH="master"
+    - env: RAILS_BRANCH="4-1-stable"
+    - env: RAILS_BRANCH="4-0-stable"

data/Gemfile

@@ -1,9 +1,7 @@
-source :rubygems
+source 'https://rubygems.org'
 
 # Specify your gem's dependencies the .gemspec
 gemspec
 
-group :development do
-  gem 'wirble'
-end
-
+gem 'rails', :github => 'rails', :branch => ENV['RAILS_RECORD_BRANCH'] if ENV['ACTIVE_RECORD_BRANCH']
+gem 'rails', ENV['RAILS_VERSION'] if ENV['RAILS_RECORD_VERSION']

data/LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2009-11 Case Commons, LLC
+Copyright (c) 2009-2014 Case Commons, LLC
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -0,0 +1,64 @@
+# pg_audit_log
+
+[![Build Status](https://secure.travis-ci.org/Casecommons/pg_audit_log.png?branch=master)](https://travis-ci.org/Casecommons/pg_audit_log)
+[![Code Climate](https://codeclimate.com/github/Casecommons/pg_audit_log.png)](https://codeclimate.com/github/Casecommons/pg_audit_log)
+[![Gem Version](https://badge.fury.io/rb/pg_audit_log.png)](https://rubygems.org/gems/pg_audit_log)
+
+## Description
+
+PostgreSQL-only database-level audit logging of all databases changes using a completely transparent stored procedure and triggers.
+Comes with specs for your project and a rake task to generate the reverse SQL to undo changes logged.
+
+All SQL `INSERT`s, `UPDATE`s, and `DELETE`s will be captured. Record columns that do not change do not generate an audit log entry.
+
+## Installation
+
+- First, enable plpgsql langauges in your postgresql instance. Execute the following as a superuser in postgres make sure your database has plpgsql enabled:
+
+        CREATE OR REPLACE PROCEDURAL LANGUAGE plpgsql;
+
+- Generate the appropriate Rails files:
+
+        rails generate pg_audit_log:install
+
+- Install the PostgreSQL function and triggers for your project:
+
+        rake pg_audit_log:install
+
+## Usage
+
+The PgAuditLog::Entry ActiveRecord model represents a single entry in the audit log table. Each entry represents a single change to a single field of a record in a table. So if you change 3 columns of a record, that will generate 3 corresponding PgAuditLog::Entry records.
+
+You can see the SQL it injects on every query by running with LOG_AUDIT_SQL
+
+### Migrations
+
+TODO
+
+### schema.rb and development_structure.sql
+
+Since schema.rb cannot represent TRIGGERs or FUNCTIONs you will need to set your environment to generate SQL instead of Ruby for your database schema and structure. In your application environment put the following:
+
+    config.active_record.schema_format = :sql
+
+And you can generate this sql using:
+
+    rake db:structure:dump
+
+## Uninstalling
+
+    rake pg_audit_log:uninstall
+
+## Performance
+
+On a 2.93GHz i7 with PostgreSQL 9.1 the audit log has an overhead of about 0.0035 seconds to each `INSERT`, `UPDATE`, or `DELETE`.
+
+## Requirements
+
+- ActiveRecord
+- PostgreSQL
+- Rails 3.2
+
+## LICENSE
+
+Copyright © 2010–2014 Case Commons, LLC. Licensed under the MIT license, available in the “LICENSE” file.

data/Rakefile

@@ -1,12 +1,7 @@
-require 'rake'
-require 'rspec/core'
+require 'bundler/gem_tasks'
 require 'rspec/core/rake_task'
 
-require 'bundler'
-Bundler::GemHelper.install_tasks
+desc 'Run specs'
+RSpec::Core::RakeTask.new
 
 task :default => :spec
-
-desc "Run all specs in spec directory (excluding plugin specs)"
-RSpec::Core::RakeTask.new(:spec)
-

data/lib/pg_audit_log.rb

@@ -1,15 +1,16 @@
 module PgAuditLog
-  IGNORED_TABLES = ["plugin_schema_migrations", "sessions", "schema_migrations"]
+  IGNORED_TABLES = [
+    'plugin_schema_migrations'.freeze,
+    'sessions'.freeze,
+    'schema_migrations'.freeze,
+  ]
 end
 
-require "active_record"
-require "pg_audit_log/version"
-
-raise "ActiveRecord #{::ActiveRecord::VERSION::MAJOR}.x unsupported!" unless ::ActiveRecord::VERSION::MAJOR == 3
-
-require "pg_audit_log/extensions/postgresql_adapter.rb"
-require "pg_audit_log/active_record"
-require "pg_audit_log/entry"
-require "pg_audit_log/function"
-require "pg_audit_log/triggers"
+require 'active_record'
+require 'pg_audit_log/version'
 
+require 'pg_audit_log/extensions/postgresql_adapter.rb'
+require 'pg_audit_log/active_record'
+require 'pg_audit_log/entry'
+require 'pg_audit_log/function'
+require 'pg_audit_log/triggers'

data/lib/pg_audit_log/entry.rb

@@ -1,14 +1,10 @@
 class PgAuditLog::Entry < ActiveRecord::Base
-  TABLE_NAME = "audit_log"
-  if ::ActiveRecord::VERSION::MAJOR == 3 && ::ActiveRecord::VERSION::MINOR >= 2
-    self.table_name = TABLE_NAME
-  else
-    set_table_name TABLE_NAME
-  end
+  TABLE_NAME = 'audit_log'.freeze
+  self.table_name = TABLE_NAME
 
   class CannotDeleteError < StandardError
     def message
-      "Audit Logs cannot be deleted!"
+      'Audit Logs cannot be deleted!'
     end
   end
 
@@ -90,5 +86,4 @@ class PgAuditLog::Entry < ActiveRecord::Base
       raise CannotDeleteError
     end
   end
-
 end

data/lib/pg_audit_log/extensions/postgresql_adapter.rb

@@ -1,16 +1,12 @@
-require "active_record/connection_adapters/postgresql_adapter"
+require 'active_record/connection_adapters/postgresql_adapter'
 
 # Did not want to reopen the class but sending an include seemingly is not working.
 class ActiveRecord::ConnectionAdapters::PostgreSQLAdapter
-  def drop_table_with_auditing(table_name, options = {})
+  def drop_table_with_auditing(table_name)
     if PgAuditLog::Triggers.tables_with_triggers.include?(table_name)
       PgAuditLog::Triggers.drop_for_table(table_name)
     end
-    if ::ActiveRecord::VERSION::MAJOR == 3 && ::ActiveRecord::VERSION::MINOR >= 2
-      drop_table_without_auditing(table_name)
-    else
-      drop_table_without_auditing(table_name, options)
-    end
+    drop_table_without_auditing(table_name)
   end
   alias_method_chain :drop_table, :auditing
 
@@ -69,14 +65,12 @@ class ActiveRecord::ConnectionAdapters::PostgreSQLAdapter
   end
   alias_method_chain :execute, :pg_audit_log
 
-  if ::ActiveRecord::VERSION::MAJOR >= 3 && ::ActiveRecord::VERSION::MINOR >= 1
-    def exec_query_with_pg_audit_log(sql, name = 'SQL', binds = [])
-      conn = exec_query_without_pg_audit_log(sql, name, binds)
-      set_audit_user_id_and_name
-      conn
-    end
-    alias_method_chain :exec_query, :pg_audit_log
+  def exec_query_with_pg_audit_log(sql, name = 'SQL', binds = [])
+    conn = exec_query_without_pg_audit_log(sql, name, binds)
+    set_audit_user_id_and_name
+    conn
   end
+  alias_method_chain :exec_query, :pg_audit_log
 
   private
 
@@ -86,5 +80,4 @@ class ActiveRecord::ConnectionAdapters::PostgreSQLAdapter
     user_unique_name = current_user.try(:unique_name) || "UNKNOWN"
     return [user_id, user_unique_name]
   end
-
 end

data/lib/pg_audit_log/function.rb

@@ -4,23 +4,23 @@ module PgAuditLog
 
     class << self
       def name
-        "audit_changes"
+        'audit_changes'
       end
 
       def users_table_name
-        "users"
+        'users'
       end
 
       def user_id_field
-        "user_id"
+        'user_id'
       end
 
       def user_name_field
-        "user_unique_name"
+        'user_unique_name'
       end
 
       def users_access_column
-        "last_accessed_at"
+        'last_accessed_at'
       end
 
       def pg_audit_log_old_style_user_id
@@ -59,11 +59,11 @@ module PgAuditLog
               unique_name varchar;
               column_name varchar;
             BEGIN
-              user_identifier := #{pg_audit_log_old_style_user_id ? "current_setting('audit.user_id')" : 'pg_temp.pg_audit_log_user_identifier()'};
+              user_identifier := #{pg_audit_log_old_style_user_id ? %q(current_setting('audit.user_id')) : 'pg_temp.pg_audit_log_user_identifier()'};
               IF user_identifier = #{DISABLED_USER} THEN
                 RETURN NULL;
               END IF;
-              unique_name := #{pg_audit_log_old_style_user_id ? "current_setting('audit.user_unique_name')" : 'pg_temp.pg_audit_log_user_unique_name()'};
+              unique_name := #{pg_audit_log_old_style_user_id ? %q(current_setting('audit.user_unique_name')) : 'pg_temp.pg_audit_log_user_unique_name()'};
               primary_key_column := NULL;
               EXECUTE 'SELECT pg_attribute.attname
                        FROM pg_index, pg_class, pg_attribute

data/lib/pg_audit_log/triggers.rb

@@ -1,6 +1,6 @@
 module PgAuditLog
   class Triggers < PgAuditLog::ActiveRecord
-    class MissingTriggers < Exception
+    class MissingTriggers < StandardError
       def initialize(tables)
         @tables = tables
       end
@@ -57,12 +57,10 @@ module PgAuditLog
       end
 
       def without_triggers
-        begin
-          disable
-          yield
-        ensure
-          enable
-        end
+        disable
+        yield
+      ensure
+        enable
       end
 
       def create_for_table(table_name)
@@ -70,11 +68,11 @@ module PgAuditLog
         PgAuditLog::Function.install unless PgAuditLog::Function.installed?
         return if tables_with_triggers.include?(table_name)
         execute <<-SQL
-        CREATE TRIGGER #{trigger_name_for_table(table_name)}
-        AFTER INSERT OR UPDATE OR DELETE
-        ON #{table_name}
-        FOR EACH ROW
-        EXECUTE PROCEDURE #{PgAuditLog::Function.name}()
+          CREATE TRIGGER #{trigger_name_for_table(table_name)}
+          AFTER INSERT OR UPDATE OR DELETE
+          ON #{table_name}
+          FOR EACH ROW
+          EXECUTE PROCEDURE #{PgAuditLog::Function.name}()
         SQL
       end
 
@@ -92,7 +90,7 @@ module PgAuditLog
       end
 
       def trigger_prefix
-        "audit_"
+        'audit_'
       end
 
       def trigger_name_for_table(table_name)

data/lib/pg_audit_log/version.rb

@@ -1,3 +1,3 @@
 module PgAuditLog
-  VERSION = "0.5.6"
+  VERSION = '0.6.0'.freeze
 end

data/pg_audit_log.gemspec

@@ -1,26 +1,27 @@
 # -*- encoding: utf-8 -*-
-$:.push File.expand_path("../lib", __FILE__)
-require "pg_audit_log/version"
+$:.push File.expand_path('../lib', __FILE__)
+require 'pg_audit_log/version'
 
-Gem::Specification.new do |s|
-  s.name        = "pg_audit_log"
-  s.version     = PgAuditLog::VERSION
-  s.platform    = Gem::Platform::RUBY
-  s.authors     = ["Case Commons, LLC"]
-  s.email       = ["casecommons-dev@googlegroups.com"]
-  s.homepage    = "https://github.com/Casecommons/pg_audit_log"
-  s.summary     = %q{postgresql only database-level audit logging of all databases changes}
-  s.description = %q{A completely transparent audit logging component for your application using a stored procedure and triggers. Comes with specs for your project and a rake task to generate the reverse SQL to undo changes logged}
-  s.post_install_message = %q{Please run PgAuditLog::Function.install (in console/migration) to install the new versions of the database functions}
+Gem::Specification.new do |spec|
+  spec.name        = 'pg_audit_log'
+  spec.version     = PgAuditLog::VERSION
+  spec.authors     = ['Case Commons, LLC']
+  spec.email       = ['casecommons-dev@googlegroups.com']
+  spec.homepage    = 'https://github.com/Casecommons/pg_audit_log'
+  spec.summary     = %q{PostgreSQL-only database-level audit logging of all databases changes.}
+  spec.description = %q{A completely transparent audit logging component for your application using a stored procedure and triggers. Comes with specs for your project and a rake task to generate the reverse SQL to undo changes logged.}
+  spec.license     = 'MIT'
 
-  s.files         = `git ls-files`.split("\n")
-  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  s.require_paths = ["lib"]
+  spec.post_install_message = %q{Please run PgAuditLog::Function.install (in console/migration) to install the new versions of the database functions}
 
-  s.add_dependency("rails", ">= 3.0.0")
-  s.add_dependency("pg", ">= 0.9.0")
-  s.add_development_dependency('rspec-rails', "= 2.7")
-  s.add_development_dependency('with_model', '>= 0.1.3')
-  s.add_development_dependency('autotest')
+  spec.files         = `git ls-files`.split($/)
+  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ['lib']
+
+  spec.add_dependency 'rails', '>= 3.2', '< 4.2'
+  spec.add_dependency 'pg', '>= 0.9.0'
+  spec.add_development_dependency 'rspec'
+  spec.add_development_dependency 'rspec-rails'
+  spec.add_development_dependency 'with_model', '>= 0.1.3'
 end

data/spec/configuration_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'spec_helper'
 
 describe "the PostgreSQL database" do
   after do
@@ -6,7 +6,6 @@ describe "the PostgreSQL database" do
   end
 
   it "has an audit log table" do
-    ActiveRecord::Base.connection.table_exists?("audit_log").should be_true
+    ActiveRecord::Base.connection.table_exists?('audit_log').should be_true
   end
-
 end

data/spec/function_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'spec_helper'
 
 describe PgAuditLog::Function do
   describe ".installed?" do
@@ -7,6 +7,7 @@ describe PgAuditLog::Function do
       before do
         PgAuditLog::Function.uninstall
       end
+
       it { should be_false }
     end
 
@@ -14,6 +15,7 @@ describe PgAuditLog::Function do
       before do
         PgAuditLog::Function.install
       end
+
       it { should be_true }
     end
   end
@@ -24,9 +26,9 @@ describe PgAuditLog::Function do
 
     context "new style" do
       it "escapes the email" do
-        subject.should_not match("SET")
+        subject.should_not match('SET')
 
-        subject.should match("FUNCTION")
+        subject.should match('FUNCTION')
         subject.should match("'o''connell@fred.com'::varchar")
       end
     end
@@ -36,12 +38,14 @@ describe PgAuditLog::Function do
         Rails = double
         Rails.stub_chain(:configuration, :pg_audit_log_old_style_user_id).and_return(true)
       end
+
       after { Object.send(:remove_const, :Rails) }
+
       it "escapes the email" do
-        subject.should match("SET")
+        subject.should match('SET')
         subject.should match("'o''connell@fred.com'")
 
-        subject.should_not match("FUNCTION")
+        subject.should_not match('FUNCTION')
         subject.should_not match("'o''connell@fred.com'::varchar")
       end
     end

data/spec/model_spec.rb

@@ -1,7 +1,6 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'spec_helper'
 
 describe PgAuditLog::Entry do
-
   subject { PgAuditLog::Entry.create! }
 
   describe ".delete" do
@@ -21,4 +20,4 @@ describe PgAuditLog::Entry do
       proc { subject.destroy }.should raise_error(PgAuditLog::Entry::CannotDeleteError)
     end
   end
-end
+end

data/spec/pg_audit_log_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'spec_helper'
 
 describe PgAuditLog do
   let(:connection) { ActiveRecord::Base.connection }
@@ -30,52 +30,51 @@ describe PgAuditLog do
       PgAuditLog::Entry.connection.execute("TRUNCATE #{PgAuditLog::Entry.quoted_table_name}")
     end
 
-    let(:attributes) { { :str => "foo", :txt => "bar", :int => 5, :date => Date.today, :dt => Time.now.midnight } }
+    let(:attributes) { { :str => 'foo', :txt => 'bar', :int => 5, :date => Date.current, :dt => Time.current.midnight } }
 
     describe "on create" do
       context "the audit log record with a primary key" do
-
         before do
           AuditedModel.create!(attributes)
         end
 
-        subject { PgAuditLog::Entry.last(:conditions => { :field_name => "str" }) }
+        subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
         it { should be }
         its(:occurred_at) { should be }
         its(:table_name) { should == AuditedModel.table_name }
-        its(:field_name) { should == "str" }
+        its(:field_name) { should == 'str' }
         its(:primary_key) { should == AuditedModel.last.id.to_s }
-        its(:operation) { should == "INSERT" }
+        its(:operation) { should == 'INSERT' }
 
         context "when a user is present" do
           before do
-            Thread.current[:current_user] = stub("User", :id => 1, :unique_name => "my current user")
+            Thread.current[:current_user] = double('User', :id => 1, :unique_name => 'my current user')
             AuditedModel.create!
           end
 
           after { Thread.current[:current_user] = nil }
 
           its(:user_id) { should == 1 }
-          its(:user_unique_name) { should == "my current user" }
+          its(:user_unique_name) { should == 'my current user' }
         end
 
         context "when no user is present" do
           its(:user_id) { should == -1 }
-          its(:user_unique_name) { should == "UNKNOWN" }
+          its(:user_unique_name) { should == 'UNKNOWN' }
         end
 
         it "captures all new values for all fields" do
           attributes.each do |field_name, value|
+            entry = PgAuditLog::Entry.where(:field_name => field_name).last
             if field_name == :dt
-              PgAuditLog::Entry.last(:conditions => { :field_name => field_name }).field_value_new.should == value.strftime("%Y-%m-%d %H:%M:%S")
+              entry.field_value_new.should == value.strftime("%Y-%m-%d %H:%M:%S")
             else
-              PgAuditLog::Entry.last(:conditions => { :field_name => field_name }).field_value_new.should == value.to_s
+              entry.field_value_new.should == value.to_s
             end
-            PgAuditLog::Entry.last(:conditions => { :field_name => field_name }).field_value_old.should be_nil
+            entry.field_value_old.should be_nil
           end
         end
-
       end
 
       context "the audit log record without a primary key" do
@@ -83,10 +82,10 @@ describe PgAuditLog do
           AuditedModelWithoutPrimaryKey.create!(attributes)
         end
 
-        subject { PgAuditLog::Entry.last(:conditions => { :field_name => "str" }) }
+        subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
         it { should be }
-        its(:field_name) { should == "str" }
+        its(:field_name) { should == 'str' }
         its(:primary_key) { should be_nil }
       end
     end
@@ -98,34 +97,34 @@ describe PgAuditLog do
         end
 
         context "when going from a value to a another value" do
-          before { @model.update_attributes!(:str => "bar") }
-          subject { PgAuditLog::Entry.last(:conditions => { :field_name => "str" }) }
+          before { @model.update_attributes!(:str => 'bar') }
+          subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
-          its(:operation) { should == "UPDATE" }
-          its(:field_value_new) { should == "bar" }
-          its(:field_value_old) { should == "foo" }
+          its(:operation) { should == 'UPDATE' }
+          its(:field_value_new) { should == 'bar' }
+          its(:field_value_old) { should == 'foo' }
         end
 
         context "when going from nil to a value" do
           let(:attributes) { {:txt => nil} }
-          before { @model.update_attributes!(:txt => "baz") }
-          subject { PgAuditLog::Entry.last(:conditions => { :field_name => "txt" }) }
+          before { @model.update_attributes!(:txt => 'baz') }
+          subject { PgAuditLog::Entry.where(:field_name => 'txt').last }
 
-          its(:field_value_new) { should == "baz" }
+          its(:field_value_new) { should == 'baz' }
           its(:field_value_old) { should be_nil }
         end
 
         context "when going from a value to nil" do
           before { @model.update_attributes!(:str => nil) }
-          subject { PgAuditLog::Entry.last(:conditions => { :field_name => "str" }) }
+          subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
           its(:field_value_new) { should be_nil }
-          its(:field_value_old) { should == "foo" }
+          its(:field_value_old) { should == 'foo' }
         end
 
         context "when the value does not change" do
-          before { @model.update_attributes!(:str => "foo") }
-          subject { PgAuditLog::Entry.last(:conditions => { :field_name => "str", :operation => "UPDATE" }) }
+          before { @model.update_attributes!(:str => 'foo') }
+          subject { PgAuditLog::Entry.where(:field_name => 'str', :operation => 'UPDATE').last }
 
           it { should_not be }
         end
@@ -133,18 +132,17 @@ describe PgAuditLog do
         context "when the value is nil and does not change" do
           let(:attributes) { {:txt => nil} }
           before { @model.update_attributes!(:txt => nil) }
-          subject { PgAuditLog::Entry.last(:conditions => { :field_name => "txt", :operation => "UPDATE" }) }
+          subject { PgAuditLog::Entry.where(:field_name => 'txt', :operation => 'UPDATE').last }
 
           it { should_not be }
         end
 
         context "when the value is a boolean" do
-
           context "going from nil -> true" do
             before { @model.update_attributes!(:bool => true) }
-            subject { PgAuditLog::Entry.last(:conditions => { :field_name => "bool", :operation => "UPDATE" }) }
+            subject { PgAuditLog::Entry.where(:field_name => 'bool', :operation => 'UPDATE').last }
 
-            its(:field_value_new) { should == "true" }
+            its(:field_value_new) { should == 'true' }
             its(:field_value_old) { should be_nil }
           end
 
@@ -153,10 +151,10 @@ describe PgAuditLog do
             before do
               @model.update_attributes!(:bool => true)
             end
-            subject { PgAuditLog::Entry.last(:conditions => { :field_name => "bool", :operation => "UPDATE" }) }
+            subject { PgAuditLog::Entry.where(:field_name => 'bool', :operation => 'UPDATE').last }
 
-            its(:field_value_new) { should == "true" }
-            its(:field_value_old) { should == "false" }
+            its(:field_value_new) { should == 'true' }
+            its(:field_value_old) { should == 'false' }
           end
 
           context "going from true -> false" do
@@ -165,48 +163,46 @@ describe PgAuditLog do
             before do
               @model.update_attributes!(:bool => false)
             end
-            subject { PgAuditLog::Entry.last(:conditions => { :field_name => "bool", :operation => "UPDATE" }) }
+            subject { PgAuditLog::Entry.where(:field_name => 'bool', :operation => 'UPDATE').last }
 
-            its(:field_value_new) { should == "false" }
-            its(:field_value_old) { should == "true" }
+            its(:field_value_new) { should == 'false' }
+            its(:field_value_old) { should == 'true' }
           end
-
         end
       end
 
       context "the audit log record without a primary key" do
         before do
           AuditedModelWithoutPrimaryKey.create!(attributes)
-          AuditedModelWithoutPrimaryKey.update_all(:str => "bar")
+          AuditedModelWithoutPrimaryKey.update_all(:str => 'bar')
         end
 
-        subject { PgAuditLog::Entry.last(:conditions => { :field_name => "str" }) }
+        subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
         its(:primary_key) { should be_nil }
       end
-
     end
 
     describe "on delete" do
-
       context "the audit log record with a primary key" do
         before do
           model = AuditedModel.create!(attributes)
           model.delete
         end
 
-        subject { PgAuditLog::Entry.last(:conditions => { :field_name => "str" }) }
+        subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
-        its(:operation) { should == "DELETE" }
+        its(:operation) { should == 'DELETE' }
 
         it "captures all new values for all fields" do
           attributes.each do |field_name, value|
+            entry = PgAuditLog::Entry.where(:field_name => field_name).last
             if field_name == :dt
-              PgAuditLog::Entry.last(:conditions => { :field_name => field_name }).field_value_old.should == value.strftime("%Y-%m-%d %H:%M:%S")
+              entry.field_value_old.should == value.strftime('%Y-%m-%d %H:%M:%S')
             else
-              PgAuditLog::Entry.last(:conditions => { :field_name => field_name }).field_value_old.should == value.to_s
+              entry.field_value_old.should == value.to_s
             end
-            PgAuditLog::Entry.last(:conditions => { :field_name => field_name }).field_value_new.should be_nil
+            entry.field_value_new.should be_nil
           end
         end
       end
@@ -217,7 +213,7 @@ describe PgAuditLog do
           AuditedModelWithoutPrimaryKey.delete_all
         end
 
-        subject { PgAuditLog::Entry.last(:conditions => { :field_name => "str" }) }
+        subject { PgAuditLog::Entry.where(:field_name => 'str').last }
 
         its(:primary_key) { should be_nil }
       end
@@ -225,7 +221,7 @@ describe PgAuditLog do
 
     describe "performance" do
       xit "should perform well" do
-        require "benchmark"
+        require 'benchmark'
         results = Benchmark.measure do
           1000.times do
             AuditedModel.create!(attributes)
@@ -238,29 +234,28 @@ describe PgAuditLog do
   end
 
   describe "during migrations" do
-
     before do
-      connection.drop_table("test_table") rescue nil
-      connection.drop_table("new_table") rescue nil
+      connection.drop_table('test_table') rescue nil
+      connection.drop_table('new_table') rescue nil
     end
 
     after do
-      connection.drop_table("test_table") rescue nil
+      connection.drop_table('test_table') rescue nil
     end
 
     describe "when creating the table" do
       it "should automatically create the trigger" do
-        PgAuditLog::Triggers.tables_with_triggers.should_not include("test_table")
-        connection.create_table("test_table")
-        PgAuditLog::Triggers.tables_with_triggers.should include("test_table")
+        PgAuditLog::Triggers.tables_with_triggers.should_not include('test_table')
+        connection.create_table('test_table')
+        PgAuditLog::Triggers.tables_with_triggers.should include('test_table')
       end
     end
 
     describe "when dropping the table" do
       it "should automatically drop the trigger" do
-        connection.create_table("test_table")
-        connection.drop_table("test_table")
-        PgAuditLog::Triggers.tables_with_triggers.should_not include("test_table")
+        connection.create_table('test_table')
+        connection.drop_table('test_table')
+        PgAuditLog::Triggers.tables_with_triggers.should_not include('test_table')
       end
     end
 
@@ -274,11 +269,11 @@ describe PgAuditLog do
       end
 
       it "should automatically drop and create the trigger" do
-        new_table_name = "new_table_#{Time.now.to_i}"
-        connection.create_table("test_table")
-        connection.rename_table("test_table", new_table_name)
+        new_table_name = "new_table_#{Time.current.to_i}"
+        connection.create_table('test_table')
+        connection.rename_table('test_table', new_table_name)
 
-        trigger_names.should_not include("audit_test_table")
+        trigger_names.should_not include('audit_test_table')
         trigger_names.should include("audit_#{new_table_name}")
         PgAuditLog::Triggers.tables_with_triggers.should include(new_table_name)
 
@@ -290,16 +285,17 @@ describe PgAuditLog do
   describe "temporary tables" do
     context "when creating them" do
       it "should be ignored" do
-        connection.create_table("some_temp_table", :temporary => true)
-        PgAuditLog::Triggers.tables_with_triggers.should_not include("some_temp_table")
-        connection.drop_table("some_temp_table")
+        connection.create_table('some_temp_table', :temporary => true)
+        PgAuditLog::Triggers.tables_with_triggers.should_not include('some_temp_table')
+        connection.drop_table('some_temp_table')
       end
     end
+
     context "when dropping them" do
       it "should be ignored" do
-        connection.create_table("some_temp_table", :temporary => true)
-        connection.drop_table("some_temp_table")
-        PgAuditLog::Triggers.tables_with_triggers.should_not include("some_temp_table")
+        connection.create_table('some_temp_table', :temporary => true)
+        connection.drop_table('some_temp_table')
+        PgAuditLog::Triggers.tables_with_triggers.should_not include('some_temp_table')
       end
     end
   end
@@ -311,9 +307,9 @@ describe PgAuditLog do
 
     context "when creating a table" do
       it "should install the function then enable the trigger on the table" do
-        connection.create_table("some_more_new_table")
-        PgAuditLog::Triggers.tables_with_triggers.should include("some_more_new_table")
-        connection.drop_table("some_more_new_table")
+        connection.create_table('some_more_new_table')
+        PgAuditLog::Triggers.tables_with_triggers.should include('some_more_new_table')
+        connection.drop_table('some_more_new_table')
       end
     end
   end
@@ -326,9 +322,9 @@ describe PgAuditLog do
     context "when creating a table" do
       it "should install the entry table then enable the trigger on the table" do
         PgAuditLog::Entry.installed?.should be_false
-        connection.create_table("another_table")
+        connection.create_table('another_table')
         PgAuditLog::Entry.installed?.should be_true
-        connection.drop_table("another_table")
+        connection.drop_table('another_table')
       end
     end
   end
@@ -336,12 +332,11 @@ describe PgAuditLog do
   describe "ignored tables" do
     context "when creating one of those tables" do
       it "should not automatically create a trigger for it" do
-        PgAuditLog::IGNORED_TABLES << "ignored_table"
-        connection.create_table("ignored_table")
-        PgAuditLog::Triggers.tables_with_triggers.should_not include("ignored_table")
-        connection.drop_table("ignored_table")
+        PgAuditLog::IGNORED_TABLES << 'ignored_table'
+        connection.create_table('ignored_table')
+        PgAuditLog::Triggers.tables_with_triggers.should_not include('ignored_table')
+        connection.drop_table('ignored_table')
       end
     end
   end
-
 end

data/spec/spec_helper.rb

@@ -1,23 +1,29 @@
-require "bundler/setup"
-require "pg_audit_log"
-require "with_model"
+require 'bundler'
+Bundler.setup
+
+require 'pg_audit_log'
+require 'with_model'
 
 connection = nil
 begin
-  ActiveRecord::Base.establish_connection(:adapter  => 'postgresql',
-                                          :database => 'pg_audit_log_test',
-                                          :min_messages => 'warning')
+  ActiveRecord::Base.establish_connection({
+    :adapter  => 'postgresql',
+    :database => 'pg_audit_log_test',
+    :min_messages => 'warning',
+  })
   connection = ActiveRecord::Base.connection
-  connection.execute("SELECT 1")
+  connection.execute('SELECT 1')
 rescue PGError => e
-  puts "-" * 80
-  puts "Unable to connect to database.  Please run:"
+  puts '-' * 80
+  puts 'Unable to connect to database.  Please run:'
   puts
-  puts "    createdb pg_audit_log_test"
-  puts "-" * 80
+  puts '    createdb pg_audit_log_test'
+  puts '-' * 80
   raise e
 end
 
+ActiveRecord::Base.default_timezone = :local
+
 RSpec.configure do |config|
   config.mock_with :rspec
   config.extend WithModel

data/spec/triggers_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'spec_helper'
 
 describe PgAuditLog::Triggers do
   before :each do
@@ -155,7 +155,5 @@ describe PgAuditLog::Triggers do
         end
       end
     end
-
   end
 end
-

metadata

@@ -1,88 +1,108 @@
 --- !ruby/object:Gem::Specification
 name: pg_audit_log
 version: !ruby/object:Gem::Version
-  version: 0.5.6
-  prerelease: 
+  version: 0.6.0
 platform: ruby
 authors:
 - Case Commons, LLC
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-09-27 00:00:00.000000000 Z
+date: 2014-05-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &70199944309260 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '3.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.2'
   type: :runtime
   prerelease: false
-  version_requirements: *70199944309260
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '3.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.2'
 - !ruby/object:Gem::Dependency
   name: pg
-  requirement: &70199944308760 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.9.0
   type: :runtime
   prerelease: false
-  version_requirements: *70199944308760
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.9.0
 - !ruby/object:Gem::Dependency
-  name: rspec-rails
-  requirement: &70199944308300 !ruby/object:Gem::Requirement
-    none: false
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - =
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.7'
+        version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70199944308300
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: with_model
-  requirement: &70199944307840 !ruby/object:Gem::Requirement
-    none: false
+  name: rspec-rails
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.1.3
+        version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70199944307840
-- !ruby/object:Gem::Dependency
-  name: autotest
-  requirement: &70199944307460 !ruby/object:Gem::Requirement
-    none: false
+  version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: with_model
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.1.3
   type: :development
   prerelease: false
-  version_requirements: *70199944307460
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.1.3
 description: A completely transparent audit logging component for your application
   using a stored procedure and triggers. Comes with specs for your project and a rake
-  task to generate the reverse SQL to undo changes logged
+  task to generate the reverse SQL to undo changes logged.
 email:
 - casecommons-dev@googlegroups.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
-- .rvmrc
+- ".gitignore"
+- ".rspec"
+- ".rvmrc"
+- ".travis.yml"
 - Gemfile
 - LICENSE
-- README.rdoc
+- README.md
 - Rakefile
-- init.rb
 - lib/generators/pg_audit_log/install_generator.rb
 - lib/generators/pg_audit_log/rspec_generator.rb
 - lib/generators/pg_audit_log/templates/lib/tasks/pg_audit_log.rake
@@ -104,36 +124,30 @@ files:
 - spec/spec_helper.rb
 - spec/triggers_spec.rb
 homepage: https://github.com/Casecommons/pg_audit_log
-licenses: []
+licenses:
+- MIT
+metadata: {}
 post_install_message: Please run PgAuditLog::Function.install (in console/migration)
   to install the new versions of the database functions
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 1103642768265560851
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 1103642768265560851
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.17
+rubygems_version: 2.2.2
 signing_key: 
-specification_version: 3
-summary: postgresql only database-level audit logging of all databases changes
+specification_version: 4
+summary: PostgreSQL-only database-level audit logging of all databases changes.
 test_files:
 - spec/configuration_spec.rb
 - spec/function_spec.rb
@@ -141,3 +155,4 @@ test_files:
 - spec/pg_audit_log_spec.rb
 - spec/spec_helper.rb
 - spec/triggers_spec.rb
+has_rdoc: 

data/README.rdoc

@@ -1,62 +0,0 @@
-= pg_audit_log
-
-* http://github.com/casecommons/pg_audit_log/
-
-== DESCRIPTION
-
-PostgreSQL only database-level audit logging of all databases changes using a completely transparent stored procedure and triggers.
-Comes with specs for your project and a rake task to generate the reverse SQL to undo changes logged}
-
-All SQL INSERTs, UPDATEs, and DELETEs will be captured. Record columns that do not change do not generate an audit log entry.
-
-Compatible with Rails 3.0.x, 3.1.x and 3.2.x
-
-On a 2.93GHz i7 with postgresql 9.1 the audit log has an overhead of about 0.0035 seconds to each INSERT, UPDATE or DELETE
-
-== INSTALL
-
-=== Enable plpgsql langauges in your postgresql instance
-
-  As a superuser in postgres make sure your database has plpgsql enabled:
-
-  CREATE OR REPLACE PROCEDURAL LANGUAGE plpgsql;
-
-=== Rails 3
-
-  $ rails generate pg_audit_log:install
-
-=== Installing the PostgreSQL function and triggers for your project
-
-  $ rake pg_audit_log:install
-
-== Using on your project
-
-The PgAuditLog::Entry ActiveRecord model represents a single entry in the audit log table. Each entry represents a single change to a single field of a record in a table. So if you change 3 columns of a record, that will generate 3 corresponding PgAuditLog::Entry records.
-
-You can see the SQL it injects on every query by running with LOG_AUDIT_SQL
-
-=== Migrations
-
-TODO
-
-=== schema.rb and development_structure.sql
-
-Since schema.rb cannot represent TRIGGERs or FUNCTIONs you will need to set your environment to generate SQL instead of ruby for your database schema and structure. In your application environment put the following:
-
-  config.active_record.schema_format = :sql
-
-And you can generate this sql using:
-
-  $ rake db:structure:dump
-
-=== Uninstalling
-
-  $ rake pg_audit_log:uninstall
-
-== REQUIREMENTS
-
-* ActiveRecord
-
-== LICENSE
-
-MIT

data/init.rb

@@ -1 +0,0 @@
-require "pg_audit_log"