lograge 0.0.2 → 0.0.3

data/lib/lograge/log_subscriber.rb

@@ -8,9 +8,14 @@ module Lograge
       message = "#{payload[:method]} #{payload[:path]} format=#{payload[:format]} action=#{payload[:params]['controller']}##{payload[:params]['action']}"
       message << extract_status(payload)
       message << runtimes(event)
+      message << location(event)
       logger.info(message)
     end
 
+    def redirect_to(event)
+      Thread.current[:lograge_location] = event.payload[:location]
+    end
+
     private
 
     def extract_status(payload)
@@ -31,5 +36,14 @@ module Lograge
       end
       message
     end
+
+    def location(event)
+      if location = Thread.current[:lograge_location]
+        Thread.current[:lograge_location] = nil
+        "location=#{location}"
+      else
+        ""
+      end
+    end  
   end
 end

data/lib/lograge/version.rb

@@ -1,3 +1,3 @@
 module Lograge
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/spec/lograge_logsubscriber_spec.rb

@@ -77,5 +77,33 @@ describe Lograge::RequestLogSubscriber do
       log_output.string.should =~ /status=500/
       log_output.string.should =~ /error='AbstractController::ActionNotFound:Route not found'/
     end
+
+    describe "with a redirect" do
+      before do
+        Thread.current[:lograge_location] = "http://www.example.com"
+      end
+
+      it "should add the location to the log line" do
+        subscriber.process_action(event)
+        log_output.string.should =~ %r{location=http://www.example.com}
+      end
+
+      it "should remove the thread local variable" do
+        subscriber.process_action(event)
+        Thread.current[:lograge_location].should == nil
+      end
+    end
+    
+    it "should not include a location by default" do
+      subscriber.process_action(event)
+      log_output.string.should_not =~ /location=/
+    end
+  end
+
+  describe "when processing a redirect" do
+    it "should store the location in a thread local variable" do
+      subscriber.redirect_to(redirect)
+      Thread.current[:lograge_location].should == "http://example.com"
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lograge
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2012-03-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70126021492960 !ruby/object:Gem::Requirement
+  requirement: &70153872860480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70126021492960
+  version_requirements: *70153872860480
 - !ruby/object:Gem::Dependency
   name: guard-rspec
-  requirement: &70126021492360 !ruby/object:Gem::Requirement
+  requirement: &70153872859680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70126021492360
+  version_requirements: *70153872859680
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70126021491300 !ruby/object:Gem::Requirement
+  requirement: &70153872858340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70126021491300
+  version_requirements: *70153872858340
 - !ruby/object:Gem::Dependency
   name: actionpack
-  requirement: &70126021489500 !ruby/object:Gem::Requirement
+  requirement: &70153872856860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +54,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70126021489500
+  version_requirements: *70153872856860
 description: Tame Rails' multi-line logging into a single line per request
 email:
 - meyer@paperplanes.de