Skip to content
This repository has been archived by the owner on Nov 8, 2023. It is now read-only.

Update for ElasticSearch authentication #407

Closed
wants to merge 1 commit into from

Conversation

maxidea-com
Copy link

@maxidea-com maxidea-com commented Nov 28, 2017

#399

es = elasticsearch.Elasticsearch(cfg.cfg["elastic"]["host"],
http_auth=(cfg.cfg["elastic"]["user"],cfg.cfg["elastic"]["pass"]),
verify_certs=False,
use_ssl=use_ssl)

@sabban sabban self-assigned this Dec 14, 2017
@sabban sabban self-requested a review December 14, 2017 09:53
@sabban
Copy link
Contributor

sabban commented Dec 14, 2017

Hi,
It's a great idea to use authentication against ES.
But I am not confident with providing security software that doesn't check ssl certificates.

Can you provide a patch that could make it optional (check the validity by default, and if the option is provided, bypass the check) ?

And if I have the time I will update the "new tool" https://github.com/nbs-system/nxtool-ng/blob/master/nxtool/log_providers/elastic.py the exact same way.

Thanks.

@wargio wargio closed this Sep 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants