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

add option to enable TLS via environment variable #9

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

Conversation

victoru
Copy link

@victoru victoru commented Oct 12, 2017

No description provided.

@edsiper
Copy link
Member

edsiper commented Nov 2, 2017

@victoru thanks for this contribution.

I am wondering if enabling TLS is enough for certification validation, is Fluent Bit able to validate Elasticsearch certificate properly by default ?

@victoru
Copy link
Author

victoru commented Nov 2, 2017

By default, I'd say yes. Fluent bit was able to validate the certificate when I used it with an AWS Elasticsearch cluster.

@edsiper
Copy link
Member

edsiper commented Feb 6, 2018

please rebase on top of master

@sbrandtb
Copy link

sbrandtb commented May 3, 2018

@edsiper Merge this? I've also implemented this in my local copy the same way

@mwl
Copy link

mwl commented Jun 7, 2018

This seems to have stalled a bit. Is there anything you need help with? (We kind of need this)

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.

4 participants