-
Notifications
You must be signed in to change notification settings - Fork 56
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
feat: Add the folder path to the vault helm values #1761
base: main
Are you sure you want to change the base?
feat: Add the folder path to the vault helm values #1761
Conversation
5a6e511
to
3accaac
Compare
6e4346f
to
844cc3b
Compare
844cc3b
to
a990eb6
Compare
Quality Gate passedIssues Measures |
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.
One question: Perhaps I miss something, but this change adds a configuration value to the helm charts, I miss the use of this configuration parameter somewhere. My understanding is, that the value has to be added to the construction of the access path to the vault in the connector, right? Does this already exist? Is this an upstream PR that is referenced?
@lgblaumeiser, yes, we already have this in the connector Declaration: @Setting(description = "The path of the folder that the secret is stored in, relative to VAULT_FOLDER_PATH", required = false, key = "edc.vault.hashicorp.folder")
private String folderPath; Usage: var folderPath = settings.getFolderPath();
var builder = baseUrl(settings)
.newBuilder()
.addPathSegments(PathUtil.trimLeadingOrEndingSlash(vaultApiPath))
.addPathSegment(entryType);
if (folderPath != null) {
builder.addPathSegments(PathUtil.trimLeadingOrEndingSlash(folderPath));
} |
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
WHAT
Add the vault.hashicorp.paths.folder to the helm chart
WHY
To allow specify the folder path, so all edc secrets could be stored directly in the data path.
Closes #1758