logstash-filter-ros2_node_info 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 91999825fb3abdd377eb0ecb66ad3ea4f6544f67947f10c34757340d6e0882ed
-  data.tar.gz: 78ce9ff789f6067e9d9055758fa613931d0a426fd7245d117b39fbfb3b97954e
+  metadata.gz: 403e766ade1210a2e82807b13d3271f0804597124afd990d1e525c65b3b3e86c
+  data.tar.gz: 54ac0dce21b3276cf0ce5cbba3f341c7bde3a575a970d82cec5356ec2fc69c85
 SHA512:
-  metadata.gz: 5a23ccdd3420ebc3bde34cf749b442bcde11efeac8770826ae560b9cf39eed511feec7c682a983b915b10c8b30a384fec977a683db1629a0f42a9a9c139c5c95
-  data.tar.gz: 74d2fcb2255a0ecd243c8391f3a56808ea237594e452a12e2263626eeb2e479a5bfbb2ca6be922037f96c6e38ed471388c839e692844e4bfe1a2c04bd998fa3d
+  metadata.gz: b9b9582e7408ea2e41d708b78ea85e8f5d2a9a3204c6e9859d6aed720dc7d48c318d5e6d1d0364498db8961d951161973e0df017b018074fea75ecdefee1449e
+  data.tar.gz: 0b5d5358d826b7143124ccdba16cf78760212160dddcd799d1549f2c4ca4b5c3babf09527ad00fdf33787643755a2abbcd074a1b97bc4169f4cbb6b15a735e58

data/lib/logstash/filters/ros2_node_info.rb

@@ -22,6 +22,11 @@ class LogStash::Filters::Ros2NodeInfo < LogStash::Filters::Base
     # Essentially a simple stack-based parser based on number of spaces at
     # the start of each line
     if @source
+      if event.get(@source).nil?
+        event.tag("_ros2_node_info_parse_failure")
+        @logger.debug? && @logger.debug("Event had no data in #{@source}")
+        return [event]
+      end
       last_cat = nil
       running_array = []
       event.get(@source).split("\n").each do |line|
@@ -41,8 +46,6 @@ class LogStash::Filters::Ros2NodeInfo < LogStash::Filters::Base
           # Element of a category - split it appropriately and append it to the
           # working array
           split_line = line.tr(" ", "").split(":")
-	  @logger.error(line)
-	  @logger.error(split_line.to_yaml)
           running_array << { "topic" => split_line[0], "type" => split_line[1] }
 	end
       end

data/logstash-filter_ros2_node_info.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name          = 'logstash-filter-ros2_node_info'
-  s.version       = '0.1.1'
+  s.version       = '0.1.2'
   s.licenses      = ['Apache-2.0']
   s.summary       = 'A plugin to parse the output of `ros2 node info` into a useful format.'
   s.homepage      = 'https://github.com/soartech/logstash-filter-ros2_node_info'

data/spec/filters/ros2_node_info_spec.rb

@@ -29,4 +29,25 @@ describe LogStash::Filters::Ros2NodeInfo do
                                                  "type"=>"rcl_interfaces/ParameterEvent"}])
     end
   end
+
+  describe "Parse nonexistent field" do
+    let(:config) do <<-CONFIG
+      filter {
+        ros2_node_info {
+          source => "wrongsource"
+        }
+      }
+    CONFIG
+    end
+
+    sample("message" => "/talker
+  Subscribers:
+    /parameter_events: rcl_interfaces/ParameterEvent
+  Publishers:
+    /chatter: std_msgs/String
+  Services:
+    /talker/set_parameters_atomically: rcl_interfaces/SetParametersAtomically") do
+      expect(subject.get("tags")).to include("_ros2_node_info_parse_failure")
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-ros2_node_info
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Chris Reffett