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 vault aws iam auth #1607

Closed

Conversation

cludden
Copy link

@cludden cludden commented Jan 25, 2023

This change introduces support for the iam flavor of vault's aws auth engine. While the referenced, unmerged PR introduces a similar feature set, this implementation opts for using the same go libraries used by the vault cli for generating the login parameters.

related-pr: #191

This change introduces support for the iam flavor of vault's aws auth
engine. While the referenced, unmerged PR introduces a similar feature
set, this implementation opts for using the same go libraries used by
the vault cli for generating the login parameters.

related-pr: hairyhenderson#191
@hairyhenderson
Copy link
Owner

@cludden thanks for working on this, however it would've been ideal if you'd opened an issue first...

In gomplate v4, which will hopefully be the next release, I'm moving over to use go-fsimpl for all datasources, and the vaultfs implementation already supports this: https://pkg.go.dev/github.com/hairyhenderson/go-fsimpl/vaultfs/vaultauth

I've been (slowly) working on incorporating that in #1336.

@cludden
Copy link
Author

cludden commented Jan 26, 2023

I normally would have, but saw there was an open, but very out-of-date PR for this very feature, so referenced that in my commit instead. Happy to close this in favor of the upcoming v4 implementation..

@cludden cludden closed this Jan 26, 2023
@hairyhenderson
Copy link
Owner

@cludden ok, fair enough! thanks for understanding 🙂

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