http_logger 0.4.3 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 42d833f6603f1388a733dcc558c3a8e23004ac2e
-  data.tar.gz: 20d21f3e5dfa0413809092499c4769bf7aa2d303
+  metadata.gz: 9d99effbf8fa7d49cb7db951595025c3a9f2f6fe
+  data.tar.gz: 85a8d71ee430d2652c4655eb553bc865c39ef746
 SHA512:
-  metadata.gz: fef1b14a5d854fab100449356b60539ed40076f61df7a4e946fb14b1e8effc50a48ef633876f02249fa5f0e5bb47ef4809e638b7c70ba27351446702fddb59aa
-  data.tar.gz: 5b063e5050d9925a12d19ffa5efbe524f0fb01f27d08c2161c5b0d7cbd1dd90d2719f857ca6693b7344bd53235be25f85a7400988210a11df1ef7f4dfb9c3a5e
+  metadata.gz: 7a6704abca3ff95642f1c4f4e527030c11628328104a7fe510c0641e2dbf3c615820bf98f1ab567c338c823a793691ba3dfca7215306c50d7364f72ff71d624b
+  data.tar.gz: 996ca33603a94dfbbbc7665214fc6745ca05f6720d6284a02ebda9b12154771927dae580e0b85611134250d1e597b5358bc32e8dd17c30417c9d0dcb0de1dc0b

data/Readme.md

@@ -24,6 +24,9 @@ HttpLogger.logger = Logger.new(...) # defaults to Rails.logger if Rails is defin
 HttpLogger.colorize = true # Default: true
 HttpLogger.ignore = [/newrelic\.com/]
 HttpLogger.log_headers = false  # Default: false
