lograge 0.1.1 → 0.1.2

data/lib/lograge/log_subscriber.rb

@@ -52,13 +52,17 @@ module Lograge
     def extract_request(payload)
       {
         :method => payload[:method],
-        :path => payload[:path],
+        :path => extract_path(payload),
         :format => extract_format(payload),
         :controller => payload[:params]['controller'],
         :action => payload[:params]['action']
       }
     end
 
+    def extract_path(payload)
+      payload[:path].split("?").first
+    end
+
     def extract_format(payload)
       if ::ActionPack::VERSION::MAJOR == 3 && ::ActionPack::VERSION::MINOR == 0
         payload[:formats].first

data/lib/lograge/version.rb

@@ -1,3 +1,3 @@
 module Lograge
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

data/spec/lograge_logsubscriber_spec.rb

@@ -22,8 +22,8 @@ describe Lograge::RequestLogSubscriber do
   let(:event) {
     ActiveSupport::Notifications::Event.new(
       'process_action.action_controller', Time.now, Time.now, 2, {
-        status: 200, format: 'application/json', method: 'GET', path: '/home', params: {
-          'controller' => 'home', 'action' => 'index'
+        status: 200, format: 'application/json', method: 'GET', path: '/home?foo=bar', params: {
+          'controller' => 'home', 'action' => 'index', 'foo' => 'bar'
         }, db_runtime: 0.02, view_runtime: 0.01
       }
     )
@@ -45,6 +45,11 @@ describe Lograge::RequestLogSubscriber do
       log_output.string.should include('/home')
     end
 
+    it "should not include the query string in the url" do
+      subscriber.process_action(event)
+      log_output.string.should_not include('?foo=bar')
+    end
+
     it "should start the log line with the HTTP method" do
       subscriber.process_action(event)
       log_output.string.starts_with?('method=GET ').should == true
@@ -112,7 +117,7 @@ describe Lograge::RequestLogSubscriber do
     end
   end
 
-    describe "when processing an action with logstash output" do
+  describe "when processing an action with logstash output" do
     before do
       require 'logstash-event'
       Lograge::log_format = :logstash

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lograge
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-08 00:00:00.000000000 Z
+date: 2012-12-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec