upstream/vault
1
0
Fork 0
mirror of https://github.com/hashicorp/vault.git synced 2026-02-03 20:40:45 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

Copy ui: Add Configuration path for KV v2 secrets into main (#11585) (#11646)

Browse source 
* Copy https://github.com/hashicorp/vault/pull/31679 into main



* ui: Add Configuration path for KV v2 secrets

* ui: Add Configuration metadata path for KV v2 secrets

* rename changelog for ent repo

---------

Co-authored-by: RamdaneBelkhir <belkhirramdane4@gmail.com>
Co-authored-by: claire bontempo <cbontempo@hashicorp.com>
Co-authored-by: claire bontempo <68122737+hellobontempo@users.noreply.github.com>
This commit is contained in:
Vault Automation 2026-01-07 17:40:37 -07:00 committed by GitHub
parent f8bbbce374
commit bb9a3cb398
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
6 changed files with 57 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
changelog/_11585.txt Normal file
View file

@ -0,0 +1,3 @@
```release-note:improvement
ui: Add "Configuration path" and "Configuration metadata path" fields to KV v2 secret paths page showing paths without /v1/ prefix for use in policies, Vault Agent configurations, and other tools that reference the logical path.
```

2
ui/lib/kv/addon/components/kv-paths-card.hbs
View file

@ -9,7 +9,7 @@
</Hds::Text::Display>
{{#if @isCondensed}}
<Hds::Text::Body @tag="p" @color="faint">
The paths to use when referring to this secret in API or CLI.
The paths to use when referring to this secret in API, policies, or CLI.
</Hds::Text::Body>
{{/if}}

10
ui/lib/kv/addon/components/kv-paths-card.js
View file

@ -39,6 +39,11 @@ export default class KvPathsCard extends Component {
snippet: namespace ? `/v1/${encodePath(namespace)}/${data}` : `/v1/${data}`,
text: 'Use this path when referring to this secret in the API.',
},
{
label: 'Configuration path',
snippet: namespace ? `${encodePath(namespace)}/${data}` : data,
text: 'Use this path in policies, Vault Agent configurations, and other tools that reference the logical path.',
},
{
label: 'CLI path',
snippet: namespace ? `-namespace=${namespace} ${cli}` : cli,
@ -52,6 +57,11 @@ export default class KvPathsCard extends Component {
snippet: namespace ? `/v1/${encodePath(namespace)}/${metadata}` : `/v1/${metadata}`,
text: `Use this path when referring to this secret's metadata in the API and permanent secret deletion.`,
},
{
label: 'Configuration metadata path',
snippet: namespace ? `${encodePath(namespace)}/${metadata}` : metadata,
text: 'Use this path in policies and other tools that reference the logical metadata path.',
},
]),
];
}

36
ui/tests/acceptance/secrets/backend/kv/kv-v2-workflow-version-history-paths-test.js
View file

