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

Feature: Add support for logstash output #3

Merged
merged 1 commit into from
Dec 14, 2022
Merged

Conversation

noqqe
Copy link
Contributor

@noqqe noqqe commented Dec 8, 2022

Hi,

I needed support for output.logstash, so I added it, would be nice to see it in the Module.
If there are any questions, comments or changes required will happily do so :)

@noqqe noqqe force-pushed the main branch 2 times, most recently from dc33d8b to 04ea936 Compare December 8, 2022 12:30
Copy link
Owner

@PacoVK PacoVK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for your contribution. I have some minor suggestions to improve the usability. If that's fine i will take care to merge this PR :)

examples/main.tf Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@noqqe
Copy link
Contributor Author

noqqe commented Dec 14, 2022

Updated docs according to the suggestions :)

@PacoVK
Copy link
Owner

PacoVK commented Dec 14, 2022

LGTM, thank you, will have a closer look and draft a release soon!

@PacoVK PacoVK merged commit 3c9fc7b into PacoVK:main Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants