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 the option to iamAuthentication in aws #160

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

romga
Copy link

@romga romga commented Aug 22, 2024

Aws has the ability to set iam authentication with RDS.

We have added this ability to the operator

@hitman99
Copy link
Member

This is cool, I hope I can check this PR soon.

@eightnoneone
Copy link

@romga Thanks for doing this. It's a requirement for how we plan to authenticate Postgres RDS connections going forward.

I'm no Go developer, but looks like the code that handles Azure fails it's checks since you've added the additional param to the CreateUserRole(). Can a function's params be configured as optional in Go?

It wasn't clear from the code, but does this deal with the operator then not creating and storing a password in the Secret? I imagine one could alter the secretTemplate to not include it for each spec.

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.

4 participants