httplog 1.4.3 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9ce910d7c04847c28f836de26906c0c50aa4642b553b298d15067bf6ee25db56
-  data.tar.gz: 8600e756f28e033b6e39ac91235f69bf772fe467b05040d88c02ce54e982ef0b
+  metadata.gz: 97cbba2cdb0af09dcf6664e290981b074a40f9c91d81be74a737a746bfc1e612
+  data.tar.gz: 228590a39ab68d89d3261f219e2bac4945f4fde315219943cbbacdfc1aaa2922
 SHA512:
-  metadata.gz: 6a76055079c7bfa4a229fda0861f90276c94cb64859ae7ce3c269e86656f3ad69fb3d5e8ed7eb2b82e9fe1facb0a73619951f099aafbbac7b0b503a52eb21046
-  data.tar.gz: f541eec119d175cc50563bc2673d60dae6246a43bcf570f9fe4db9a6295697ea94f7ffddb709aa8c9d0264f274f48c42bca3322d3364c1edd47298a984421363
+  metadata.gz: d857f2dc7142fb7243371afd4d3993b65ced6092ecb6ac361abbd2957bc1884e18131ca707b3e2c9dc694ad9996c09e12ddf7b31f978d8d55def35322159a861
+  data.tar.gz: eafbb1ef94500b468083f8073049749e6073d2ec423fadae8f6aca3221624bb914f28a489305f35cd5fb222b247eb81f5b2446f4694f919959267d83c0bfcbe7

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+## 1.5.0 - 2021-05-20
+
+* Support for Ruby 2.7 and frozen strings
+* Development dependency updates
+* Dropped support for net/http v3
+* Performance tweaks
+* Fix for RestClient body read issue (WARNING: this may be reverted, see [#105](https://github.com/trusche/httplog/issues/105))
+
 ## 1.4.3 - 2020-06-10
 
 * Masking `password` parameter by default... doh.

data/README.md

@@ -1,7 +1,5 @@
 ## httplog
 
-**VERSION 1.4.0 and 1.4.1 HAVE BEEN YANKED** from rubygems.org due to [this issue](https://github.com/trusche/httplog/issues/89), please update to version 1.4.2 or higher. Sorry about that...
-
 [![Gem Version](https://badge.fury.io/rb/httplog.svg)](http://badge.fury.io/rb/httplog) [![Build Status](https://travis-ci.org/trusche/httplog.svg?branch=master)](https://travis-ci.org/trusche/httplog) [![Code Climate](https://codeclimate.com/github/trusche/httplog.svg)](https://codeclimate.com/github/trusche/httplog)
 [![Release Version](https://img.shields.io/github/release/trusche/httplog.svg)](https://img.shields.io/github/release/trusche/httplog.svg)
 <a href="https://www.bearer.sh?ref=httplog"><img src="/bearer-badge.png" height="20px"/></a>
@@ -12,7 +10,7 @@ Requires ruby >= 2.5
 
 This gem works with the following ruby modules and libraries:
 
-* [Net::HTTP](http://www.ruby-doc.org/stdlib-1.9.3/libdoc/net/http/rdoc/index.html)
+* [Net::HTTP](http://www.ruby-doc.org/stdlib-1.9.3/libdoc/net/http/rdoc/index.html) v4+
 * [Ethon](https://github.com/typhoeus/ethon)
 * [Excon](https://github.com/geemus/excon)
 * [OpenURI](http://www.ruby-doc.org/stdlib-1.9.3/libdoc/open-uri/rdoc/index.html)
@@ -103,6 +101,9 @@ HttpLog.configure do |config|
 
   # Mask the values of sensitive request parameters
   config.filter_parameters = %w[password]
+  
+  # Customize the prefix with a proc or lambda
+  config.prefix = ->{ "[httplog] #{Time.now} " }
 end
 ```
 
@@ -267,9 +268,6 @@ a suggestion for a fix, please open an issue or, even better, submit a pull requ
 
 * Benchmarking only covers the time between starting the HTTP request and receiving the response. It does *not* cover the time it takes to establish the TCP connection.
 
-* When using [REST Client](https://github.com/rest-client/rest-client), POST requests might be missing the requests
-  data. See #54 for details.
-
 ### Running the specs
 
 Make sure you have the necessary dependencies installed by running `bundle install`.

data/httplog.gemspec

@@ -26,14 +26,15 @@ Gem::Specification.new do |gem|
 
   gem.add_development_dependency 'ethon', ['~> 0.11']
   gem.add_development_dependency 'excon', ['~> 0.60']
-  gem.add_development_dependency 'faraday', ['~> 0.14']
+  gem.add_development_dependency 'faraday', ['~> 1.3']
   gem.add_development_dependency 'guard-rspec', ['~> 4.7']
   gem.add_development_dependency 'http', ['~> 4.0']
   gem.add_development_dependency 'httparty', ['~> 0.16']
   gem.add_development_dependency 'httpclient', ['~> 2.8']
+  gem.add_development_dependency 'rest-client', ['~> 2.0']
   gem.add_development_dependency 'listen', ['~> 3.0']
   gem.add_development_dependency 'patron', ['~> 0.12']
-  gem.add_development_dependency 'rake', ['~> 12.3']
+  gem.add_development_dependency 'rake', ['~> 13.0']
   gem.add_development_dependency 'rspec', ['~> 3.7']
   gem.add_development_dependency 'simplecov', ['~> 0.15']
   gem.add_development_dependency 'thin', ['~> 1.7']

data/lib/httplog/adapters/net_http.rb

@@ -11,12 +11,22 @@ module Net
       bm = Benchmark.realtime do
         @response = orig_request(req, body, &block)
       end
+      body_stream  = req.body_stream
+      request_body = if body_stream
+                       body_stream.to_s # read and rewind for RestClient::Payload::Base
+                       body_stream.rewind if body_stream.respond_to?(:rewind) # RestClient::Payload::Base has no method rewind
+                       body_stream.read
+                     elsif req.body.nil? || req.body.empty?
+                       body
+                     else
+                       req.body
+                     end
 
       if HttpLog.url_approved?(url) && started?
         HttpLog.call(
           method: req.method,
           url: url,
-          request_body: req.body.nil? || req.body.empty? ? body : req.body,
+          request_body: request_body,
           request_headers: req.each_header.collect,
           response_code: @response.code,
           response_body: @response.body,

data/lib/httplog/http_log.rb

@@ -121,22 +121,24 @@ module HttpLog
         raise BodyParsingError, '(not available yet)'
       end
 
-      body = body.to_s if defined?(HTTP::Response::Body) && body.is_a?(HTTP::Response::Body)
-      body = body.dup
+      body_copy = body.dup
+      body_copy = body.to_s if defined?(HTTP::Response::Body) && body.is_a?(HTTP::Response::Body)
+      return nil if body_copy.nil? || body_copy.empty?
 
-      if encoding =~ /gzip/ && body && !body.empty?
+
+      if encoding =~ /gzip/
         begin
-          sio = StringIO.new(body.to_s)
+          sio = StringIO.new(body_copy.to_s)
           gz = Zlib::GzipReader.new(sio)
-          body = gz.read
+          body_copy = gz.read
         rescue Zlib::GzipFile::Error
           log("(gzip decompression failed)")
         end
       end
 
-      result = utf_encoded(body.to_s, content_type)
+      result = utf_encoded(body_copy.to_s, content_type)
 
-      if mask_body && body && !body.empty?
+      if mask_body
         if content_type =~ /json/
           result = begin
                      masked_data config.json_parser.load(result)
@@ -259,12 +261,14 @@ module HttpLog
       # in its entirety.
       return (config.filter_parameters.include?(key.downcase) ? PARAM_MASK : msg) if key
 
-      # Otherwise, we'll parse Strings for key=valye pairs...
+      # Otherwise, we'll parse Strings for key=value pairs,
+      # for name="key"\n value...
       case msg
       when *string_classes
         config.filter_parameters.reduce(msg) do |m,key|
           scrubbed = m.to_s.encode('UTF-8', invalid: :replace, undef: :replace)
-          scrubbed.gsub(/(#{key})=[^&]+/i, "#{key}=#{PARAM_MASK}")
+          scrubbed.to_s.gsub(/(#{key})=[^&]+/i, "#{key}=#{PARAM_MASK}")
+            .gsub(/name="#{key}"\s+\K[\s\w]+/, "#{PARAM_MASK}\r\n") # multi-part Faraday
         end
       # ...and recurse over hashes
       when *hash_classes

data/lib/httplog/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HttpLog
-  VERSION = '1.4.3'.freeze
+  VERSION = '1.5.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: httplog
 version: !ruby/object:Gem::Version
-  version: 1.4.3
+  version: 1.5.0
 platform: ruby
 authors:
 - Thilo Rusche
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-06-11 00:00:00.000000000 Z
+date: 2021-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ethon
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '1.3'
 - !ruby/object:Gem::Dependency
   name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
@@ -108,6 +108,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.8'
+- !ruby/object:Gem::Dependency
+  name: rest-client
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: listen
   requirement: !ruby/object:Gem::Requirement
@@ -142,14 +156,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.3'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.3'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -274,7 +288,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: Log outgoing HTTP requests.