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

TLS client certificate #80

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adrianlzt
Copy link

Add options to be able to login to ES using client certificates.
Also an option to ignore checking certificate validity.

Add options to be able to login to ES using client certificates.
Also an option to ignore checking certificate validity.
@adrianlzt
Copy link
Author

Example config:

input {
  elasticsearch {
    hosts => "elastic:443"
    query => ''
    index => 'test'
    ssl => true
    ssl_verify => false
    ca_file => '/certs/admin-ca'
    client_cert_file => '/certs/admin-cert'
    client_key_file => '/certs/admin-key'
  }
}

@adrianlzt
Copy link
Author

Fixes #71

@andrewvc
Copy link
Contributor

Thanks for the patch.

The tricky thing here is adding options consistently across the ES input/output/filter

This is a bit of a weird time as we're trying to centralize into one repo here: logstash-plugins/logstash-integration-elasticsearch#1

Would you mind holding off on this patch till that happens? Otherwise we'll need three simultaneous PRs. We don't want to continue our current pattern of inconsistently adding features.

@adrianlzt
Copy link
Author

adrianlzt commented Oct 27, 2017 via email

@roniez
Copy link

roniez commented Feb 20, 2020

This function is highly sought after.

@hvdijk-p5
Copy link

Are there still plans to introduce this feature?

The feature logstash-plugins/logstash-integration-elasticsearch#1 was mentioned already more than a year ago with no recent updates. So I am curious if there are plans for this in the nearby future.

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

Successfully merging this pull request may close these issues.

5 participants