+HttpLogger.log_request_body  = false  # Default: true
+HttpLogger.log_response_body = false  # Default: true
+HttpLogger.level = :info # Desired log level as a symbol. Default: :debug
 ```
 
 ## Alternative

data/VERSION

@@ -1 +1 @@
-0.4.3
+0.5.0

data/http_logger.gemspec

@@ -2,16 +2,15 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: http_logger 0.4.3 ruby lib
+# stub: http_logger 0.5.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "http_logger"
-  s.version = "0.4.3"
+  s.version = "0.5.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.require_paths = ["lib"]
   s.authors = ["Bogdan Gusiev"]
-  s.date = "2014-04-04"
+  s.date = "2014-10-12"
   s.description = "This gem keep an eye on every Net::HTTP library usage and dump all request and response data to the log file"
   s.email = "agresso@gmail.com"
   s.extra_rdoc_files = [
@@ -36,7 +35,8 @@ Gem::Specification.new do |s|
   ]
   s.homepage = "http://github.com/railsware/http_logger"
   s.licenses = ["MIT"]
-  s.rubygems_version = "2.2.2"
+  s.require_paths = ["lib"]
+  s.rubygems_version = "2.1.10"
   s.summary = "Log your http api calls just like SQL queries"
 
   if s.respond_to? :specification_version then

data/lib/http_logger.rb

@@ -25,15 +25,21 @@ class HttpLogger
   class << self
     attr_accessor :collapse_body_limit
     attr_accessor :log_headers
+    attr_accessor :log_request_body
+    attr_accessor :log_response_body
     attr_accessor :logger
     attr_accessor :colorize
     attr_accessor :ignore
+    attr_accessor :level
   end
 
   self.log_headers = false
+  self.log_request_body = true
+  self.log_response_body = true
   self.colorize = true
   self.collapse_body_limit = 5000
   self.ignore = []
+  self.level = :debug
 
   def self.perform(*args, &block)
     instance.perform(*args, &block)
@@ -83,9 +89,11 @@ class HttpLogger
   HTTP_METHODS_WITH_BODY = Set.new(%w(POST PUT GET))
   
   def log_request_body(request)
-    if HTTP_METHODS_WITH_BODY.include?(request.method)
-      if (body = request.body) && !body.empty?
-        log("Request body", truncate_body(body))
+    if self.class.log_request_body
+      if HTTP_METHODS_WITH_BODY.include?(request.method)
+        if (body = request.body) && !body.empty?
+          log("Request body", truncate_body(body))
+        end
       end
     end
   end
@@ -101,11 +109,13 @@ class HttpLogger
   end
 
   def log_response_body(body)
-    if body.is_a?(Net::ReadAdapter)
-      log("Response body", "<impossible to log>")
-    else
-      if body && !body.empty?
-        log("Response body", truncate_body(body))
+    if self.class.log_response_body
+      if body.is_a?(Net::ReadAdapter)
+        log("Response body", "<impossible to log>")
+      else
+        if body && !body.empty?
+          log("Response body", truncate_body(body))
+        end
       end
     end
   end
@@ -139,9 +149,8 @@ class HttpLogger
     end
   end
 
-
   def log(message, dump)
-    self.logger.debug(format_log_entry(message, dump))
+    self.logger.send(self.class.level, format_log_entry(message, dump))
   end
 
   def format_log_entry(message, dump = nil)
@@ -206,5 +215,3 @@ if defined?(Rails)
     end
   end
 end
-
-

data/spec/http_logger_spec.rb

@@ -70,6 +70,7 @@ describe HttpLogger do
     end
 
   end
+
   describe "put request" do
     let(:request) do
       http = Net::HTTP.new(uri.host, uri.port)
@@ -94,6 +95,23 @@ describe HttpLogger do
     it {should include("a=hello&b=1")}
   end
 
+  context "when request body logging is off" do
+
+    before(:each) do
+      HttpLogger.log_request_body = false
+    end
+
+    let(:request) do
+      Net::HTTP.post_form(uri, {})
+    end
+
+    it { should_not include("Request body") }
+
+    after(:each) do
+      HttpLogger.log_request_body = true
+    end
+  end
+
   context "with long response body" do
 
     let(:url) do
@@ -107,6 +125,24 @@ describe HttpLogger do
 
   end
 
+  context "when response body logging is off" do
+
+    before(:each) do
+      HttpLogger.log_response_body = false
+    end
+
+    let(:url) do
+      FakeWeb.register_uri(:get, "http://github.com", :body => long_body)
+      "http://github.com"
+    end
+
+    it { should_not include("Response body") }
+
+    after(:each) do
+      HttpLogger.log_response_body = true
+    end
+  end
+
   context "ignore option is set" do
 
     let(:url) { "http://rpm.newrelic.com/hello/world"}
@@ -121,4 +157,19 @@ describe HttpLogger do
       HttpLogger.ignore = []
     end
   end
+
+  context "when level is set" do
+
+    let(:url) { "http://rpm.newrelic.com/hello/world"}
+
+    before(:each) do
+      HttpLogger.level = :info
+    end
+
+    it { should_not be_empty }
+    
+    after(:each) do
+      HttpLogger.level = :debug
+    end
+  end
 end

metadata

@@ -1,83 +1,83 @@
 --- !ruby/object:Gem::Specification
 name: http_logger
 version: !ruby/object:Gem::Version
-  version: 0.4.3
+  version: 0.5.0
 platform: ruby
 authors:
 - Bogdan Gusiev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-04 00:00:00.000000000 Z
+date: 2014-10-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: debugger
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: jeweler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: fakeweb
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: This gem keep an eye on every Net::HTTP library usage and dump all request
@@ -88,8 +88,8 @@ extensions: []
 extra_rdoc_files:
 - LICENSE.txt
 files:
-- ".document"
-- ".rspec"
+- .document
+- .rspec
 - Gemfile
 - Gemfile.lock
 - LICENSE.txt
@@ -113,17 +113,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.1.10
 signing_key: 
 specification_version: 4
 summary: Log your http api calls just like SQL queries