search

sensu-plugins / sensu-plugins-haproxy

This plugin provides native HAProxy instrumentation for monitoring and metrics collection, including: service health and backend server metrics.
http://sensu-plugins.io
MIT License
11 stars 37 forks source link

Check script needs ssl flag #12

Closed foozmeat closed 7 years ago

foozmeat commented 8 years ago

pull request incoming

Evesy commented 7 years ago

Hey @foozmeat

Any chance of opening a PR against this repository for this change? I see the existing PR is just against the master branch of your forked repository & it'd be nice to get this change implemented.

Unless I'm mistaken the SSL option should be passed in as a hash too:

res = Net::HTTP.start(config[:stats_source], config[:port], config[:use_ssl]) do |http|

Should be:

res = Net::HTTP.start(config[:stats_source], config[:port], use_ssl: config[:use_ssl]) do |http|

Thanks!

Evesy commented 7 years ago

@majormoses @foozmeat I guess this one can be closed now?

foozmeat commented 7 years ago

yes that's fine. I'll switch back to your repo.