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

InfluxDB TLS #6405

Closed
devcrust opened this issue Mar 13, 2019 · 2 comments
Closed

InfluxDB TLS #6405

devcrust opened this issue Mar 13, 2019 · 2 comments
Assignees
Milestone

Comments

@devcrust
Copy link

Describe the bug
First of all, thanks a lot for integrating InfluxDB as a database plugin!

When using TLS configuration either via "pem_bundle" or "pem_json" there is no way how to provide a new certificate or private key once the old one has expired.

To Reproduce
Steps to reproduce the behavior:

  1. Configure the InfluxDB secret engine using TLS via either "pem_bundle" or "pem_json"
  2. Rotate the database root credentials
  3. Try to reconfigure the secret engine using the same initial username and password but a new certificate and private key using either "pem_bundle" or "pem_json"

Expected behavior
Either provide a similar behaviour like the PostgreSQL plugin (using physical ca, cert and key files), or provide a dedicated API endpoint to update the TLS settings once required.

Environment:

  • Vault Server Version (retrieve with vault status): 1.0.3
  • Vault CLI Version (retrieve with vault version): 1.0.3
  • Server Operating System/Architecture: Docker (official Vault image)
@jefferai
Copy link
Member

Can you provide more info on the exact calls you're making? The code doesn't treat these values differently for updating purposes so it's not clear what might be going on.

@briankassouf briankassouf added this to the 1.1.1 milestone Mar 15, 2019
@jefferai jefferai modified the milestones: 1.1.1, 1.1.2 Apr 10, 2019
@briankassouf briankassouf modified the milestones: 1.1.2, 1.1.3 Apr 29, 2019
@briankassouf briankassouf modified the milestones: 1.1.3, 1.2 May 21, 2019
@briankassouf briankassouf modified the milestones: 1.2, 1.3 Jul 8, 2019
@mgaffney
Copy link
Member

Hi @devcrust, thanks for reporting this!

I investigated this issue and found that InfluxDB does not actually support client certificates but Vault's influxdb-database-plugin can be configured to use a client certificate.

I opened #7118 to remove support for configuring client certificates in influxdb-database-plugin which I think is the source of your issue.

Can you try to reproduce your problem without using a client certificate?

@mgaffney mgaffney self-assigned this Jul 15, 2019
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