diff --git a/ui/app/adapters/pki/pki-certificate.js b/ui/app/adapters/pki/cert.js similarity index 100% rename from ui/app/adapters/pki/pki-certificate.js rename to ui/app/adapters/pki/cert.js diff --git a/ui/app/adapters/pki-config.js b/ui/app/adapters/pki/pki-config.js similarity index 98% rename from ui/app/adapters/pki-config.js rename to ui/app/adapters/pki/pki-config.js index efc908929a..c94d30d1c7 100644 --- a/ui/app/adapters/pki-config.js +++ b/ui/app/adapters/pki/pki-config.js @@ -2,7 +2,7 @@ import AdapterError from '@ember-data/adapter/error'; import { hash, resolve } from 'rsvp'; import { capitalize } from '@ember/string'; import { set } from '@ember/object'; -import ApplicationAdapter from './application'; +import ApplicationAdapter from '../application'; export default ApplicationAdapter.extend({ namespace: 'v1', diff --git a/ui/app/components/generate-credentials.js b/ui/app/components/generate-credentials.js index e369730070..630c0a0ba0 100644 --- a/ui/app/components/generate-credentials.js +++ b/ui/app/components/generate-credentials.js @@ -16,7 +16,7 @@ const MODEL_TYPES = { backIsListLink: true, }, 'pki-issue': { - model: 'pki/pki-certificate', + model: 'pki/cert', title: 'Issue Certificate', }, 'pki-sign': { diff --git a/ui/app/helpers/options-for-backend.js b/ui/app/helpers/options-for-backend.js index fdc4b1fd5c..bf52622288 100644 --- a/ui/app/helpers/options-for-backend.js +++ b/ui/app/helpers/options-for-backend.js @@ -34,13 +34,13 @@ const SECRET_BACKENDS = { editComponent: 'pki/role-pki-edit', }, { - name: 'certs', + name: 'cert', modelPrefix: 'cert/', label: 'Certificates', searchPlaceholder: 'Filter certificates', item: 'certificates', create: 'Create role', - tab: 'certs', + tab: 'cert', listItemPartial: 'secret-list/pki-cert-item', editComponent: 'pki/pki-cert-show', }, diff --git a/ui/app/models/pki-ca-certificate.js b/ui/app/models/pki-ca-certificate.js index 51249644b6..28b07084d2 100644 --- a/ui/app/models/pki-ca-certificate.js +++ b/ui/app/models/pki-ca-certificate.js @@ -1,6 +1,6 @@ import { attr } from '@ember-data/model'; import { computed } from '@ember/object'; -import Certificate from './pki/pki-certificate'; +import Certificate from './pki/cert'; export default Certificate.extend({ DISPLAY_FIELDS: computed(function () { diff --git a/ui/app/models/pki-certificate-sign.js b/ui/app/models/pki-certificate-sign.js index 718c7220fc..afc937f89c 100644 --- a/ui/app/models/pki-certificate-sign.js +++ b/ui/app/models/pki-certificate-sign.js @@ -1,7 +1,7 @@ import { attr } from '@ember-data/model'; import { copy } from 'ember-copy'; import { computed } from '@ember/object'; -import Certificate from './pki/pki-certificate'; +import Certificate from './pki/cert'; import { combineFieldGroups } from 'vault/utils/openapi-to-attrs'; export default Certificate.extend({ diff --git a/ui/app/models/pki/pki-certificate.js b/ui/app/models/pki/cert.js similarity index 100% rename from ui/app/models/pki/pki-certificate.js rename to ui/app/models/pki/cert.js diff --git a/ui/app/models/pki-config.js b/ui/app/models/pki/pki-config.js similarity index 100% rename from ui/app/models/pki-config.js rename to ui/app/models/pki/pki-config.js diff --git a/ui/app/routes/vault/cluster/secrets/backend/list.js b/ui/app/routes/vault/cluster/secrets/backend/list.js index 7552287a07..dd5fbfd350 100644 --- a/ui/app/routes/vault/cluster/secrets/backend/list.js +++ b/ui/app/routes/vault/cluster/secrets/backend/list.js @@ -80,7 +80,7 @@ export default Route.extend({ ssh: 'role-ssh', transform: this.modelTypeForTransform(tab), aws: 'role-aws', - pki: tab === 'certs' ? 'pki/pki-certificate' : 'pki/pki-role', + pki: `pki/${tab || 'pki-role'}`, // secret or secret-v2 cubbyhole: 'secret', kv: secretEngine.get('modelTypeForKV'), @@ -130,7 +130,7 @@ export default Route.extend({ afterModel(model) { const { tab } = this.paramsFor(this.routeName); const backend = this.enginePathParam(); - if (!tab || tab !== 'certs') { + if (!tab || tab !== 'cert') { return; } return all( @@ -138,7 +138,7 @@ export default Route.extend({ // possible that there is no certificate for them in order to know, // we fetch them specifically on the list page, and then unload the // records if there is no `certificate` attribute on the resultant model - ['ca', 'crl', 'ca_chain'].map((id) => this.store.queryRecord('pki/pki-certificate', { id, backend })) + ['ca', 'crl', 'ca_chain'].map((id) => this.store.queryRecord('pki/cert', { id, backend })) ).then( (results) => { results.rejectBy('certificate').forEach((record) => record.unloadRecord()); diff --git a/ui/app/routes/vault/cluster/secrets/backend/secret-edit.js b/ui/app/routes/vault/cluster/secrets/backend/secret-edit.js index 7375144e5c..3b6b796fe3 100644 --- a/ui/app/routes/vault/cluster/secrets/backend/secret-edit.js +++ b/ui/app/routes/vault/cluster/secrets/backend/secret-edit.js @@ -101,7 +101,7 @@ export default Route.extend(UnloadModelRoute, { ssh: 'role-ssh', transform: this.modelTypeForTransform(secret), aws: 'role-aws', - pki: secret && secret.startsWith('cert/') ? 'pki/pki-certificate' : 'pki/pki-role', + pki: secret && secret.startsWith('cert/') ? 'pki/cert' : 'pki/pki-role', cubbyhole: 'secret', kv: backendModel.get('modelTypeForKV'), keymgmt: `keymgmt/${options.queryParams?.itemType || 'key'}`, @@ -230,7 +230,7 @@ export default Route.extend(UnloadModelRoute, { if (!secret) { secret = '\u0020'; } - if (modelType === 'pki/pki-certificate') { + if (modelType === 'pki/cert') { secret = secret.replace('cert/', ''); } if (modelType.startsWith('transform/')) { diff --git a/ui/app/routes/vault/cluster/settings/configure-secret-backend/section.js b/ui/app/routes/vault/cluster/settings/configure-secret-backend/section.js index b4aa5da1a2..ffb1e8d6a2 100644 --- a/ui/app/routes/vault/cluster/settings/configure-secret-backend/section.js +++ b/ui/app/routes/vault/cluster/settings/configure-secret-backend/section.js @@ -2,6 +2,7 @@ import AdapterError from '@ember-data/adapter/error'; import { set } from '@ember/object'; import Route from '@ember/routing/route'; +// ARG TODO glimmerize const SECTIONS_FOR_TYPE = { pki: ['cert', 'urls', 'crl', 'tidy'], }; @@ -9,14 +10,21 @@ export default Route.extend({ fetchModel() { const { section_name: sectionName } = this.paramsFor(this.routeName); const backendModel = this.modelFor('vault.cluster.settings.configure-secret-backend'); - const modelType = `${backendModel.get('type')}-config`; + const type = backendModel.get('type'); + let modelType; + if (type === 'pki') { + // pki models are in models/pki + modelType = `${type}/${type}-config`; + } else { + modelType = `${type}-config`; + } return this.store .queryRecord(modelType, { backend: backendModel.id, section: sectionName, }) .then((model) => { - model.set('backendType', backendModel.get('type')); + model.set('backendType', type); model.set('section', sectionName); return model; }); diff --git a/ui/app/serializers/pki/pki-certificate.js b/ui/app/serializers/pki/cert.js similarity index 100% rename from ui/app/serializers/pki/pki-certificate.js rename to ui/app/serializers/pki/cert.js diff --git a/ui/app/serializers/pki-config.js b/ui/app/serializers/pki/pki-config.js similarity index 100% rename from ui/app/serializers/pki-config.js rename to ui/app/serializers/pki/pki-config.js diff --git a/ui/app/templates/components/pki/pki-cert-show.hbs b/ui/app/templates/components/pki/pki-cert-show.hbs index a6ba5f89ff..d08e2732dd 100644 --- a/ui/app/templates/components/pki/pki-cert-show.hbs +++ b/ui/app/templates/components/pki/pki-cert-show.hbs @@ -75,7 +75,7 @@
- + Back
diff --git a/ui/app/templates/vault/cluster/secrets/backend/list.hbs b/ui/app/templates/vault/cluster/secrets/backend/list.hbs index f143f46a0e..76f96ad763 100644 --- a/ui/app/templates/vault/cluster/secrets/backend/list.hbs +++ b/ui/app/templates/vault/cluster/secrets/backend/list.hbs @@ -1,5 +1,5 @@ {{#if this.filterFocused}} {{#if this.filterMatchesKey}} diff --git a/ui/app/templates/vault/cluster/secrets/backend/overview.hbs b/ui/app/templates/vault/cluster/secrets/backend/overview.hbs index e1b2e2d8cf..6fea24a5f9 100644 --- a/ui/app/templates/vault/cluster/secrets/backend/overview.hbs +++ b/ui/app/templates/vault/cluster/secrets/backend/overview.hbs @@ -1,5 +1,5 @@ 0, 'lists certs'); await listPage.secrets.objectAt(0).click();