[Authz] Migrated authorized routes owned by @elastic/obs-ux-infra_ser…

…vices-team
elastic · Oct 29, 2024 · f269ea7 · f269ea7
1 parent c7b8ca0
commit f269ea7
Show file tree

Hide file tree

Showing 6 changed files with 53 additions and 10 deletions.
diff --git a/x-pack/plugins/observability_solution/profiling/server/routes/apm.ts b/x-pack/plugins/observability_solution/profiling/server/routes/apm.ts
@@ -34,8 +34,12 @@ export function registerTopNFunctionsAPMTransactionsRoute({
   router.get(
     {
       path: paths.APMTransactions,
+      security: {
+        authz: {
+          requiredPrivileges: ['profiling', 'apm'],
+        },
+      },
       options: {
-        tags: ['access:profiling', 'access:apm'],
         timeout: { idleSocket: IDLE_SOCKET_TIMEOUT },
       },
       validate: { query: querySchema },

diff --git a/x-pack/plugins/observability_solution/profiling/server/routes/flamechart.ts b/x-pack/plugins/observability_solution/profiling/server/routes/flamechart.ts
@@ -23,7 +23,12 @@ export function registerFlameChartSearchRoute({
   router.get(
     {
       path: paths.Flamechart,
-      options: { tags: ['access:profiling'], timeout: { idleSocket: IDLE_SOCKET_TIMEOUT } },
+      security: {
+        authz: {
+          requiredPrivileges: ['profiling'],
+        },
+      },
+      options: { timeout: { idleSocket: IDLE_SOCKET_TIMEOUT } },
       validate: {
         query: schema.object({
           timeFrom: schema.number(),

diff --git a/x-pack/plugins/observability_solution/profiling/server/routes/functions.ts b/x-pack/plugins/observability_solution/profiling/server/routes/functions.ts
@@ -34,7 +34,12 @@ export function registerTopNFunctionsSearchRoute({
   router.get(
     {
       path: paths.TopNFunctions,
-      options: { tags: ['access:profiling'], timeout: { idleSocket: IDLE_SOCKET_TIMEOUT } },
+      security: {
+        authz: {
+          requiredPrivileges: ['profiling'],
+        },
+      },
+      options: { timeout: { idleSocket: IDLE_SOCKET_TIMEOUT } },
       validate: { query: querySchema },
     },
     async (context, request, response) => {

diff --git a/x-pack/plugins/observability_solution/profiling/server/routes/setup/route.ts b/x-pack/plugins/observability_solution/profiling/server/routes/setup/route.ts
@@ -27,7 +27,11 @@ export function registerSetupRoute({
   router.get(
     {
       path: paths.HasSetupESResources,
-      options: { tags: ['access:profiling'] },
+      security: {
+        authz: {
+          requiredPrivileges: ['profiling'],
+        },
+      },
       validate: false,
     },
     async (context, request, response) => {
@@ -62,7 +66,11 @@ export function registerSetupRoute({
   router.post(
     {
       path: paths.HasSetupESResources,
-      options: { tags: ['access:profiling'] },
+      security: {
+        authz: {
+          requiredPrivileges: ['profiling'],
+        },
+      },
       validate: false,
     },
     async (context, request, response) => {
@@ -166,7 +174,11 @@ export function registerSetupRoute({
   router.get(
     {
       path: paths.SetupDataCollectionInstructions,
-      options: { tags: ['access:profiling'] },
+      security: {
+        authz: {
+          requiredPrivileges: ['profiling'],
+        },
+      },
       validate: false,
     },
     async (context, request, response) => {

diff --git a/x-pack/plugins/observability_solution/profiling/server/routes/storage_explorer/route.ts b/x-pack/plugins/observability_solution/profiling/server/routes/storage_explorer/route.ts
@@ -29,7 +29,11 @@ export function registerStorageExplorerRoute({
   router.get(
     {
       path: paths.StorageExplorerSummary,
-      options: { tags: ['access:profiling'] },
+      security: {
+        authz: {
+          requiredPrivileges: ['profiling'],
+        },
+      },
       validate: {
         query: schema.object({
           indexLifecyclePhase: schema.oneOf([
@@ -112,7 +116,11 @@ export function registerStorageExplorerRoute({
   router.get(
     {
       path: paths.StorageExplorerHostStorageDetails,
-      options: { tags: ['access:profiling'] },
+      security: {
+        authz: {
+          requiredPrivileges: ['profiling'],
+        },
+      },
       validate: {
         query: schema.object({
           indexLifecyclePhase: schema.oneOf([
@@ -156,7 +164,11 @@ export function registerStorageExplorerRoute({
   router.get(
     {
       path: paths.StorageExplorerIndicesStorageDetails,
-      options: { tags: ['access:profiling'] },
+      security: {
+        authz: {
+          requiredPrivileges: ['profiling'],
+        },
+      },
       validate: {
         query: schema.object({
           indexLifecyclePhase: schema.oneOf([

diff --git a/x-pack/plugins/observability_solution/profiling/server/routes/topn.ts b/x-pack/plugins/observability_solution/profiling/server/routes/topn.ts
@@ -171,7 +171,12 @@ export function queryTopNCommon({
   router.get(
     {
       path: pathName,
-      options: { tags: ['access:profiling'], timeout: { idleSocket: IDLE_SOCKET_TIMEOUT } },
+      security: {
+        authz: {
+          requiredPrivileges: ['profiling'],
+        },
+      },
+      options: { timeout: { idleSocket: IDLE_SOCKET_TIMEOUT } },
       validate: {
         query: schema.object({
           timeFrom: schema.number(),