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

Telegraf to accept tokens on v1 InfluxDB input plugins #9445

Closed
sjwang90 opened this issue Jun 28, 2021 · 1 comment · Fixed by #13610
Closed

Telegraf to accept tokens on v1 InfluxDB input plugins #9445

sjwang90 opened this issue Jun 28, 2021 · 1 comment · Fixed by #13610
Assignees
Labels
area/influxdb feature request Requests for new plugin and for new features to existing plugins

Comments

@sjwang90
Copy link
Contributor

Feature Request

Opening a feature request kicks off a discussion.

Proposal:

Add tokens (alongside existing username/password) to the InfluxDB Listener v1 plugin.

  • Use telegraf as an aggregator in front of the Cloud 2 API with v1 capability.
  • Even though v2 listener https://docs.influxdata.com/telegraf/v1.18/plugins/#influxdb_v2_listener exists,
    I was hoping the inputs.influxdb_listener would provide similarly a “fake” v2 with token = user:password (just like InfluxDB 1.8) on the same port as the regular v1 API (the v2 listener I think would require a different port).

Current behavior:

Telegraf InfluxDB Listener v1 input plugin only supports username/password not tokens.

Desired behavior:

Allow for token configuration in inputs.influxdb_listener.
The token would just be what’s listed in compatibility API: https://docs.influxdata.com/influxdb/v2.0/reference/api/influxdb-1x/#basic-authentication

Use case:

Make migration experience from 1.x to 2.x more seamless.

For example, while using the Go library (https://github.com/influxdata/influxdb-client-go#influxdb-18-api-compatibility)
and noticed it didn't work with Telegraf.

@sjwang90 sjwang90 added feature request Requests for new plugin and for new features to existing plugins area/influxdb labels Jun 28, 2021
@sjwang90
Copy link
Contributor Author

sjwang90 commented Aug 2, 2021

We will implement this feature work as part of the initiative in #9478 issue to combine v1/v2 plugins and make sure any influxdb has compatibility with both versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/influxdb feature request Requests for new plugin and for new features to existing plugins
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants