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

Redshift - Add username customization #12016

Merged
merged 2 commits into from
Jul 8, 2021
Merged

Conversation

MilenaHC
Copy link
Contributor

@MilenaHC MilenaHC commented Jul 8, 2021

Adds the ability to customize username generation for dynamic users in Redshift.

Uses the new field username_template with the go template language.

@MilenaHC MilenaHC requested a review from a team July 8, 2021 14:38
@MilenaHC
Copy link
Contributor Author

MilenaHC commented Jul 8, 2021

Test

Ran existing acceptance tests and manually tested that username customization works as expected

=== RUN   TestRedshift_Initialize
--- PASS: TestRedshift_Initialize (0.48s)
=== RUN   TestRedshift_NewUser
--- PASS: TestRedshift_NewUser (2.85s)
=== RUN   TestRedshift_NewUser_NoCreationStatement_ShouldError
--- PASS: TestRedshift_NewUser_NoCreationStatement_ShouldError (0.42s)
=== RUN   TestRedshift_UpdateUser_Expiration
--- PASS: TestRedshift_UpdateUser_Expiration (11.14s)
=== RUN   TestRedshift_UpdateUser_Password
--- PASS: TestRedshift_UpdateUser_Password (3.49s)
=== RUN   TestRedshift_DeleteUser
--- PASS: TestRedshift_DeleteUser (6.53s)
=== RUN   TestRedshift_DefaultUsernameTemplate
--- PASS: TestRedshift_DefaultUsernameTemplate (2.78s)
=== RUN   TestRedshift_CustomUsernameTemplate
--- PASS: TestRedshift_CustomUsernameTemplate (2.70s)
PASS
ok      github.com/hashicorp/vault/plugins/database/redshift    30.531s

@MilenaHC MilenaHC added this to the 1.8 milestone Jul 8, 2021
Copy link
Contributor

@jasonodonnell jasonodonnell left a comment

Choose a reason for hiding this comment

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

LGTM!

@MilenaHC MilenaHC merged commit e507785 into main Jul 8, 2021
@MilenaHC MilenaHC deleted the username-customization-redshift branch July 8, 2021 15:29
MilenaHC added a commit that referenced this pull request Jul 8, 2021
* username customization for redshift

* adding changelog and updating api-docs
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