-
Notifications
You must be signed in to change notification settings - Fork 4.5k
🐙 octavia-cli: secret management #10885
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
Conversation
28ecc10
to
69e4eba
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM amazing @alafanechere
include_hash: | ||
title: "Include hash" | ||
description: "If true, include a hash with each data block." | ||
include_checksum: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alafanechere are you chancing to checksum?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here the original field name for this S3 destination connector was include_checksum
. I mistakenly renamed it to include_hash
while doing a brutal search and replace it when renaming other code occurrences of checksum
to hash
.
What
We want secret management to avoid storing credentials as plain text.
How
${MY_ENV_VAR}
checksum
tohash
(because it's a hash and not a checksum...)Recommended reading order
🚨 User Impact 🚨
Configuration can use
${MY_ENV_VAR}
strings that will be expanded to the environment variable value at run time.