sensu-plugins / sensu-plugins-http

This plugin provides native HTTP instrumentation for monitoring and metrics collection, including: response code, JSON response, HTTP last modified, SSL expiry, and metrics via `curl`.
http://sensu-plugins.io
MIT License
30 stars 96 forks source link

metrics-http-json-deep.rb: fix key normalization #66

Closed ushi-as closed 7 years ago

ushi-as commented 7 years ago

String#tr accepts two strings as arguments. Not a RegExp.

See: https://ruby-doc.org/core-2.3.3/String.html#method-i-tr

It raises: no implicit conversion of Regexp into String at the moment.

Pull Request Checklist

General

New Plugins

Purpose

Known Compatablity Issues

eheydrick commented 7 years ago

Thanks for the PR. I ended up merging #65 which does virtually the same thing. The fix is in 2.0.2 which is now live on Rubygems.