-
-
Notifications
You must be signed in to change notification settings - Fork 143
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Support for IAM + AWS Managed Kafka (MSK) #198
Comments
How does authentication with MSK work? Anything SASL based? Maybe we can make use of the options we support. |
+1 I'm also interested in that. Here's the library that AWS provides for accessing its managed cluster using SASL and IAM - https://github.com/aws/aws-msk-iam-auth. |
Hm..i could not find a go library for it :( It guess it would be possible to implement it. We support SASL, and SASL implementations should be pluggable. |
FYI: we use sarama on kaf, but i think it should be possible to build a wrapper around this for use in kaf/sarama. However, i don't have the time for it - if somebody is interested, i'll be happy to accept a PR. |
Hello, I have a working version in this branch: https://github.com/bpaquet/kaf/tree/msk_iam This cannot be merged, because this PR need to be merged before in Sarama. But in the mean time, you can checkout the branch above, build and enjoy :) |
FWIW: https://github.com/aws/aws-msk-iam-sasl-signer-go |
#313 addresses this |
Top effort @Dawnflash. I was just starting to investigate a kaf yesterday and this was my first hurdle. In the middle of testing your fork but initial testing shows it's working well 👍 Thanks for your efforts! |
Neat project 👋 hopefully I'm not missing something obvious here (I checked the examples folder) but it doesn't seem like there is any way to configure kaf to use IAM in order to connect to an AWS managed kafka cluster.
This would be an incredibly useful feature. I am happy to try my best to contribute :) though I have never written golang beyond a couple tutorials
The text was updated successfully, but these errors were encountered: