api_hammer 0.13.2 → 0.13.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0dc5455b30c8dc15be3424b468aa3fce4816deae
-  data.tar.gz: 035a1c11f8da0d6d57d23a1af044e1db1c463fb1
+  metadata.gz: 0ec8f15a064560aa1f4f50123bbcf2a334130488
+  data.tar.gz: c29ed9ef11362ef5a872d617d656b3a56c879198
 SHA512:
-  metadata.gz: 379b3299919248418c89c1969f82176d8e84354b1175e09fb5a5dea3463d51901f462f610d8f734fa5b60966286bccfeef7bc5bdd2effef8c4b7e52c59149689
-  data.tar.gz: 6ba8b12bf7906f3c6933aa83f5c05bbc7ff7bd65e1e73f0f876d9783ff95b2649ed2b5937ba427b1d1383e504a333adee92ea75b499ad1d2878ca06cf8af8ad0
+  metadata.gz: c681350085a2cbcfbfc208f988ce91700e98b5350a1bd13d26ca06008a256e5a159c40b88b2f236944097549c334908b05af1ae30ce907d4421187f76d84bf2e
+  data.tar.gz: 92ae18c0f12d10abb11829e129fcd648142910b0ccbcbcaeeaf06ec072785c4bcfea46841002e3bef16149859652a32b40273d7d52ef87b14f3fa1391abce6da

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+# v0.13.3
+- fix bug when logging non-ascii bodies with filtration enabled
+
 # v0.13.2
 - fix with_indifferent_access usage when we don't depend on activesupport
 

data/lib/api_hammer/request_logger.rb

@@ -117,7 +117,7 @@ module ApiHammer
         if content_type_attrs.text?
           if (400..599).include?(status.to_i) || body.size < LARGE_BODY_SIZE
             # log bodies if they are not large, or if there was an error (either client or server) 
-            data[role]['body'] = parsed_body.jsonifiable.filtered(@options.reject { |k,v| ![:filter_keys].include?(k) }).body
+            data[role]['body'] = parsed_body.filtered(@options.reject { |k,v| ![:filter_keys].include?(k) }).jsonifiable.body
           else
             # otherwise, log id and uuid fields 
             body_object = parsed_body.object

data/lib/api_hammer/version.rb

@@ -1,3 +1,3 @@
 module ApiHammer
-  VERSION = "0.13.2"
+  VERSION = "0.13.3"
 end

data/test/request_logger_test.rb

@@ -1,3 +1,4 @@
+# -*- coding: utf-8 -*-
 proc { |p| $:.unshift(p) unless $:.any? { |lp| File.expand_path(lp) == p } }.call(File.expand_path('.', File.dirname(__FILE__)))
 require 'helper'
 require 'logger'
@@ -66,6 +67,16 @@ describe ApiHammer::RequestLogger do
 
         assert_includes(logio.string, %q("body":"{\"pin\": \"[FILTERED]\"}"))
       end
+
+      it 'filters unicode' do
+        body = %Q({"key": "Björn"})
+        app = proc { |env| [200, {"Content-Type" => 'application/json; charset=utf-8'}, [body]] }
+        app = ApiHammer::RequestLogger.new(app, logger)
+        app.call(Rack::MockRequest.env_for('/')).last.close
+
+        assert_includes(logio.string, %q("body":"{\"key\": \"Björn\"}"))
+      end
+
       it 'filters nested' do
         body = %Q({"object": {"pin": "foobar"}})
         app = proc { |env| [200, {"Content-Type" => 'application/json; charset=UTF8'}, [body]] }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: api_hammer
 version: !ruby/object:Gem::Version
-  version: 0.13.2
+  version: 0.13.3
 platform: ruby
 authors:
 - Ethan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-28 00:00:00.000000000 Z
+date: 2016-04-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack