diff --git a/ui/app/services/permissions.js b/ui/app/services/permissions.js index 612de50610aa..5e3de8b7df95 100644 --- a/ui/app/services/permissions.js +++ b/ui/app/services/permissions.js @@ -2,7 +2,6 @@ import Service, { inject as service } from '@ember/service'; import { task } from 'ember-concurrency'; const API_PATHS = { - secrets: { engine: 'cubbyhole/' }, access: { methods: 'sys/auth', entities: 'identity/entities', diff --git a/ui/app/templates/vault/cluster.hbs b/ui/app/templates/vault/cluster.hbs index b82ae1173cb8..f7a551ee2ab7 100644 --- a/ui/app/templates/vault/cluster.hbs +++ b/ui/app/templates/vault/cluster.hbs @@ -17,18 +17,16 @@ {{/if}} - {{#if (has-permission 'secrets')}} -
  • - {{#link-to - "vault.cluster.secrets" - current-when="vault.cluster.secrets vault.cluster.settings.mount-secret-backend vault.cluster.settings.configure-secret-backend" - invokeAction=(action Nav.closeDrawer) - data-test-navbar-item='secrets' - }} - Secrets - {{/link-to}} -
  • - {{/if}} +
  • + {{#link-to + "vault.cluster.secrets" + current-when="vault.cluster.secrets vault.cluster.settings.mount-secret-backend vault.cluster.settings.configure-secret-backend" + invokeAction=(action Nav.closeDrawer) + data-test-navbar-item='secrets' + }} + Secrets + {{/link-to}} +
  • {{#if (has-permission 'access')}}
  • {{#link-to diff --git a/ui/tests/acceptance/cluster-test.js b/ui/tests/acceptance/cluster-test.js index 9463efef24f2..8518405673c0 100644 --- a/ui/tests/acceptance/cluster-test.js +++ b/ui/tests/acceptance/cluster-test.js @@ -39,21 +39,6 @@ module('Acceptance | cluster', function(hooks) { await logout.visit(); }); - test('shows nav item if user does have permission', async function(assert) { - const read_secrets_policy = `' - path "cubbyhole/" { - capabilities = ["read"] - }, - '`; - - const userToken = await tokenWithPolicy('show-secrets-nav', read_secrets_policy); - await logout.visit(); - await authPage.login(userToken); - - assert.dom('[data-test-navbar-item=secrets]').exists(); - await logout.visit(); - }); - test('enterprise nav item links to first route that user has access to', async function(assert) { const read_rgp_policy = `' path "sys/policies/rgp" {