sensu-plugins-sensu 3.0.0 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 74be22e9979ca4bce08680ae287bbdc861afff3afc3783fccd98a1d0a9fbd528
-  data.tar.gz: bd7f1ed317662a1892ed771d7d74b3e8e6a7f8936fa0efd93942631b62b34821
+  metadata.gz: 6133a1d932a4e24c9866faaf379348fcc320fdb6d00afbefd7cd934dcaf95f01
+  data.tar.gz: 6feac0be3ec402a6374a97823dd095421cd5aa34666a444da2dc68c8261169e2
 SHA512:
-  metadata.gz: bc717f238a82fcd66675ff1b5b4c441c2f065f2d90f92c89d784f84262d4dbdcebb0c74f91f7be9700b129f98fdc9c765c319d93fe13f83d02c57242c2836fbc
-  data.tar.gz: c2c3a8b78237a623c30484fbdb8e193abced393ae1cbf4e9055cfa0d358c9435d9faddeee4afa58a5896a55262b501f5cf7774c794d392593251409d56061c8a
+  metadata.gz: 8097b173310aa6a407c16a8c89505c2a794fabf3fab36e25b09c77c93d3af34046a986e0001061d8d27efccdc24328ffcb246075396ccd89a5b66c5e14824a55
+  data.tar.gz: 568f7b0e700cb97df368f6343aa57b79c9a34726ba077397bce5f713cd38a35752aa5f2c911aed61ec4895fff1d99147aafb63780eb3e66e0e63351357c4d041

data/CHANGELOG.md

