fluent-plugin-extract_query_params 0.0.8 → 0.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1396de021b24f6213a916e19163285ae71c510fa
-  data.tar.gz: 142ad5a3809f685a61a197cdf055af0a1aa295e9
+  metadata.gz: 243cee66d6590dad3efd4c9be522b08a6da6052f
+  data.tar.gz: 82c9372e02ae589f74b3c5227296b61e64828bdb
 SHA512:
-  metadata.gz: 50f99f2bdd7ca5d0f83d197c0a8d7beb39c491f160b7dfde64a388023a51e9e971e2f2173db2b88d593b5c3e00d23f5dbeb3afd4c8156feeac110a995090d04f
-  data.tar.gz: f92c5e653b1311665e7ec04e39631a453cbc88bd55195f69e56bd559dcbfbfbb124b6f544a97b5af887a697649dd1c2d2e25ae408726d2fe3c5928b6ad02f770
+  metadata.gz: 90f9964ec60de897f67db8763daa866c571f570fd060d05cf9ec2db82724a11b46bc2fbf3a2ea3615c7dbfd89267b6ed6e7172304e1e8fad36b15707a04b9443
+  data.tar.gz: 3b10f330ea23bf970870276ffd57f0455aac173cbd2095d6af95337189231d4cc64ceebefc4e178f413c95512ccf042a959dfe3982c0dbfbe0ad98a0821ada64

data/fluent-plugin-extract_query_params.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |gem|
   gem.name          = 'fluent-plugin-extract_query_params'
-  gem.version       = '0.0.8'
+  gem.version       = '0.0.9'
   gem.authors       = ['Kentaro Kuribayashi']
   gem.email         = ['kentarok@gmail.com']
   gem.homepage      = 'http://github.com/kentaro/fluent-plugin-extract_query_params'

data/lib/fluent/plugin/out_extract_query_params.rb

@@ -90,7 +90,7 @@ module Fluent
 
           unless url.query.nil?
             url.query.split('&').each do |pair|
-              key, value = pair.split('=').map { |i| URI.unescape(i) }
+              key, value = pair.split('=', 2).map { |i| URI.unescape(i) }
               next if (key.nil? || key.empty?) && (!@permit_blank_key || value.nil? || value.empty?)
               key ||= ''
               value ||= ''

data/test/plugin/test_out_extract_query_params.rb

@@ -202,6 +202,10 @@ class ExtractQueryParamsOutputTest < Test::Unit::TestCase
   DIRTY_PATH_KEY_ONLY_3 = '/dummy?baz=qux&foo'
   DIRTY_PATH_VALUE_ONLY_1 = '/dummy?=bar&baz=qux'
   DIRTY_PATH_VALUE_ONLY_2 = '/dummy?baz=qux&=bar'
+  DIRTY_PATH_BASE64_1 = '/dummy?foo=ZXh0cmE=&baz=qux'
+  DIRTY_PATH_BASE64_2 = '/dummy?baz=qux&foo=ZXh0cmE='
+  DIRTY_PATH_BASE64_3 = '/dummy?foo=cGFkZGluZw==&baz=qux'
+  DIRTY_PATH_BASE64_4 = '/dummy?baz=qux&foo=cGFkZGluZw=='
 
   def test_emit_with_dirty_paths
     d = create_driver(%[
@@ -218,10 +222,14 @@ class ExtractQueryParamsOutputTest < Test::Unit::TestCase
       d.emit({ 'path' => DIRTY_PATH_KEY_ONLY_3 })
       d.emit({ 'path' => DIRTY_PATH_VALUE_ONLY_1 })
       d.emit({ 'path' => DIRTY_PATH_VALUE_ONLY_2 })
+      d.emit({ 'path' => DIRTY_PATH_BASE64_1 })
+      d.emit({ 'path' => DIRTY_PATH_BASE64_2 })
+      d.emit({ 'path' => DIRTY_PATH_BASE64_3 })
+      d.emit({ 'path' => DIRTY_PATH_BASE64_4 })
     }
     emits = d.emits
 
-    assert_equal 9, emits.count
+    assert_equal 13, emits.count
 
     r = emits.shift[2]
     assert_equal 2, r.size
@@ -271,6 +279,30 @@ class ExtractQueryParamsOutputTest < Test::Unit::TestCase
     assert_equal 2, r.size
     assert_equal DIRTY_PATH_VALUE_ONLY_2, r['path']
     assert_equal 'qux',                   r['baz']
+
+    r = emits.shift[2]
+    assert_equal 3, r.size
+    assert_equal DIRTY_PATH_BASE64_1, r['path']
+    assert_equal 'qux',               r['baz']
+    assert_equal 'ZXh0cmE=',          r['foo']
+
+    r = emits.shift[2]
+    assert_equal 3, r.size
+    assert_equal DIRTY_PATH_BASE64_2, r['path']
+    assert_equal 'qux',               r['baz']
+    assert_equal 'ZXh0cmE=',          r['foo']
+
+    r = emits.shift[2]
+    assert_equal 3, r.size
+    assert_equal DIRTY_PATH_BASE64_3, r['path']
+    assert_equal 'qux',               r['baz']
+    assert_equal 'cGFkZGluZw==',      r['foo']
+
+    r = emits.shift[2]
+    assert_equal 3, r.size
+    assert_equal DIRTY_PATH_BASE64_4, r['path']
+    assert_equal 'qux',               r['baz']
+    assert_equal 'cGFkZGluZw==',      r['foo']
   end
 
   def test_emit_with_permit_blank_key

metadata

@@ -1,41 +1,41 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-extract_query_params
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.9
 platform: ruby
 authors:
 - Kentaro Kuribayashi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-20 00:00:00.000000000 Z
+date: 2015-04-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   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: fluentd
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Fluentd plugin to extract key/values from URL query parameters.
@@ -45,7 +45,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".gitignore"
 - Gemfile
 - LICENSE
 - README.md
@@ -64,17 +64,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.0.14
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Fluentd plugin to extract key/values from URL query parameters