Skip to content

Commit e1db0bf

Browse files
multanimajormoses
authored andcommitted
Remove unexplicit dependency on ActiveSupport
The dependency wasn't declared in the plugin dependencies (I'm not sure how it could have worked before?)
1 parent cae3e80 commit e1db0bf

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@ This CHANGELOG follows the format listed [here](https://github.com/sensu-plugins
55

66
## [Unreleased]
77

8+
### Fixed
9+
- Remove unexplicit dependency on ActiveSupport (@multani)
10+
811
## [2.2.1] - 2018-10-16
912
### Security
1013
- updated yard dependency to `~> 0.9.11` per: https://nvd.nist.gov/vuln/detail/CVE-2017-17042 (@majormoses)

bin/metric-postgres-statsdb.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@
2929
# for details.
3030
#
3131

32-
require 'active_support/core_ext/hash'
3332
require 'sensu-plugins-postgres/pgpass'
3433
require 'sensu-plugin/metric/cli'
3534
require 'pg'
@@ -95,7 +94,8 @@ def run
9594
]
9695
con.exec(request.join(' ')) do |result|
9796
result.each do |row|
98-
row.except('datid', 'stats_reset').each do |key, value|
97+
row.each do |key, value|
98+
next if %w[datid stats_reset].include?(key)
9999
output "#{config[:scheme]}.statsdb.#{config[:database]}.#{key}", value.to_s, timestamp
100100
end
101101
end

0 commit comments

Comments
 (0)