@@ -6,6 +6,27 @@ Which is based on [Keep A Changelog](http://keepachangelog.com/)
 
 ## [Unreleased]
 
+## [4.0.0] - 2018-07-18
+### Security
+- updated `yard` dependency to `~> 0.9.11` per: https://nvd.nist.gov/vuln/detail/CVE-2017-17042 which closes attacks against a yard server loading arbitrary files (@majormoses)
+- updated rubocop dependency to `~> 0.51.0` per: https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2017-8418. (@majormoses)
+
+### Breaking Changes
+- removing ruby support for `< 2.3` versions as they are EOL (@majormoses)
+- metrics-aggregate.rb: removed support for `sensu` api versions lower than `0.24` you can read about it [here](https://github.com/sensu/sensu/issues/1218) (@majormoses)
+
+### Removed
+- gemnasium badge as github offers native feature and they were bought by gitlab and no longer available as a standalone product (@majormoses)
+
+### Added
+- slack badge (@majormoses)
+
+### Changed
+- check-stale-results.rb: improve error message when there is no api key in sensu settings (@majormoses)
+- bumped dependency of `sensu-plugin` to `~> 2.5`  (@majormoses)
+- appeasing the cops (@majormoses)
+
+## [3.0.0] - 2018-05-17
 ### Breaking Change
 - bumped dependency of `sensu-plugin` to 2.x you can read about it [here](https://github.com/sensu-plugins/sensu-plugin/blob/master/CHANGELOG.md#v200---2017-03-29)
 
@@ -122,7 +143,8 @@ Which is based on [Keep A Changelog](http://keepachangelog.com/)
 ### Added
 - initial release
 
-[Unreleased]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/3.0.0...HEAD
+[Unreleased]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/4.0.0...HEAD
+[4.0.0]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/3.0.0...4.0.0
 [3.0.0]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/2.5.0...3.0.0
 [2.5.0]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/2.4.1...2.5.0
 [2.4.1]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/2.4.0...2.4.1

data/README.md

@@ -4,7 +4,7 @@
 [![Gem Version](https://badge.fury.io/rb/sensu-plugins-sensu.svg)](http://badge.fury.io/rb/sensu-plugins-sensu)
 [![Code Climate](https://codeclimate.com/github/sensu-plugins/sensu-plugins-sensu/badges/gpa.svg)](https://codeclimate.com/github/sensu-plugins/sensu-plugins-sensu)
 [![Test Coverage](https://codeclimate.com/github/sensu-plugins/sensu-plugins-sensu/badges/coverage.svg)](https://codeclimate.com/github/sensu-plugins/sensu-plugins-sensu)
-[![Dependency Status](https://gemnasium.com/sensu-plugins/sensu-plugins-sensu.svg)](https://gemnasium.com/sensu-plugins/sensu-plugins-sensu)
+[![Community Slack](https://slack.sensu.io/badge.svg)](https://slack.sensu.io/badge)
 
 ## Functionality
 

data/bin/check-aggregate.rb

@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: false
+
 #
 # Check Aggregate
 # ===
@@ -171,7 +173,7 @@ class CheckAggregate < Sensu::Plugin::Check::CLI
     aggregate[:results].delete_if do |entry|
       begin
         api_request("/stashes/silence/#{entry[:client]}/#{config[:check]}")
-        if entry[:status] == 0
+        if entry[:status].zero?
           aggregate[:ok] = aggregate[:ok] - 1
         elsif entry[:status] == 1
           aggregate[:warning] = aggregate[:warning] - 1
@@ -192,7 +194,7 @@ class CheckAggregate < Sensu::Plugin::Check::CLI
   def collect_output(aggregate)
     output = ''
     aggregate[:results].each do |entry|
-      output << entry[:output] + "\n" unless entry[:status] == 0
+      output << entry[:output] + "\n" unless entry[:status].zero?
     end
     aggregate[:outputs] = [output]
   end
@@ -208,7 +210,7 @@ class CheckAggregate < Sensu::Plugin::Check::CLI
 
   def named_aggregate_results
     results = api_request("/aggregates/#{config[:check]}?max_age=#{config[:age]}")[:results]
-    warning "No aggregates found in last #{config[:age]} seconds" if %w(ok warning critical unknown).all? { |x| results[x.to_sym] == 0 }
+    warning "No aggregates found in last #{config[:age]} seconds" if %w[ok warning critical unknown].all? { |x| results[x.to_sym].zero? }
     results
   end
 
@@ -263,7 +265,7 @@ class CheckAggregate < Sensu::Plugin::Check::CLI
     if config[:debug]
       message += "\n" + aggregate.to_s
     end
-    aggregate[:outputs].each do |output, _count|
+    aggregate[:outputs].each_key do |output|
       matched = regex.match(output.to_s)
       unless matched.nil?
         key = matched[1]

data/bin/check-stale-results.rb

@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: false
+
 #
 # check-stale-results.rb
 #
@@ -54,7 +56,7 @@ class CheckStaleResults < Sensu::Plugin::Check::CLI
 
   def humanize(secs)
     [[60, :seconds], [60, :minutes], [24, :hours], [1000, :days]].map do |count, name|
-      if secs > 0
+      if secs.positive?
         secs, n = secs.divmod(count)
         "#{n.to_i} #{name}"
       end
@@ -69,7 +71,11 @@ class CheckStaleResults < Sensu::Plugin::Check::CLI
 
   def api_request(method, path)
     unless settings.key?('api')
-      raise 'api.json settings not found.'
+      unknown <<~HEREDOC
+        sensu does not have an api config stanza set, please configure it in
+        either /etc/sensu/config.json or in any config that is loaded by sensu
+        such as /etc/sensu/conf.d/api.json
+      HEREDOC
     end
     uri = get_uri(path)
     Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https') do |http|
@@ -85,7 +91,7 @@ class CheckStaleResults < Sensu::Plugin::Check::CLI
   def results
     res = []
     req = api_request(:GET, '/results')
-    res = JSON.parse(req.body) if req && req.code == '200'
+    res = JSON.parse(req.body) if req&.code == '200'
     res
   end
 

data/bin/handler-purge-stale-results.rb

@@ -1,3 +1,6 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: false
+
 #
 # handler-purge-stale-results.rb
 #
@@ -36,7 +39,7 @@ class HandlerPurgeStaleResults < Sensu::Handler
   def results
     res = []
     req = api_request(:GET, '/results')
-    res = JSON.parse(req.body) if req && req.code == '200'
+    res = JSON.parse(req.body) if req&.code == '200'
     res
   end
 
@@ -54,33 +57,33 @@ class HandlerPurgeStaleResults < Sensu::Handler
           else
             deleted << "#{result['client']} - #{result['check']['name']}"
           end
-        rescue
+        rescue StandardError
           failed << "#{result['client']} - #{result['check']['name']} (Caught exception: #{$ERROR_INFO})"
         end
       end
     end
 
     if !deleted.empty? || !failed.empty?
-      msg = <<EOF
-From: Sensu <#{config[:mail_sender]}>
-To: <#{config[:mail_recipient]}>
-Subject: Purge stale check results
+      msg = <<~MESSAGE
+        From: Sensu <#{config[:mail_sender]}>
+        To: <#{config[:mail_recipient]}>
+        Subject: Purge stale check results
 
-This is a notification concerning the #{self.class.name} sensu handler running at #{Socket.gethostname}
+        This is a notification concerning the #{self.class.name} sensu handler running at #{Socket.gethostname}
 
-* Summary
+        * Summary
 
-    Deleted: #{deleted.size}
-    Failed to delete: #{failed.size}
+            Deleted: #{deleted.size}
+            Failed to delete: #{failed.size}
 
-* Failed to delete check results:
+        * Failed to delete check results:
 
-#{failed.map { |m| "    #{m}" }.join("\n")}
+        #{failed.map { |m| "    #{m}" }.join("\n")}
 
-* Deleted check results:
+        * Deleted check results:
 
-#{deleted.map { |m| "    #{m}" }.join("\n")}
-EOF
+        #{deleted.map { |m| "    #{m}" }.join("\n")}
+      MESSAGE
 
       Net::SMTP.start(config[:mail_server]) do |smtp|
         smtp.send_message(msg, config[:mail_sender], config[:mail_recipient])

data/bin/handler-sensu-deregister.rb

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: false
 
 require 'rubygems'
 require 'sensu-handler'
@@ -21,13 +22,13 @@ class Deregister < Sensu::Handler
   end
 
   def delete_sensu_client!
-    if config[:invalidate] && config[:invalidate_expire]
-      response = api_request(:DELETE, '/clients/' + @event['client']['name'] + "?invalidate=#{config[:invalidate]}&#{config[:invalidate_expire]}").code
-    elsif config[:invalidate]
-      response = api_request(:DELETE, '/clients/' + @event['client']['name'] + "?invalidate=#{config[:invalidate]}").code
-    else
-      response = api_request(:DELETE, '/clients/' + @event['client']['name']).code
-    end
+    response = if config[:invalidate] && config[:invalidate_expire]
+                 api_request(:DELETE, '/clients/' + @event['client']['name'] + "?invalidate=#{config[:invalidate]}&#{config[:invalidate_expire]}").code
+               elsif config[:invalidate]
+                 api_request(:DELETE, '/clients/' + @event['client']['name'] + "?invalidate=#{config[:invalidate]}").code
+               else
+                 api_request(:DELETE, '/clients/' + @event['client']['name']).code
+               end
     deletion_status(response)
   end
 

data/bin/handler-sensu.rb

@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: false
+
 #
 ##################
 # Sensu Remediator

data/bin/metrics-aggregate.rb

@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: false
+
 #
 # Aggregate Metrics
 #
@@ -115,27 +117,12 @@ class AggregateMetrics < Sensu::Plugin::Metric::CLI::Generic
     acquire_checks.each do |check|
       aggregate = get_aggregate(check['name'])
       puts "#{check['name']} aggregates: #{aggregate}" if config[:debug]
-      aggregate.each do |result, count|
-        # in 0.24 they changed the api results for aggregates this helps
-        # maintain backwards compatibility with 0.23 and newer versions.
-        if count.is_a?(Hash)
-          count.each do |x, y|
-            output metric_name: x,
-                   value: y,
-                   graphite_metric_path: "#{config[:scheme]}.#{check['name']}.#{x}",
-                   statsd_metric_name: "#{config[:scheme]}.#{check['name']}.#{x}",
-                   influxdb_measurement: config[:measurement],
-                   tags: {
-                     check: check['name'],
-                     host: Socket.gethostname
-                   },
-                   timestamp: timestamp
-          end
-        else
-          output metric_name: result,
-                 value: count,
-                 graphite_metric_path: "#{config[:scheme]}.#{check['name']}.#{result}",
-                 statsd_metric_name: "#{config[:scheme]}.#{check['name']}.#{result}",
+      aggregate.each_value do |count|
+        count.each do |x, y|
+          output metric_name: x,
+                 value: y,
+                 graphite_metric_path: "#{config[:scheme]}.#{check['name']}.#{x}",
+                 statsd_metric_name: "#{config[:scheme]}.#{check['name']}.#{x}",
                  influxdb_measurement: config[:measurement],
                  tags: {
                    check: check['name'],

data/bin/metrics-delete-expired-stashes.rb

@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: false
+
 #
 # Delete stashes when their 'expires' timestamp is exceeded
 # ===
@@ -14,12 +16,11 @@ require 'rest-client'
 require 'json'
 require 'socket'
 
-include Sensu::Plugin::Utils
-
 class CheckSilenced < Sensu::Plugin::Metric::CLI::Generic
+  include Sensu::Plugin::Utils
   default_host = begin
                    settings['api']['host']
-                 rescue
+                 rescue StandardError
                    'localhost'
                  end
 
@@ -81,7 +82,7 @@ class CheckSilenced < Sensu::Plugin::Metric::CLI::Generic
     all_stashes.each do |stash|
       filtered_stashes << stash if stash['path'] =~ /^#{@config[:filter]}\/.*/
     end
-    return filtered_stashes
+    filtered_stashes
   rescue Errno::ECONNREFUSED
     warning 'Connection refused'
   rescue RestClient::RequestTimeout
@@ -99,9 +100,9 @@ class CheckSilenced < Sensu::Plugin::Metric::CLI::Generic
     stashes = acquire_stashes
     now = Time.now.to_i
     @count = 0
-    if stashes.count > 0
+    if stashes.count.positive?
       stashes.each do |stash|
-        if stash['content'].key?('expires') && now - stash['content']['expires'] > 0
+        if stash['content'].key?('expires') && now - stash['content']['expires'] .positive?
           delete_stash(stash) unless config[:noop]
           @count += 1
         end

data/bin/metrics-events.rb

@@ -1,4 +1,6 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: false
+
 #
 # Event Metrics
 #

data/lib/sensu-plugins-sensu.rb

@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require 'sensu-plugins-sensu/version'

data/lib/sensu-plugins-sensu/version.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module SensuPluginsSensu
   # This defines the version of the gem
   module Version
-    MAJOR = 3
+    MAJOR = 4
     MINOR = 0
     PATCH = 0
 

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: sensu-plugins-sensu
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 4.0.0
 platform: ruby
 authors:
 - Sensu-Plugins and contributors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-03-17 00:00:00.000000000 Z
+date: 2018-07-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: sensu-plugin
+  name: chronic_duration
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: 0.10.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: 0.10.6
 - !ruby/object:Gem::Dependency
   name: rest-client
   requirement: !ruby/object:Gem::Requirement
@@ -39,19 +39,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.8.0
 - !ruby/object:Gem::Dependency
-  name: chronic_duration
+  name: sensu-plugin
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.10.6
+        version: '2.5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.10.6
+        version: '2.5'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -109,75 +109,75 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0.10'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.40.0
+        version: '10.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.40.0
+        version: '10.0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: redcarpet
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '3.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.1'
+        version: '3.2'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '3.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '3.1'
 - !ruby/object:Gem::Dependency
-  name: redcarpet
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: 0.51.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: 0.51.0
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: 0.9.11
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: 0.9.11
 description: This plugin provides monitoring and metrics for Sensu.
 email: "<sensu-users@googlegroups.com>"
 executables:
@@ -223,7 +223,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.0.0
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -231,7 +231,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 2.7.7
 signing_key: 
 specification_version: 4
 summary: Sensu plugins for sensu