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

Cherry-pick #12355 to 7.2: Enforce presence of Certificate Authorities, Certificate file and Key when using LoadTLSServerConfig #12365

Merged
merged 1 commit into from
May 30, 2019

Conversation

ph
Copy link
Contributor

@ph ph commented May 30, 2019

Cherry-pick of PR #12355 to 7.2 branch. Original message:

When TLS is enabled for a Server we enforce CA, Certificate and the private key to make sure that the communication is correctly encrypted.

… when using LoadTLSServerConfig (elastic#12355)

* Enforce presence of Certificate Authorities, Certificate file and Key
when using LoadTLSServerConfig


(cherry picked from commit 8589309)
@ph ph requested a review from a team as a code owner May 30, 2019 01:33
@ph ph merged commit 0a4fa5f into elastic:7.2 May 30, 2019
ph added a commit to ph/beats that referenced this pull request Jun 4, 2019
…cate Authorities, Certificate file and Key when using LoadTLSServerConfig (elastic#12365)"

This reverts commit 0a4fa5f.
ph added a commit that referenced this pull request Jun 4, 2019
…thorities, Certificate file and Key when using LoadTLSServerConfig (#12365)" (#12420)

This reverts commit 0a4fa5f.
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…horities, Certificate file and Key when using LoadTLSServerConfig (elastic#12365)

* Enforce presence of Certificate Authorities, Certificate file and Key
when using LoadTLSServerConfig


(cherry picked from commit fdd0b75)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…cate Authorities, Certificate file and Key when using LoadTLSServerConfig (elastic#12365)" (elastic#12420)

This reverts commit 8e3d18d.
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.

2 participants