search

bernd / statsd-influxdb-backend

A naive InfluxDB backend for StatsD
ISC License
172 stars 74 forks source link

Added SSL support. #3

Closed stephenmcd closed 10 years ago

stephenmcd commented 10 years ago

This change simply adds support for a ssl config option, to allow for when InfluxDB is being hosted over SSL.

The way it's implemented is that when ssl is set to true, the https module is used for requests to InfluxDB, instead of the http module.

Thanks a lot for the library, it's working great.

bernd commented 10 years ago

Thanks for the PR! :grinning:

Do we need to add some options to further configure the https client? (like verfiy true/false, certificate paths, etc) I could imagine that people might run into problems without those options.

stephenmcd commented 10 years ago

Do we need to add some options to further configure the https client?

No idea really, I'm not familiar with the ecosystem.

bernd commented 10 years ago

Okay, I guess people will speak up if there are more options needed. :grinning:

Thank you!

aspring commented 9 years ago

Would it be possible to get a version cut with this PR?

bernd commented 9 years ago

@aspring I just published v0.4.0 to npmjs. Have fun! :smiley: