search

influxdata / influxdb-relay

Service to replicate InfluxDB data for high availability
MIT License
855 stars 350 forks source link

Support skip-tls-verification for https output #18

Closed alxdm closed 8 years ago

alxdm commented 8 years ago

Hi.

On staging we use self signed certificates for InfluxDB and we need skip TLS certificate verification. Could you check pull-request and decide apply changes or not. I understand it's insecure and need be careful with such options.

Thanks.

alxdm commented 8 years ago

@gunnaraasen I applied changes to transport constructor. Maybe should add some other options to http.Transport constructor (like in http.DefaultTransport)? What do you think? Thanks.

joelegasse commented 8 years ago

@alexdemidov This looks good, thanks!

Can you update your changes to have them occur in the newHTTPBackend and newSimplePoster functions that got merged with the buffering update?

alxdm commented 8 years ago

@joelegasse I have merged this pull-request with master branch. Could you have a look, please?

joelegasse commented 8 years ago

@alexdemidov This looks good. Thanks again.

alxdm commented 8 years ago

Thank you.