http_logger 0.0.1 → 0.1.0

data/Gemfile

@@ -6,4 +6,5 @@ group :development do
   gem "bundler", "~> 1.0.0"
   gem "jeweler", "~> 1.6.0"
   gem "rcov", ">= 0"
+  gem "fakeweb"
 end

data/Gemfile.lock

@@ -2,6 +2,7 @@ GEM
   remote: http://rubygems.org/
   specs:
     diff-lcs (1.1.2)
+    fakeweb (1.3.0)
     git (1.2.5)
     jeweler (1.6.0)
       bundler (~> 1.0.0)
@@ -23,6 +24,7 @@ PLATFORMS
 
 DEPENDENCIES
   bundler (~> 1.0.0)
+  fakeweb
   jeweler (~> 1.6.0)
   rcov
   rspec (~> 2.3.0)

data/Readme.md

@@ -24,3 +24,7 @@ Simple gem that logs your HTTP api requests just like database queries
 
 Net::HTTP has a builtin logger that can be set via \#set\_debug\_output.
 This method is only available at the instance level and it is not always accessible if used inside of a library. Also output of builtin debugger is not formed well for API debug purposes.
+
+## Integration
+
+If you are using Net::HTTP#request hackers like FakeWeb make sure you require http\_logger after all others because http\_logger always calls "super", rather than others.

data/VERSION

@@ -1 +1 @@
-0.0.1
+0.1.0

data/http_logger.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{http_logger}
-  s.version = "0.0.1"
+  s.version = "0.1.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Bogdan Gusiev"]
-  s.date = %q{2011-05-07}
+  s.date = %q{2011-05-08}
   s.description = %q{This gem keep an eye on every Net::HTTP library usage and dump all request and response data to the log file}
   s.email = %q{agresso@gmail.com}
   s.extra_rdoc_files = [
@@ -46,17 +46,20 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.6.0"])
       s.add_development_dependency(%q<rcov>, [">= 0"])
+      s.add_development_dependency(%q<fakeweb>, [">= 0"])
     else
       s.add_dependency(%q<rspec>, ["~> 2.3.0"])
       s.add_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_dependency(%q<jeweler>, ["~> 1.6.0"])
       s.add_dependency(%q<rcov>, [">= 0"])
+      s.add_dependency(%q<fakeweb>, [">= 0"])
     end
   else
     s.add_dependency(%q<rspec>, ["~> 2.3.0"])
     s.add_dependency(%q<bundler>, ["~> 1.0.0"])
     s.add_dependency(%q<jeweler>, ["~> 1.6.0"])
     s.add_dependency(%q<rcov>, [">= 0"])
+    s.add_dependency(%q<fakeweb>, [">= 0"])
   end
 end
 

data/lib/http_logger.rb

@@ -32,7 +32,7 @@ class Net::HTTP
     response = request_without_logging(request, body, &block)
     response
   ensure
-    if self.started? && self.logger
+    if self.require_logging?(request)
       url = "http#{"s" if self.use_ssl?}://#{self.address}:#{self.port}#{request.path}"
       ofset = Time.now - time
       log("HTTP #{request.method} (%0.2fms)" % (ofset * 1000), url)
@@ -45,6 +45,18 @@ class Net::HTTP
     end
   end
 
+  def require_logging?(request)
+    fakeweb = if defined?(::FakeWeb)
+                uri = FakeWeb::Utility.request_uri_as_string(self, request)
+                method = request.method.downcase.to_sym
+                FakeWeb.registered_uri?(method, uri)
+              else
+                false
+              end
+    self.logger && (self.started? || fakeweb)
+  end
+
+
   protected
   def log(message, dump)
     self.logger.debug(format_log_entry(message, dump))

data/spec/http_logger_spec.rb

@@ -1,11 +1,24 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
 describe "HttpLogger" do
-  it "should log request" do
-    uri = URI.parse("http://google.com/")
 
+  subject do
+    uri = URI.parse("http://google.com/")
     response = Net::HTTP.get_response(uri)
-    File.read(LOGFILE).should_not be_empty
+    File.read(LOGFILE)
+  end
+
+  it { should_not be_empty }
+
+  context "with FakeWeb" do
+    before(:all) do
+      require 'fakeweb'
+    end
+    it {should_not be_empty}
 
+    after(:all) do
+      Object.send(:remove_const, "FakeWeb")
+    end
+    
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: http_logger
 version: !ruby/object:Gem::Version 
-  hash: 29
+  hash: 27
   prerelease: false
   segments: 
   - 0
-  - 0
   - 1
-  version: 0.0.1
+  - 0
+  version: 0.1.0
 platform: ruby
 authors: 
 - Bogdan Gusiev
@@ -15,14 +15,12 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-05-07 00:00:00 +03:00
+date: 2011-05-08 00:00:00 +03:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: rspec
-  type: :development
-  prerelease: false
-  version_requirements: &id001 !ruby/object:Gem::Requirement 
+  requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -33,12 +31,12 @@ dependencies:
         - 3
         - 0
         version: 2.3.0
-  requirement: *id001
+  prerelease: false
+  type: :development
+  version_requirements: *id001
 - !ruby/object:Gem::Dependency 
   name: bundler
-  type: :development
-  prerelease: false
-  version_requirements: &id002 !ruby/object:Gem::Requirement 
+  requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -49,12 +47,12 @@ dependencies:
         - 0
         - 0
         version: 1.0.0
-  requirement: *id002
+  prerelease: false
+  type: :development
+  version_requirements: *id002
 - !ruby/object:Gem::Dependency 
   name: jeweler
-  type: :development
-  prerelease: false
-  version_requirements: &id003 !ruby/object:Gem::Requirement 
+  requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -65,12 +63,26 @@ dependencies:
         - 6
         - 0
         version: 1.6.0
-  requirement: *id003
+  prerelease: false
+  type: :development
+  version_requirements: *id003
 - !ruby/object:Gem::Dependency 
   name: rcov
-  type: :development
+  requirement: &id004 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
   prerelease: false
-  version_requirements: &id004 !ruby/object:Gem::Requirement 
+  type: :development
+  version_requirements: *id004
+- !ruby/object:Gem::Dependency 
+  name: fakeweb
+  requirement: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -79,7 +91,9 @@ dependencies:
         segments: 
         - 0
         version: "0"
-  requirement: *id004
+  prerelease: false
+  type: :development
+  version_requirements: *id005
 description: This gem keep an eye on every Net::HTTP library usage and dump all request and response data to the log file
 email: agresso@gmail.com
 executables: []