@ -91,12 +91,16 @@ module('Acceptance | kv-v2 workflow | version history, paths', function (hooks)
`/vault/secrets-engines/${this.urlPath}/paths`,
'navigates to secret paths route'
);
assert.dom(PAGE.infoRow).exists({ count: 3 }, 'shows 3 rows of information');
assert.dom(PAGE.infoRow).exists({ count: 5 }, 'shows 5 rows of information');
assert.dom(PAGE.infoRowValue('API path')).hasText(`/v1/${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('Configuration path')).hasText(`${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('CLI path')).hasText(`-mount="${this.backend}" "${this.secretPath}"`);
assert
.dom(PAGE.infoRowValue('API path for metadata'))
.hasText(`/v1/${this.backend}/metadata/${this.secretPath}`);
assert
.dom(PAGE.infoRowValue('Configuration metadata path'))
.hasText(`${this.backend}/metadata/${this.secretPath}`);
});
});
@ -117,12 +121,16 @@ module('Acceptance | kv-v2 workflow | version history, paths', function (hooks)
`/vault/secrets-engines/${this.urlPath}/paths`,
'navigates to secret paths route'
);
assert.dom(PAGE.infoRow).exists({ count: 3 }, 'shows 3 rows of information');
assert.dom(PAGE.infoRow).exists({ count: 5 }, 'shows 5 rows of information');
assert.dom(PAGE.infoRowValue('API path')).hasText(`/v1/${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('Configuration path')).hasText(`${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('CLI path')).hasText(`-mount="${this.backend}" "${this.secretPath}"`);
assert
.dom(PAGE.infoRowValue('API path for metadata'))
.hasText(`/v1/${this.backend}/metadata/${this.secretPath}`);
assert
.dom(PAGE.infoRowValue('Configuration metadata path'))
.hasText(`${this.backend}/metadata/${this.secretPath}`);
});
});
@ -143,12 +151,16 @@ module('Acceptance | kv-v2 workflow | version history, paths', function (hooks)
`/vault/secrets-engines/${this.urlPath}/paths`,
'navigates to secret paths route'
);
assert.dom(PAGE.infoRow).exists({ count: 3 }, 'shows 3 rows of information');
assert.dom(PAGE.infoRow).exists({ count: 5 }, 'shows 5 rows of information');
assert.dom(PAGE.infoRowValue('API path')).hasText(`/v1/${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('Configuration path')).hasText(`${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('CLI path')).hasText(`-mount="${this.backend}" "${this.secretPath}"`);
assert
.dom(PAGE.infoRowValue('API path for metadata'))
.hasText(`/v1/${this.backend}/metadata/${this.secretPath}`);
assert
.dom(PAGE.infoRowValue('Configuration metadata path'))
.hasText(`${this.backend}/metadata/${this.secretPath}`);
});
});
@ -191,12 +203,16 @@ module('Acceptance | kv-v2 workflow | version history, paths', function (hooks)
`/vault/secrets-engines/${this.urlPath}/paths`,
'navigates to secret paths route'
);
assert.dom(PAGE.infoRow).exists({ count: 3 }, 'shows 3 rows of information');
assert.dom(PAGE.infoRow).exists({ count: 5 }, 'shows 5 rows of information');
assert.dom(PAGE.infoRowValue('API path')).hasText(`/v1/${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('Configuration path')).hasText(`${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('CLI path')).hasText(`-mount="${this.backend}" "${this.secretPath}"`);
assert
.dom(PAGE.infoRowValue('API path for metadata'))
.hasText(`/v1/${this.backend}/metadata/${this.secretPath}`);
assert
.dom(PAGE.infoRowValue('Configuration metadata path'))
.hasText(`${this.backend}/metadata/${this.secretPath}`);
});
});
@ -217,12 +233,16 @@ module('Acceptance | kv-v2 workflow | version history, paths', function (hooks)
`/vault/secrets-engines/${this.urlPath}/paths`,
'navigates to secret paths route'
);
assert.dom(PAGE.infoRow).exists({ count: 3 }, 'shows 3 rows of information');
assert.dom(PAGE.infoRow).exists({ count: 5 }, 'shows 5 rows of information');
assert.dom(PAGE.infoRowValue('API path')).hasText(`/v1/${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('Configuration path')).hasText(`${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('CLI path')).hasText(`-mount="${this.backend}" "${this.secretPath}"`);
assert
.dom(PAGE.infoRowValue('API path for metadata'))
.hasText(`/v1/${this.backend}/metadata/${this.secretPath}`);
assert
.dom(PAGE.infoRowValue('Configuration metadata path'))
.hasText(`${this.backend}/metadata/${this.secretPath}`);
});
});
@ -301,12 +321,16 @@ path "${this.backend}/*" {
`/vault/secrets-engines/${this.urlPath}/paths`,
'navigates to secret paths route'
);
assert.dom(PAGE.infoRow).exists({ count: 3 }, 'shows 3 rows of information');
assert.dom(PAGE.infoRow).exists({ count: 5 }, 'shows 5 rows of information');
assert.dom(PAGE.infoRowValue('API path')).hasText(`/v1/${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('Configuration path')).hasText(`${this.backend}/data/${this.secretPath}`);
assert.dom(PAGE.infoRowValue('CLI path')).hasText(`-mount="${this.backend}" "${this.secretPath}"`);
assert
.dom(PAGE.infoRowValue('API path for metadata'))
.hasText(`/v1/${this.backend}/metadata/${this.secretPath}`);
assert
.dom(PAGE.infoRowValue('Configuration metadata path'))
.hasText(`${this.backend}/metadata/${this.secretPath}`);
});
});
});

7
ui/tests/integration/components/kv/kv-paths-card-test.js
View file

@ -63,6 +63,7 @@ module('Integration | Component | kv-v2 | KvPathsCard', function (hooks) {
const paths = [
{ label: 'API path', expected: `/v1/${this.backend}/data/${this.path}` },
{ label: 'Configuration path', expected: `${this.backend}/data/${this.path}` },
{ label: 'CLI path', expected: `-mount="${this.backend}" "${this.path}"` },
];
for (const [index, path] of paths.entries()) {
@ -84,8 +85,10 @@ module('Integration | Component | kv-v2 | KvPathsCard', function (hooks) {
test('it renders copyable paths', async function (assert) {
const paths = [
{ label: 'API path', expected: `/v1/${this.backend}/data/${this.path}` },
{ label: 'Configuration path', expected: `${this.backend}/data/${this.path}` },
{ label: 'CLI path', expected: `-mount="${this.backend}" "${this.path}"` },
{ label: 'API path for metadata', expected: `/v1/${this.backend}/metadata/${this.path}` },
{ label: 'Configuration metadata path', expected: `${this.backend}/metadata/${this.path}` },
];
await this.renderComponent();
@ -106,6 +109,10 @@ module('Integration | Component | kv-v2 | KvPathsCard', function (hooks) {
label: 'API path',
expected: `/v1/${backend}/data/${path}`,
},
{
label: 'Configuration path',
expected: `${backend}/data/${path}`,
},
{ label: 'CLI path', expected: `-mount="${this.backend}" "${this.path}"` },
{
label: 'API path for metadata',

12
ui/tests/integration/components/kv/page/kv-page-overview-test.js
View file

@ -116,7 +116,7 @@ module('Integration | Component | kv-v2 | Page::Secret::Overview', function (hoo
assert
.dom(overviewCard.container('Paths'))
.hasText(
`Paths The paths to use when referring to this secret in API or CLI. API path /v1/${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
`Paths The paths to use when referring to this secret in API, policies, or CLI. API path /v1/${this.backend}/data/${this.path} Configuration path ${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
);
assert
.dom(overviewCard.container('Subkeys'))
@ -162,7 +162,7 @@ module('Integration | Component | kv-v2 | Page::Secret::Overview', function (hoo
assert
.dom(overviewCard.container('Paths'))
.hasText(
`Paths The paths to use when referring to this secret in API or CLI. API path /v1/${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
`Paths The paths to use when referring to this secret in API, policies, or CLI. API path /v1/${this.backend}/data/${this.path} Configuration path ${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
);
assert
.dom(overviewCard.container('Subkeys'))
@ -191,7 +191,7 @@ module('Integration | Component | kv-v2 | Page::Secret::Overview', function (hoo
assert
.dom(overviewCard.container('Paths'))
.hasText(
`Paths The paths to use when referring to this secret in API or CLI. API path /v1/${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
`Paths The paths to use when referring to this secret in API, policies, or CLI. API path /v1/${this.backend}/data/${this.path} Configuration path ${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
);
assert.dom(overviewCard.container('Subkeys')).doesNotExist();
});
@ -206,7 +206,7 @@ module('Integration | Component | kv-v2 | Page::Secret::Overview', function (hoo
assert
.dom(overviewCard.container('Paths'))
.hasText(
`Paths The paths to use when referring to this secret in API or CLI. API path /v1/${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
`Paths The paths to use when referring to this secret in API, policies, or CLI. API path /v1/${this.backend}/data/${this.path} Configuration path ${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
);
});
});
@ -250,7 +250,7 @@ module('Integration | Component | kv-v2 | Page::Secret::Overview', function (hoo
assert
.dom(overviewCard.container('Paths'))
.hasText(
`Paths The paths to use when referring to this secret in API or CLI. API path /v1/${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
`Paths The paths to use when referring to this secret in API, policies, or CLI. API path /v1/${this.backend}/data/${this.path} Configuration path ${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
);
});
@ -326,7 +326,7 @@ module('Integration | Component | kv-v2 | Page::Secret::Overview', function (hoo
assert
.dom(overviewCard.container('Paths'))
.hasText(
`Paths The paths to use when referring to this secret in API or CLI. API path /v1/${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
`Paths The paths to use when referring to this secret in API, policies, or CLI. API path /v1/${this.backend}/data/${this.path} Configuration path ${this.backend}/data/${this.path} CLI path -mount="${this.backend}" "${this.path}"`
);
});