http_logger 0.2.1 → 0.2.2

data/VERSION

@@ -1 +1 @@
-0.2.1
+0.2.2

data/http_logger.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "http_logger"
-  s.version = "0.2.1"
+  s.version = "0.2.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Bogdan Gusiev"]
-  s.date = "2012-04-13"
+  s.date = "2012-07-26"
   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 = [

data/lib/http_logger.rb

@@ -1,4 +1,5 @@
 require 'net/http'
+require "uri"
 
 # Usage:
 #
@@ -38,7 +39,7 @@ class Net::HTTP
     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)
+      log("HTTP #{request.method} (%0.2fms)" % (ofset * 1000), URI.decode(url))
       request.each_capitalized { |k,v| log("HTTP request header", "#{k}: #{v}") } if self.class.log_headers
       if request.is_a?(::Net::HTTP::Post) || request.is_a?(::Net::HTTP::Put)
         log("#{request.class.to_s.upcase} params", request.body)

data/spec/http_logger_spec.rb

@@ -11,7 +11,7 @@ describe "HttpLogger" do
   end
 
   let(:url) { "http://google.com/" }
-  let(:uri) { URI.parse("http://google.com/") }
+  let(:uri) { URI.parse(url) }
   let(:request) do
     Net::HTTP.get_response(uri)
   end
@@ -23,6 +23,14 @@ describe "HttpLogger" do
 
   it { should_not be_empty }
 
+  context "when url has escaped chars" do
+
+    let(:url) { "http://google.com?query=a%20b"}
+
+    it { subject.should include("query=a b")}
+    
+  end
+
   context "when headers logging is on" do
 
     before(:each) do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: http_logger
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-13 00:00:00.000000000 Z
+date: 2012-07-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &14480200 !ruby/object:Gem::Requirement
+  requirement: &22052400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *14480200
+  version_requirements: *22052400
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &14479200 !ruby/object:Gem::Requirement
+  requirement: &22070740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *14479200
+  version_requirements: *22070740
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &15004260 !ruby/object:Gem::Requirement
+  requirement: &22098840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *15004260
+  version_requirements: *22098840
 - !ruby/object:Gem::Dependency
   name: fakeweb
-  requirement: &15003560 !ruby/object:Gem::Requirement
+  requirement: &22133980 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +54,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *15003560
+  version_requirements: *22133980
 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
@@ -93,7 +93,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 734133790092156030
+      hash: 869087870965774398
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: