Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

insecure_skip_verify not supported for influxdb input plugin #3008

Closed
kjschnei001 opened this issue Jul 11, 2017 · 8 comments
Closed

insecure_skip_verify not supported for influxdb input plugin #3008

kjschnei001 opened this issue Jul 11, 2017 · 8 comments

Comments

@kjschnei001
Copy link

kjschnei001 commented Jul 11, 2017

Bug report

Despite being sort of documented as part of the InfluxDB input plugin, https://github.com/influxdata/telegraf/tree/master/plugins/inputs/influxdb, the insecure_skip_verify option is not actually supported.

Relevant telegraf.conf:

[[inputs.influxdb]]
insecure_skip_verify = true
urls = ["https://127.0.0.1:8084/debug/vars"]

System info:

Telegraf v1.3.3 (git: release-1.3 46db92a)
CentOS Linux release 7.2.1511 (Core)

Steps to reproduce:

  1. Configure the influxdb telegraf input plugin with insecure_skip_verify

Expected behavior:

insecure_skip_verify = true will disable ssl verification of the influxdb certificate when the plugin connects to monitor it

Actual behavior:

2017/07/11 21:09:05 E! Error parsing /etc/telegraf/telegraf.d/default_inputs.conf, line 10: field corresponding to `insecure_skip_verify' is not defined in `*influxdb.InfluxDB'
@danielnelson danielnelson modified the milestone: 1.3.4 Jul 11, 2017
@danielnelson
Copy link
Contributor

Support for this is only added in master, and will be in 1.4.0. #2773

If you are following links from https://docs.influxdata.com, note that they are linked to the master branch and are not specific to the version selected. Sorry about this confusion.

@danielnelson
Copy link
Contributor

I opened an issue about the links to the docs influxdata/docs.influxdata.com-ARCHIVE#1195

@nickidw
Copy link

nickidw commented Oct 14, 2018

Support for this is only added in master, and will be in 1.4.0. #2773

If you are following links from https://docs.influxdata.com, note that they are linked to the master branch and are not specific to the version selected. Sorry about this confusion.

I'm running 1.8.2 on Windows, and the error message indicates it's not yet supported.

C:\Program Files\Telegraf>telegraf --console --test 2018/10/14 18:33:15 I! Using config file: C:\Program Files\Telegraf\telegraf.con f 2018/10/14 18:33:15 E! Error parsing C:\Program Files\Telegraf\telegraf.conf, li ne 79: field corresponding to insecure-skip-verify' is not defined in `*influxd
b.InfluxDB'

C:\Program Files\Telegraf>`

@nickidw
Copy link

nickidw commented Oct 14, 2018

If I remove the option I get:
2018-10-14T16:37:24Z W! [outputs.influxdb] when writing to [https://35.231.195.253:8086]: database "telegraf" creation failed: Post https://35.231.195.253:8086/query: x509: certificate signed by unknown authority 2018-10-14T16:37:24Z I! Starting Telegraf 1.8.1 2018-10-14T16:37:24Z I! Loaded inputs: inputs.win_perf_counters 2018-10-14T16:37:24Z I! Loaded aggregators: 2018-10-14T16:37:24Z I! Loaded processors:

@nickidw
Copy link

nickidw commented Oct 14, 2018

The option in the config file should actually be InsecureSkipVerify = true

I changed my config to the above and now it works.

@glinton
Copy link
Contributor

glinton commented Oct 15, 2018

@nickidw did you try with insecure_skip_verify = true? Underscores (_) should have worked, hyphens (-) would not. Open a new issue if using the underscored version doesn't work for you.

@nickidw
Copy link

nickidw commented Oct 15, 2018 via email

@nickidw nickidw mentioned this issue Oct 15, 2018
3 tasks
@nickidw
Copy link

nickidw commented Oct 15, 2018

Apologies, seems I missed the trivial difference between underscore and dash. Thank you for the assistance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants