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

Improve credentials management #44

Closed
2 of 3 tasks
zhenik opened this issue Oct 27, 2020 · 0 comments · Fixed by #53
Closed
2 of 3 tasks

Improve credentials management #44

zhenik opened this issue Oct 27, 2020 · 0 comments · Fixed by #53
Assignees
Milestone

Comments

@zhenik
Copy link
Contributor

zhenik commented Oct 27, 2020

Feature description

Improve credentials management in the example/standalone

  1. (OPTIONAL) Not stored in the clean text in terraform state
  2. (DEFAULT) User should be able to provide own credentials via config (terraform variables)
  3. (OPTIONAL) User should be able to use existing creds from the vault (consul-template)

Q1. Can we have creds in output vars for the module with flag sensitive without breaking requirement #1? (NO)

Why is it needed?

Credentials management improvement improves security

Suggestion(s)/solution(s) [Optional]

Make a switch between user-provided credentials and creds which are managed by vault

Definition of done

User can provide own creds or use vault for creds management

Checklist (after created issue)

  • Added label(s)
  • Added to project
  • Added to milestone
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant