-
Notifications
You must be signed in to change notification settings - Fork 89
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
Support Fleet output Logstash ssl fields #498
Conversation
💚 CLA has been signed |
eh hold on. certificate_authorities isnt working. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking the time to add this into the provider! Sorry the clients a bit nasty to work with right now :)
@daemitus are you able to take a look at the test failures here? |
Now segfaults. I'll have to run the test locally for this one. |
wish I had seen |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one! Thanks for taking the time to get this one in :)
re: #497
Add in the 3 ssl fields to the logstash-type fleet output resource.
I've run it on my dev stack a few times and I'm happy with most everything except the anonymous nested SSL struct that gets generated. Can that get yanked out?