scottk/vault
1
0
Fork 0
mirror of https://github.com/optim-enterprises-bv/vault.git synced 2025-10-29 09:42:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

UI: Allow token configuration tune from namespace (#24147)

Browse Source 
* check methodType instead of type when checking to remove token_type from payload

* Add changelog
This commit is contained in:
Chelsea Shaw
2023-11-15 16:18:53 -06:00
committed by GitHub
parent 7a8ced4d36
commit 87f09c8b84
3 changed files with 20 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
changelog/24147.txt Normal file
View File

@@ -0,0 +1,3 @@
```release-note:bug
ui: Fix error when tuning token auth configuration within namespace
```

2
ui/app/components/auth-config-form/options.js
View File

@@ -32,7 +32,7 @@ export default AuthConfigComponent.extend({
data.description = this.model.description;
// token_type should not be tuneable for the token auth method.
if (this.model.type === 'token') {
if (this.model.methodType === 'token') {
delete data.token_type;
}

16
ui/tests/acceptance/auth-list-test.js
View File

@@ -151,4 +151,20 @@ module('Acceptance | auth backend list', function (hooks) {
}
}
});
test('enterprise: token config within namespace', async function (assert) {
const ns = 'ns-wxyz';
await runCmd(`write sys/namespaces/${ns} -f`);
await authPage.loginNs(ns);
// go directly to token configure route
await visit('/vault/settings/auth/configure/token/options');
await fillIn('[data-test-input="description"]', 'My custom description');
await click('[data-test-save-config="true"]');
assert.strictEqual(currentURL(), '/vault/access', 'successfully saves and navigates away');
await click('[data-test-auth-backend-link="token"]');
assert
.dom('[data-test-row-value="Description"]')
.hasText('My custom description', 'description was saved');
await runCmd(`delete sys/namespaces/${ns}`);
});
});