sensu-plugins-sensu 2.4.0 → 2.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: c0f7c7aa6934af44de715311900e9ab3663ca102
-  data.tar.gz: 2d8836d088e11c69a0ebd9cd319f8ed3960c9a93
+SHA256:
+  metadata.gz: f7a6f59e123f564d0fa00fd5b58a3c8e3a559e33f229a7e1d67acf7385bbe0fa
+  data.tar.gz: a1ea1deef5b7202d7f14e2ed1f27184dffba0a45ddf9112dc3b13451cc7f651d
 SHA512:
-  metadata.gz: 6873fec7d2ad254a5a087f2703d22bd8c3a526acfb6f1ba475ac244b45296af5a648beb5a6ad62941b94daef82ac98301b2599aa127e7fb834e79126a7279f59
-  data.tar.gz: f9dbe703320e5b1550fed23abbb810971a09a6906dfa9a042614f7a866dfc8cc39557a9658160f2ed9fc3ca8c523186964a2a32569c7d01b84a3790b70f92354
+  metadata.gz: 2e5df05bc2315148cf07e5e561a91d0c910aaf3d764f6208e8d663aa2248b3271678866e5da478a6187d3e8857c250092bb4095f2c022bf283e389ecc5d49f19
+  data.tar.gz: abcd67c206dc5cb1dd3c7aa544a525378c240f663c55e23b5e089b937d23430facf147d5f5db8c3267a3c875509090e0f13dcb152edc155f6cc2d3a67f74f624

data/CHANGELOG.md

@@ -6,6 +6,13 @@ Which is based on [Keep A Changelog](http://keepachangelog.com/)
 
 ## [Unreleased]
 
+## [2.4.1] - 2018-01-23
+### Fixed
+- Removed brackets that were added around `subscribers` in the `trigger_remediation` method in #15. This resulted in a successful submission with an HTTP 202, however as this resulted in the subscribers key being an array of arrays which meant that the remediation never was actually scheduled. This fixes it by doing a couple of things, first we remove the extra brackets therefore solving the problem. That being said I decided to add some additional validation of the data to ensure that minimally what is being passed in is an array and the first element is a string, if that is not the case we either error out when unable to determine a fix with a helpful message or in the case of nested arrays attempt to flatten them. (@drhey) (@majormoses)
+
+### Changed
+- updated changelog guidelines location (@majormoses)
+
 ## [2.4.0] - 2017-10-12
 ### Added
 - `--debug` Option to display results hash at end of output message.
@@ -105,7 +112,8 @@ Which is based on [Keep A Changelog](http://keepachangelog.com/)
 ### Added
 - initial release
 
-[Unreleased]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/2.3.1...HEAD
+[Unreleased]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/2.4.1...HEAD
+[2.4.1]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/2.4.0...2.4.1
 [2.4.0]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/2.3.1...2.4.0
 [2.3.1]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/2.3.0...2.3.1
 [2.3.0]: https://github.com/sensu-plugins/sensu-plugins-sensu/compare/2.2.2...2.3.0

data/bin/handler-sensu.rb

@@ -132,8 +132,20 @@ class Remediator < Sensu::Handler
 
   # Issue a check via the API
   def trigger_remediation(check, subscribers)
+    if subscribers.class != Array
+      p "subscribers: #{subscribers} must be an array"
+      exit 3
+    elsif subscribers.first.class == Array
+      subscribers.flatten
+      p "subscribers: #{subscribers} must be a flat array of strings, we auto flattened: #{subscribers}"
+    end
     api_request(:POST, '/request') do |req|
-      req.body = JSON.dump('check' => check, 'subscribers' => [subscribers], 'creator' => 'sensu-plugins-sensu', 'reason' => 'Auto remediation triggered')
+      req.body = JSON.dump(
+        'check' => check,
+        'subscribers' => subscribers,
+        'creator' => 'sensu-plugins-sensu',
+        'reason' => 'Auto remediation triggered'
+      )
     end
   end
 end

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

@@ -3,7 +3,7 @@ module SensuPluginsSensu
   module Version
     MAJOR = 2
     MINOR = 4
-    PATCH = 0
+    PATCH = 1
 
     VER_STRING = [MAJOR, MINOR, PATCH].compact.join('.')
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sensu-plugins-sensu
 version: !ruby/object:Gem::Version
-  version: 2.4.0
+  version: 2.4.1
 platform: ruby
 authors:
 - Sensu-Plugins and contributors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-12 00:00:00.000000000 Z
+date: 2018-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sensu-plugin
@@ -231,7 +231,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.14
+rubygems_version: 2.7.4
 signing_key: 
 specification_version: 4
 summary: Sensu plugins for sensu