fluent-plugin-rewrite 0.0.8 → 0.0.9

data/fluent-plugin-rewrite.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |gem|
   gem.name          = "fluent-plugin-rewrite"
-  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-rewrite"

data/lib/fluent/plugin/out_rewrite.rb

@@ -41,6 +41,8 @@ module Fluent
     end
 
     def emit(tag, es, chain)
+      _tag = tag.clone
+
       if @remove_prefix and
         ((tag.start_with?(@removed_prefix_string) && tag.length > @removed_length) || tag == @remove_prefix)
         tag = tag[@removed_length..-1] || ''
@@ -52,7 +54,11 @@ module Fluent
 
       es.each do |time, record|
         filtered_tag, record = rewrite(tag, record)
-        Engine.emit(filtered_tag, time, record) if filtered_tag && record
+        if filtered_tag && record && _tag != filtered_tag
+          Engine.emit(filtered_tag, time, record)
+        else
+          $log.warn "Can not emit message because the tag(#{tag}) has not changed. Dropped record #{record}"
+        end
       end
 
       chain.next

data/test/plugin/test_out_rewrite.rb

@@ -289,6 +289,8 @@ class RewriteOutputTest < Test::Unit::TestCase
     assert_equal({ "path" => "/entries/1" }, emits[3][2])
 
     d2 = create_driver(%[
+      add_prefix filtered
+
       <rule>
         key     path
         pattern \\?.+$
@@ -301,7 +303,38 @@ class RewriteOutputTest < Test::Unit::TestCase
     emits = d2.emits
 
     assert_equal 1, emits.size
-    assert_equal('test', emits[0][0])
+    assert_equal('filtered.test', emits[0][0])
     assert_equal({ "path" => "/foo" }, emits[0][2])
+
+    # Test for not emit if the tag has not changed.
+    d3 = create_driver(%[
+      <rule>
+        key     path
+        pattern \\?.+$
+        replace
+      </rule>
+    ])
+    d3.run do
+      d3.emit({ "path" => "/foo?bar=1" })
+    end
+
+    assert_equal 0, d3.emits.size
+
+    # Emit message if the rewrite tag rules have been defined, even if (add|remove)_prefix option is not set.
+    d4 = create_driver(%[
+      <rule>
+        key           path
+        pattern       ^\/(users|entries)
+        append_to_tag true
+      </rule>
+    ])
+    d4.run do
+      d4.emit({ "path" => "/users/studio3104" })
+    end
+    emits = d4.emits
+
+    assert_equal 1, emits.size
+    assert_equal('test.users', emits[0][0])
+    assert_equal({ "path" => "/users/studio3104" }, emits[0][2])
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-rewrite
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.9
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-07-04 00:00:00.000000000 Z
+date: 2013-01-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -104,4 +104,3 @@ summary: Fluentd plugin to rewrite tags/values along with pattern matching and r
 test_files:
 - test/plugin/test_out_rewrite.rb
 - test/test_helper.rb
-has_rdoc: