fluent-plugin-detect-exceptions 0.0.10 → 0.0.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 22408c782d12b0ae5b6c632e67b4d2145272e137
-  data.tar.gz: e2ab10c60dd80aab406de9b12135d6abcc1ff45e
+SHA256:
+  metadata.gz: f846941463ccc37acd71280e182d5ae7b0ffd0c95c609bfbfe67de64f86ff478
+  data.tar.gz: 1454c4ddae5ba4efa214972f2109f0a71d4984e87b49cefa3541cb87d92d6cc7
 SHA512:
-  metadata.gz: 01ed3e8e3e473ed25e2c6f9c4979bcb57453524c5bf559a337ab9acea679e8224acd07c239daf9313cee067061a67106a1953f7dc5cb2f3d8345e10996923201
-  data.tar.gz: e1ba01791f3ba5c10d8b402efd1e678724cacdb68868ebf644f57e0f19b0cacd18dc27f869e707039d4a82f05d6894b3d78f6a377a6bf9578aae02552b48dcb3
+  metadata.gz: 585f0de72a4f845f39c0f7e753ca3661ad335b10c18d26930afc19bab246b205cb9d68d1fa506c87e5ea674fc9b1ce51b04c278aed5590d997875ea9b50b21d8
+  data.tar.gz: 987c453e5667c1736507500859a1f46612a56451f791afaf194574380afd31527da73f9ebd87de53faf2c40e5929ce0cdd0f2c3fbd0c9d4c89a2137bad40a9a1

data/README.rdoc

@@ -42,6 +42,16 @@ will also install and configure the gem.
 
 The plugin supports the following parameters:
 
+=== Required
+
+[remove_tag_prefix]  The prefix to remove from the input tag when outputting
+                     a record. A prefix has to be a complete tag part.
+                     Example: If remove_tag_prefix is set to 'foo', the input
+                     tag foo.bar.baz is transformed to bar.baz and the input tag
+                     'foofoo.bar' is not modified.
+
+=== Optional
+
 [message]  Name of the field in the JSON record that contains the
            single-line log messages that shall be scanned for exceptions.
            If this is set to '', the plugin will try 'message' and 'log',
@@ -49,12 +59,6 @@ The plugin supports the following parameters:
            This parameter is only applicable to structured (JSON) log streams.
            Default: ''.
 
-[remove_tag_prefix]  The prefix to remove from the input tag when outputting
-                     a record. A prefix has to be a complete tag part.
-                     Example: If remove_tag_prefix is set to 'foo', the input
-                     tag foo.bar.baz is transformed to bar.baz and the input tag
-                     'foofoo.bar' is not modified. Default: empty string.
-
 [languages]  A list of language for which exception stack traces shall be
              detected. The values in the list can be separated by commas or
              written as JSON list.
@@ -66,6 +70,12 @@ The plugin supports the following parameters:
                             forwarded. If not set, incomplete exceptions stacks
                             are not flushed.
 
+[force_line_breaks]  Force line breaks between each lines when comibining exception stacks.
+                     This is useful if your exception is formatted
+                     as a single line. i.e., logs retrieved from the docker's
+                     logging driver don't have any line break.
+                     Default: false.
+
 [max_lines]  Maximum number of lines in a detected exception stack trace.
              If this maximum number is exceeded, the exception stack trace
              that has been detected so far will be output as a single
@@ -98,12 +108,22 @@ The plugin supports the following parameters:
 Example configuration:
 
     <match **>
+      @type detect_exceptions
       remove_tag_prefix foo
       message log
       languages java, python
       multiline_flush_interval 0.1
     </match>
 
+== Extending language support
+
+Supporting a new language requires new detection rules in this gem and
+additional changes in the
+{Stackdriver Error Reporting}[https://cloud.google.com/error-reporting/]
+service. Please contact our product team by
+{filing a support case}[https://cloud.google.com/support-hub/#google-cloud-platform]
+if you'd like to see support for a new language.
+
 == Copyright
 
 Copyright:: Copyright 2016 Google Inc. All rights reserved.

data/Rakefile

@@ -24,15 +24,25 @@ Rake::TestTask.new(:test) do |test|
 end
 
 # Building the gem will use the local file mode, so ensure it's world-readable.
-desc 'Check plugin file permissions'
-task :check_perms do
-  plugin = 'lib/fluent/plugin/out_detect_exceptions.rb'
-  mode = File.stat(plugin).mode & 0o777
-  raise "Unexpected mode #{mode.to_s(8)} for #{plugin}" unless
-    mode & 0o444 == 0o444
+# https://github.com/GoogleCloudPlatform/fluent-plugin-detect-exceptions/issues/32
+desc 'Fix file permissions'
+task :fix_perms do
+  files = [
+    'lib/fluent/plugin/*.rb'
+  ].flat_map do |file|
+    file.include?('*') ? Dir.glob(file) : [file]
+  end
+
+  files.each do |file|
+    mode = File.stat(file).mode & 0o777
+    next unless mode & 0o444 != 0o444
+    puts "Changing mode of #{file} from #{mode.to_s(8)} to "\
+         "#{(mode | 0o444).to_s(8)}"
+    chmod mode | 0o444, file
+  end
 end
 
 desc 'Run unit tests and RuboCop to check for style violations'
-task all: [:test, :rubocop, :check_perms]
+task all: [:test, :rubocop, :fix_perms]
 
 task default: :all

data/fluent-plugin-detect-exceptions.gemspec

@@ -11,9 +11,9 @@ eos
   gem.homepage      = \
     'https://github.com/GoogleCloudPlatform/fluent-plugin-detect-exceptions'
   gem.license       = 'Apache-2.0'
-  gem.version       = '0.0.10'
-  gem.authors       = ['Thomas Schickinger', 'Igor Peshansky']
-  gem.email         = ['schickin@google.com', 'igorp@google.com']
+  gem.version       = '0.0.14'
+  gem.authors       = ['Stackdriver Agents']
+  gem.email         = ['stackdriver-agents@google.com']
   gem.required_ruby_version = Gem::Requirement.new('>= 2.0')
 
   gem.files         = Dir['**/*'].keep_if { |file| File.file?(file) }

data/lib/fluent/plugin/exception_detector.rb

@@ -13,7 +13,7 @@
 # limitations under the License.
 #
 module Fluent
-  Struct.new('Rule', :from_state, :pattern, :to_state)
+  Struct.new('Rule', :from_states, :pattern, :to_state)
 
   # Configuration of the state machine that detects exceptions.
   module ExceptionDetectorConfig
@@ -41,8 +41,10 @@ module Fluent
       end
     end
 
-    def self.rule(from_state, pattern, to_state)
-      Struct::Rule.new(from_state, pattern, to_state)
+    def self.rule(from_state_or_states, pattern, to_state)
+      from_state_or_states = [from_state_or_states] unless
+        from_state_or_states.is_a?(Array)
+      Struct::Rule.new(from_state_or_states, pattern, to_state)
     end
 
     def self.supported
@@ -50,12 +52,29 @@ module Fluent
     end
 
     JAVA_RULES = [
-      rule(:start_state,
+      rule([:start_state, :java_start_exception],
            /(?:Exception|Error|Throwable|V8 errors stack trace)[:\r\n]/,
+           :java_after_exception),
+      rule(:java_after_exception, /^[\t ]*nested exception is:[\t ]*/,
+           :java_start_exception),
+      rule(:java_after_exception, /^[\r\n]*$/, :java_after_exception),
+      rule([:java_after_exception, :java], /^[\t ]+(?:eval )?at /, :java),
+
+      rule([:java_after_exception, :java],
+           # C# nested exception.
+           /^[\t ]+--- End of inner exception stack trace ---$/,
            :java),
-      rule(:java, /^[\t ]+(?:eval )?at /, :java),
-      rule(:java, /^[\t ]*(?:Caused by|Suppressed):/, :java),
-      rule(:java, /^[\t ]*... \d+\ more/, :java)
+
+      rule([:java_after_exception, :java],
+           # C# exception from async code.
+           /^--- End of stack trace from previous (?x:
+           )location where exception was thrown ---$/,
+           :java),
+
+      rule([:java_after_exception, :java], /^[\t ]*(?:Caused by|Suppressed):/,
+           :java_after_exception),
+      rule([:java_after_exception, :java],
+           /^[\t ]*... \d+ (?:more|common frames omitted)/, :java)
     ].freeze
 
     PYTHON_RULES = [
@@ -76,13 +95,14 @@ module Fluent
 
     GO_RULES = [
       rule(:start_state, /\bpanic: /, :go_after_panic),
+      rule(:start_state, /http: panic serving/, :go_goroutine),
       rule(:go_after_panic, /^$/, :go_goroutine),
+      rule([:go_after_panic, :go_after_signal, :go_frame_1],
+           /^$/, :go_goroutine),
       rule(:go_after_panic, /^\[signal /, :go_after_signal),
-      rule(:go_after_signal, /^$/, :go_goroutine),
       rule(:go_goroutine, /^goroutine \d+ \[[^\]]+\]:$/, :go_frame_1),
       rule(:go_frame_1, /^(?:[^\s.:]+\.)*[^\s.():]+\(|^created by /,
            :go_frame_2),
-      rule(:go_frame_1, /^$/, :go_goroutine),
       rule(:go_frame_2, /^\s/, :go_frame_1)
     ].freeze
 
@@ -169,7 +189,9 @@ module Fluent
         rule_config.each do |r|
           target = ExceptionDetectorConfig::RuleTarget.new(r[:pattern],
                                                            r[:to_state])
-          @rules[r[:from_state]] << target
+          r[:from_states].each do |from_state|
+            @rules[from_state] << target
+          end
         end
       end
 
@@ -234,20 +256,22 @@ module Fluent
     # message_field may contain the empty string. In this case, the
     # TraceAccumulator 'learns' the field name from the first record by checking
     # for some pre-defined common field names of text logs.
-    # The named parameters max_lines and max_bytes limit the maximum amount
+    # The option parameter can be used to pass the following parameters:
+    # force_line_breaks adds line breaks when combining exception stacks
+    # max_lines and max_bytes limit the maximum amount
     # of data to be buffered. The default value 0 indicates 'no limit'.
-    def initialize(message_field, languages, max_lines: 0, max_bytes: 0,
-                   &emit_callback)
+    def initialize(message_field, languages, **options, &emit_callback)
       @exception_detector = Fluent::ExceptionDetector.new(*languages)
-      @max_lines = max_lines
-      @max_bytes = max_bytes
       @message_field = message_field
+      @force_line_breaks = options[:force_line_breaks] || false
+      @max_lines = options[:max_lines]           || 0
+      @max_bytes = options[:max_bytes]           || 0
+      @emit = emit_callback
       @messages = []
       @buffer_start_time = Time.now
       @buffer_size = 0
       @first_record = nil
       @first_timestamp = nil
-      @emit = emit_callback
     end
 
     def push(time_sec, record)
@@ -338,8 +362,14 @@ module Fluent
         @buffer_start_time = Time.now
       end
       unless message.nil?
-        @messages << message
-        @buffer_size += message.length
+        message_with_line_break =
+          if @force_line_breaks && !@messages.empty? && !message.include?("\n")
+            "\n" + message
+          else
+            message
+          end
+        @messages << message_with_line_break
+        @buffer_size += message_with_line_break.length
       end
     end
   end

data/lib/fluent/plugin/out_detect_exceptions.rb

@@ -22,14 +22,16 @@ module Fluent
   # an exception stack trace, they forwarded as a single, combined JSON
   # object. Otherwise, the input log data is forwarded as is.
   class DetectExceptionsOutput < Output
+    desc 'The prefix to be removed from the input tag when outputting a record.'
+    config_param :remove_tag_prefix, :string
     desc 'The field which contains the raw message text in the input JSON data.'
     config_param :message, :string, default: ''
-    desc 'The prefix to be removed from the input tag when outputting a record.'
-    config_param :remove_tag_prefix, :string, default: ''
     desc 'The interval of flushing the buffer for multiline format.'
     config_param :multiline_flush_interval, :time, default: nil
     desc 'Programming languages for which to detect exceptions. Default: all.'
     config_param :languages, :array, value_type: :string, default: []
+    desc 'Force live breaks when combining exception stacks. Default: false.'
+    config_param :force_line_breaks, :bool, default: false
     desc 'Maximum number of lines to flush (0 means no limit). Default: 1000.'
     config_param :max_lines, :integer, default: 1000
     desc 'Maximum number of bytes to flush (0 means no limit). Default: 0.'
@@ -91,9 +93,13 @@ module Fluent
         unless @accumulators.key?(log_id)
           out_tag = tag.sub(/^#{Regexp.escape(@remove_tag_prefix)}\./, '')
           @accumulators[log_id] =
-            Fluent::TraceAccumulator.new(@message, @languages,
-                                         max_lines: @max_lines,
-                                         max_bytes: @max_bytes) do |t, r|
+            Fluent::TraceAccumulator.new(
+              @message,
+              @languages,
+              force_line_breaks: @force_line_breaks,
+              max_lines: @max_lines,
+              max_bytes: @max_bytes
+            ) do |t, r|
               router.emit(out_tag, t, r)
             end
         end

data/test/plugin/test_exception_detector.rb

@@ -59,7 +59,35 @@ Caused by: com.example.myproject.MyProjectServletException
     at org.mortbay.jetty.servlet.ServletHolder.handle(ServletHolder.java:511)
     at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1166)
     at com.example.myproject.OpenSessionInViewFilter.doFilter(OpenSessionInViewFilter.java:30)
-    ... 27 more
+    ... 27 common frames omitted
+END
+
+  NESTED_JAVA_EXC = <<END.freeze
+java.lang.RuntimeException: javax.mail.SendFailedException: Invalid Addresses;
+  nested exception is:
+com.sun.mail.smtp.SMTPAddressFailedException: 550 5.7.1 <[REDACTED_EMAIL_ADDRESS]>... Relaying denied
+
+	at com.nethunt.crm.api.server.adminsync.AutomaticEmailFacade.sendWithSmtp(AutomaticEmailFacade.java:236)
+	at com.nethunt.crm.api.server.adminsync.AutomaticEmailFacade.sendSingleEmail(AutomaticEmailFacade.java:285)
+	at com.nethunt.crm.api.server.adminsync.AutomaticEmailFacade.lambda$sendSingleEmail$3(AutomaticEmailFacade.java:254)
+	at java.util.Optional.ifPresent(Optional.java:159)
+	at com.nethunt.crm.api.server.adminsync.AutomaticEmailFacade.sendSingleEmail(AutomaticEmailFacade.java:253)
+	at com.nethunt.crm.api.server.adminsync.AutomaticEmailFacade.sendSingleEmail(AutomaticEmailFacade.java:249)
+	at com.nethunt.crm.api.email.EmailSender.lambda$notifyPerson$0(EmailSender.java:80)
+	at com.nethunt.crm.api.util.ManagedExecutor.lambda$execute$0(ManagedExecutor.java:36)
+	at com.nethunt.crm.api.util.RequestContextActivator.lambda$withRequestContext$0(RequestContextActivator.java:36)
+	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
+	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
+	at java.base/java.lang.Thread.run(Thread.java:748)
+Caused by: javax.mail.SendFailedException: Invalid Addresses;
+  nested exception is:
+com.sun.mail.smtp.SMTPAddressFailedException: 550 5.7.1 <[REDACTED_EMAIL_ADDRESS]>... Relaying denied
+
+	at com.sun.mail.smtp.SMTPTransport.rcptTo(SMTPTransport.java:2064)
+	at com.sun.mail.smtp.SMTPTransport.sendMessage(SMTPTransport.java:1286)
+	at com.nethunt.crm.api.server.adminsync.AutomaticEmailFacade.sendWithSmtp(AutomaticEmailFacade.java:229)
+	... 12 more
+Caused by: com.sun.mail.smtp.SMTPAddressFailedException: 550 5.7.1 <[REDACTED_EMAIL_ADDRESS]>... Relaying denied
 END
 
   NODE_JS_EXC = <<END.freeze
@@ -214,6 +242,26 @@ created by main.main
 	server.go:20 +0x91
 END
 
+  GO_HTTP = <<END.freeze
+2019/01/15 07:48:05 http: panic serving [::1]:54143: test panic
+goroutine 24 [running]:
+net/http.(*conn).serve.func1(0xc00007eaa0)
+	/usr/local/go/src/net/http/server.go:1746 +0xd0
+panic(0x12472a0, 0x12ece10)
+	/usr/local/go/src/runtime/panic.go:513 +0x1b9
+main.doPanic(0x12f0ea0, 0xc00010e1c0, 0xc000104400)
+	/Users/ingvar/src/go/src/httppanic.go:8 +0x39
+net/http.HandlerFunc.ServeHTTP(0x12be2e8, 0x12f0ea0, 0xc00010e1c0, 0xc000104400)
+	/usr/local/go/src/net/http/server.go:1964 +0x44
+net/http.(*ServeMux).ServeHTTP(0x14a17a0, 0x12f0ea0, 0xc00010e1c0, 0xc000104400)
+	/usr/local/go/src/net/http/server.go:2361 +0x127
+net/http.serverHandler.ServeHTTP(0xc000085040, 0x12f0ea0, 0xc00010e1c0, 0xc000104400)
+	/usr/local/go/src/net/http/server.go:2741 +0xab
+net/http.(*conn).serve(0xc00007eaa0, 0x12f10a0, 0xc00008a780)
+	/usr/local/go/src/net/http/server.go:1847 +0x646
+created by net/http.(*Server).Serve
+	/usr/local/go/src/net/http/server.go:2851 +0x2f5
+END
   CSHARP_EXC = <<END.freeze
 System.Collections.Generic.KeyNotFoundException: The given key was not present in the dictionary.
   at System.Collections.Generic.Dictionary`2[System.String,System.Collections.Generic.Dictionary`2[System.Int32,System.Double]].get_Item (System.String key) [0x00000] in <filename unknown>:0
@@ -228,6 +276,33 @@ System.Collections.Generic.KeyNotFoundException: The given key was not present i
   at System.Threading.Thread.StartInternal () [0x00000] in <filename unknown>:0
 END
 
+  CSHARP_NESTED_EXC = <<END.freeze
+System.InvalidOperationException: This is the outer exception ---> System.InvalidOperationException: This is the inner exception
+  at ExampleApp.NestedExceptionExample.LowestLevelMethod() in c:/ExampleApp/ExampleApp/NestedExceptionExample.cs:line 33
+  at ExampleApp.NestedExceptionExample.ThirdLevelMethod() in c:/ExampleApp/ExampleApp/NestedExceptionExample.cs:line 28
+  at ExampleApp.NestedExceptionExample.SecondLevelMethod() in c:/ExampleApp/ExampleApp/NestedExceptionExample.cs:line 18
+  --- End of inner exception stack trace ---
+  at ExampleApp.NestedExceptionExample.SecondLevelMethod() in c:/ExampleApp/ExampleApp/NestedExceptionExample.cs:line 22
+  at ExampleApp.NestedExceptionExample.TopLevelMethod() in c:/ExampleApp/ExampleApp/NestedExceptionExample.cs:line 11
+  at ExampleApp.Program.Main(String[] args) in c:/ExampleApp/ExampleApp/Program.cs:line 11
+END
+
+  CSHARP_ASYNC_EXC = <<END.freeze
+System.InvalidOperationException: This is an exception
+   at ExampleApp2.AsyncExceptionExample.LowestLevelMethod() in c:/ExampleApp/ExampleApp/AsyncExceptionExample.cs:line 36
+   at ExampleApp2.AsyncExceptionExample.<ThirdLevelMethod>d__2.MoveNext() in c:/ExampleApp/ExampleApp/AsyncExceptionExample.cs:line 31
+--- End of stack trace from previous location where exception was thrown ---
+   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
+   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
+   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
+   at ExampleApp2.AsyncExceptionExample.<SecondLevelMethod>d__1.MoveNext() in c:/ExampleApp/ExampleApp/AsyncExceptionExample.cs:line 25
+--- End of stack trace from previous location where exception was thrown ---
+   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
+   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
+   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
+   at ExampleApp2.AsyncExceptionExample.<TopLevelMethod>d__0.MoveNext() in c:/ExampleApp/ExampleApp/AsyncExceptionExample.cs:line 14
+END
+
   RUBY_EXC = <<END.freeze
  NoMethodError (undefined method `resursivewordload' for #<BooksController:0x007f8dd9a0c738>):
   app/controllers/books_controller.rb:69:in `recursivewordload'
@@ -531,6 +606,7 @@ END
   def test_java
     check_exception(JAVA_EXC, false)
     check_exception(COMPLEX_JAVA_EXC, false)
+    check_exception(NESTED_JAVA_EXC, false)
   end
 
   def test_js
@@ -541,6 +617,8 @@ END
 
   def test_csharp
     check_exception(CSHARP_EXC, false)
+    check_exception(CSHARP_NESTED_EXC, false)
+    check_exception(CSHARP_ASYNC_EXC, false)
   end
 
   def test_python
@@ -556,6 +634,7 @@ END
     check_exception(GO_EXC, false)
     check_exception(GO_ON_GAE_EXC, false)
     check_exception(GO_SIGNAL_EXC, false)
+    check_exception(GO_HTTP, false)
   end
 
   def test_ruby
@@ -598,6 +677,8 @@ END
     check_exception(GO_ON_GAE_EXC, false)
     check_exception(GO_SIGNAL_EXC, false)
     check_exception(CSHARP_EXC, false)
+    check_exception(CSHARP_NESTED_EXC, false)
+    check_exception(CSHARP_ASYNC_EXC, false)
     check_exception(V8_JS_EXC, false)
     check_exception(RUBY_EXC, false)
     check_exception(DART_ERR, false)

data/test/plugin/test_out_detect_exceptions.rb

@@ -28,6 +28,8 @@ END
 
   DEFAULT_TAG = 'prefix.test.tag'.freeze
 
+  DEFAULT_TAG_STRIPPED = 'test.tag'.freeze
+
   ARBITRARY_TEXT = 'This line is not an exception.'.freeze
 
   JAVA_EXC = <<END.freeze
@@ -74,6 +76,17 @@ END
     log_entry
   end
 
+  def feed_lines_without_line_breaks(driver, t, *messages, stream: nil)
+    count = 0
+    messages.each do |m|
+      m.each_line do |line|
+        line.delete!("\n")
+        driver.emit(log_entry(line, count, stream), t + count)
+        count += 1
+      end
+    end
+  end
+
   def feed_lines(driver, t, *messages, stream: nil)
     count = 0
     messages.each do |m|
@@ -125,7 +138,9 @@ END
     }
 
     test_cases.each do |language, exception|
-      cfg = "languages #{language}"
+      cfg = %(
+#{CONFIG}
+languages #{language})
       d = create_driver(cfg)
       t = Time.now.to_i
 
@@ -156,12 +171,12 @@ END
 
       # Validate that each line received is emitted separately as expected.
       router_mock.should_receive(:emit)
-                 .once.with(DEFAULT_TAG, Integer,
+                 .once.with(DEFAULT_TAG_STRIPPED, Integer,
                             'message' => json_line_with_exception,
                             'count' => 0)
 
       router_mock.should_receive(:emit)
-                 .once.with(DEFAULT_TAG, Integer,
+                 .once.with(DEFAULT_TAG_STRIPPED, Integer,
                             'message' => json_line_without_exception,
                             'count' => 1)
 
@@ -174,7 +189,9 @@ END
   end
 
   def test_single_language_config
-    cfg = 'languages java'
+    cfg = %(
+#{CONFIG}
+languages java)
     d = create_driver(cfg)
     t = Time.now.to_i
     d.run do
@@ -185,7 +202,9 @@ END
   end
 
   def test_multi_language_config
-    cfg = 'languages python, java'
+    cfg = %(
+#{CONFIG}
+languages python, java)
     d = create_driver(cfg)
     t = Time.now.to_i
     d.run do
@@ -196,7 +215,9 @@ END
   end
 
   def test_split_exception_after_timeout
-    cfg = 'multiline_flush_interval 1'
+    cfg = %(
+#{CONFIG}
+multiline_flush_interval 1)
     d = create_driver(cfg)
     t1 = 0
     t2 = 0
@@ -227,6 +248,12 @@ END
     assert_equal(make_logs(t1, JAVA_EXC + "  at x\n  at y\n"), d.events)
   end
 
+  def test_remove_tag_prefix_is_required
+    cfg = ''
+    e = assert_raises(Fluent::ConfigError) { create_driver(cfg) }
+    assert_match(/remove_tag_prefix/, e.message)
+  end
+
   def get_out_tags(remove_tag_prefix, original_tag)
     cfg = "remove_tag_prefix #{remove_tag_prefix}"
     d = create_driver(cfg, original_tag)
@@ -243,8 +270,42 @@ END
     assert_equal(['prefix.plus.rest.of.the.tag'], tags)
   end
 
+  def test_force_line_breaks_false
+    cfg = %(
+#{CONFIG}
+force_line_breaks true)
+    d = create_driver(cfg)
+    t = Time.now.to_i
+    d.run do
+      feed_lines(d, t, JAVA_EXC)
+    end
+    expected = JAVA_EXC
+    assert_equal(make_logs(t, *expected), d.events)
+  end
+
+  def test_force_line_breaks_true
+    cfg = %(
+#{CONFIG}
+force_line_breaks true)
+    d = create_driver(cfg)
+    t = Time.now.to_i
+    d.run do
+      feed_lines_without_line_breaks(d, t, JAVA_EXC)
+    end
+    # Expected: the first two lines of the exception are buffered and combined.
+    # Then the max_lines setting kicks in and the rest of the Python exception
+    # is logged line-by-line (since it's not an exception stack in itself).
+    # For the following Java stack trace, the two lines of the first exception
+    # are buffered and combined. So are the first two lines of the second
+    # exception. Then the rest is logged line-by-line.
+    expected = JAVA_EXC.chomp
+    assert_equal(make_logs(t, *expected), d.events)
+  end
+
   def test_flush_after_max_lines
-    cfg = 'max_lines 2'
+    cfg = %(
+#{CONFIG}
+max_lines 2)
     d = create_driver(cfg)
     t = Time.now.to_i
     d.run do
@@ -263,7 +324,9 @@ END
   end
 
   def test_separate_streams
-    cfg = 'stream stream'
+    cfg = %(
+#{CONFIG}
+stream stream)
     d = create_driver(cfg)
     t = Time.now.to_i
     d.run do

metadata

@@ -1,15 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fluent-plugin-detect-exceptions
 version: !ruby/object:Gem::Version
-  version: 0.0.10
+  version: 0.0.14
 platform: ruby
 authors:
-- Thomas Schickinger
-- Igor Peshansky
+- Stackdriver Agents
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-04-12 00:00:00.000000000 Z
+date: 2021-09-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fluentd
@@ -87,15 +86,13 @@ description: |2
      same stack trace into one multi-line message.
      This is an official Google Ruby gem.
 email:
-- schickin@google.com
-- igorp@google.com
+- stackdriver-agents@google.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
 - CONTRIBUTING
 - Gemfile
-- Gemfile.lock
 - LICENSE
 - README.rdoc
 - Rakefile
@@ -125,8 +122,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.14
+rubygems_version: 3.0.8
 signing_key: 
 specification_version: 4
 summary: fluentd output plugin for combining stack traces as multi-line JSON logs

data/Gemfile.lock

@@ -1,66 +0,0 @@
-PATH
-  remote: .
-  specs:
-    fluent-plugin-detect-exceptions (0.0.10)
-      fluentd (>= 0.10)
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    ast (2.4.0)
-    cool.io (1.5.3)
-    dig_rb (1.0.1)
-    flexmock (2.3.6)
-    fluentd (1.1.0)
-      cool.io (>= 1.4.5, < 2.0.0)
-      dig_rb (~> 1.0.0)
-      http_parser.rb (>= 0.5.1, < 0.7.0)
-      msgpack (>= 0.7.0, < 2.0.0)
-      serverengine (>= 2.0.4, < 3.0.0)
-      sigdump (~> 0.2.2)
-      strptime (>= 0.2.2, < 1.0.0)
-      tzinfo (~> 1.0)
-      tzinfo-data (~> 1.0)
-      yajl-ruby (~> 1.0)
-    http_parser.rb (0.6.0)
-    msgpack (1.2.4)
-    parser (2.5.0.3)
-      ast (~> 2.4.0)
-    power_assert (1.1.1)
-    powerpack (0.1.1)
-    rainbow (2.2.2)
-      rake
-    rake (10.5.0)
-    rubocop (0.42.0)
-      parser (>= 2.3.1.1, < 3.0)
-      powerpack (~> 0.1)
-      rainbow (>= 1.99.1, < 3.0)
-      ruby-progressbar (~> 1.7)
-      unicode-display_width (~> 1.0, >= 1.0.1)
-    ruby-progressbar (1.9.0)
-    serverengine (2.0.6)
-      sigdump (~> 0.2.2)
-    sigdump (0.2.4)
-    strptime (0.2.3)
-    test-unit (3.2.7)
-      power_assert
-    thread_safe (0.3.6)
-    tzinfo (1.2.5)
-      thread_safe (~> 0.1)
-    tzinfo-data (1.2018.3)
-      tzinfo (>= 1.0.0)
-    unicode-display_width (1.3.0)
-    yajl-ruby (1.3.1)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  flexmock (~> 2.0)
-  fluent-plugin-detect-exceptions!
-  rake (~> 10.3)
-  rubocop (= 0.42.0)
-  test-unit (~> 3.0)
-
-BUNDLED WITH
-   1.16.1