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

Enable mount accessor export for vault_github_auth_backend #350

Conversation

cvbarros
Copy link
Contributor

This PR adds accessor attribute for vault_github_auth_backend resource. This is useful to integrate an auth mount with resources such as vault_identity_entity_group_alias and vault_identity_entity_alias.

Copy link
Contributor

@tyrannosaurus-becks tyrannosaurus-becks left a comment

Choose a reason for hiding this comment

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

Fantastic. Thank you!

if err != nil {
return fmt.Errorf("error reading github auth mount from '%q': %s", path, err)
}
log.Printf("[INFO] Read github auth mount from '%q'", path)
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor thing I learned recently - if you use %q it automatically adds quotes surrounding the field. Check this out. This is in no way a blocker for the PR, just an interesting tidbit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, Becca. Really nice tip to know!

@tyrannosaurus-becks tyrannosaurus-becks merged commit 810c71c into hashicorp:master Mar 29, 2019
dandandy pushed a commit to dandandy/terraform-provider-vault that referenced this pull request Jun 17, 2021
…nt-accessor

Enable mount accessor export for vault_github_auth_backend
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