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

Add support for AWS MSK IAM authentication #1

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

korsosa
Copy link
Owner

@korsosa korsosa commented Sep 7, 2022

Planning to merge this change upstream, please visit the PR in the official Logstash plugins repository for more information:
logstash-plugins#126

@99ttz
Copy link

99ttz commented Sep 23, 2022

is it possible to get a build published until the PR gets approved? getting from source code to ruby gem is a head scratcher :(. thank you!

@korsosa
Copy link
Owner Author

korsosa commented Sep 29, 2022

Hey @99ttz ! sure, I uploaded a release:
https://github.com/korsosa/logstash-integration-kafka/releases/tag/logstash-integration-kafka-10.12.0-iam

I have an open pull-request upstream and waiting on Elastic to merge this into the main Kafka plugin. If that happens, then this MSK IAM auth support will be part of the official Kafka plugin.
I suggest to follow the PR logstash-plugins#126 and once it's merged, switch back to the official Kafka plugin from this custom build.

@99ttz
Copy link

99ttz commented Sep 29, 2022

thank you very much!

@korsosa
Copy link
Owner Author

korsosa commented Oct 1, 2022

Hey @99ttz, I wrote a Github gist on how to build the Logstash plugin. I couldn't build the plugin based on the Logstash readmes or Elastic's documentation but found a way to build it in Docker and wanted to document it for anyone who might need it (also for myself).

If you'd like to build your own Logstash plugin gem, feel free to give it a go:
https://gist.github.com/korsosa/d58b8fdaa733716c19837795d75257ea

@otavioprado
Copy link

@robbavey when this PR will be merged?

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.

3 participants