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.

Sign up for GitHub

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

Jump to bottom

Switch influxdb client dependency to use github.com/influxdata/influxdb1-client #6088

Closed
mark-rushakoff opened this issue Jan 23, 2019 · 0 comments · Fixed by #12262
Closed

Switch influxdb client dependency to use github.com/influxdata/influxdb1-client #6088

mark-rushakoff opened this issue Jan 23, 2019 · 0 comments · Fixed by #12262
Labels
ecosystem enhancement secret/database

Comments

@mark-rushakoff
Copy link

@jefferai I saw you mention a dependency on github.com/influxdata/platform in golang/go#29798, so I ran go mod why, from the gomodtry2 branch, to see the source of the dependency:

# github.com/influxdata/platform
github.com/hashicorp/vault/plugins/database/influxdb
github.com/influxdata/influxdb/client/v2
github.com/influxdata/influxdb/models
github.com/influxdata/platform/models

We have recently split the client into its own self-contained package, https://github.com/influxdata/influxdb1-client, so you should depend on that instead to reduce the dependency bloat.
@bcmills bcmills mentioned this issue Jan 25, 2019
Merged
tsaarni added a commit to Nordix/vault that referenced this issue Aug 4, 2021
@tsaarni
Update influxdb client
62f9008 
* Influxdb v1 client has been split into separate module from main influxdb
  code base. This changes uses the correct client, which also allows us to
  get updates and avoids confusing some vuln scanners that flagged previous
  version incorrectly with vulnerabilities. Fixes hashicorp#6088.
@tsaarni tsaarni mentioned this issue Aug 4, 2021
Merged
tsaarni added a commit to Nordix/vault that referenced this issue Aug 17, 2021
@tsaarni
Update influxdb client
6629a68 
* Influxdb v1 client has been split into separate module from main influxdb
  code base. This changes uses the correct client, which also allows us to
  get updates and avoids confusing some vuln scanners that flagged previous
  version incorrectly with vulnerabilities. Fixes hashicorp#6088.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ecosystem enhancement secret/database
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update influxdb client Nordix/vault
3 participants
@mark-rushakoff @michelvocks @pmmukh