feat(ocm): add basic permissions to ocm backend plugin

PatAKnight · PatAKnight · commit 85fede3ee80b · 2024-04-22T11:52:19.000-04:00
diff --git a/plugins/ocm-backend/package.json b/plugins/ocm-backend/package.json
@@ -42,15 +42,18 @@
   "configSchema": "config.d.ts",
   "dependencies": {
     "@backstage/backend-common": "^0.21.6",
+    "@backstage/backend-dynamic-feature-service": "^0.2.8",
     "@backstage/backend-plugin-api": "^0.6.16",
     "@backstage/backend-tasks": "^0.5.21",
     "@backstage/catalog-client": "^1.6.3",
     "@backstage/catalog-model": "^1.4.5",
     "@backstage/config": "^1.2.0",
+    "@backstage/errors": "^1.2.4",
     "@backstage/plugin-catalog-node": "^1.11.0",
     "@backstage/plugin-kubernetes-common": "^0.7.5",
+    "@backstage/plugin-permission-common": "0.7.13",
+    "@backstage/plugin-permission-node": "0.7.27",
     "@janus-idp/backstage-plugin-ocm-common": "2.3.0",
-    "@backstage/backend-dynamic-feature-service": "^0.2.8",
     "@kubernetes/client-node": "^0.20.0",
     "express": "^4.18.2",
     "express-promise-router": "^4.1.1",
@@ -59,11 +62,9 @@
   },
   "devDependencies": {
     "@backstage/cli": "0.26.2",
-    "@janus-idp/cli": "1.8.0",
     "@backstage/plugin-auth-node": "0.4.11",
     "@backstage/plugin-catalog-backend": "1.21.0",
-    "@backstage/plugin-permission-common": "0.7.13",
-    "@backstage/plugin-permission-node": "0.7.27",
+    "@janus-idp/cli": "1.8.0",
     "@types/express": "4.17.20",
     "@types/supertest": "2.0.16",
     "msw": "1.3.2",
diff --git a/plugins/ocm-backend/src/service/router.test.ts b/plugins/ocm-backend/src/service/router.test.ts
@@ -1,4 +1,5 @@
 import { ConfigReader } from '@backstage/config';
+import { AuthorizeResult } from '@backstage/plugin-permission-common';
 
 import express from 'express';
 import { setupServer } from 'msw/node';
@@ -26,11 +27,33 @@ const logger = createLogger({
   transports: [new transports.Console({ silent: true })],
 });
 
+const mockedAuthorize = jest.fn().mockImplementation(async () => [
+  {
+    result: AuthorizeResult.ALLOW,
+  },
+]);
+
+const mockedAuthorizeConditional = jest.fn().mockImplementation(async () => [
+  {
+    result: AuthorizeResult.ALLOW,
+  },
+]);
+
+const mockPermissionEvaluator = {
+  authorize: mockedAuthorize,
+  authorizeConditional: mockedAuthorizeConditional,
+};
+
+const mockDiscovery = {
+  getBaseUrl: jest.fn(),
+  getExternalBaseUrl: jest.fn(),
+};
+
 describe('createRouter', () => {
   let app: express.Express;
 
   beforeAll(async () => {
-    jest.resetAllMocks();
+    jest.clearAllMocks();
     const router = await createRouter({
       logger: logger,
       config: new ConfigReader({
@@ -46,11 +69,28 @@ describe('createRouter', () => {
           },
         },
       }),
+      permissions: mockPermissionEvaluator,
+      discovery: mockDiscovery,
     });
     app = express().use(router);
   });
 
   describe('GET /status', () => {
+    it('should deny access when getting all clusters', async () => {
+      mockedAuthorize.mockImplementationOnce(async () => [
+        { result: AuthorizeResult.DENY },
+      ]);
+
+      const result = await request(app).get('/status');
+
+      expect(mockedAuthorize).toHaveBeenCalled();
+
+      expect(result.statusCode).toBe(403);
+      expect(result.body.error).toEqual({
+        name: 'NotAllowedError',
+        message: 'Unauthorized',
+      });
+    });
     it('should get all clusters', async () => {
       const result = await request(app).get('/status');
 
@@ -177,6 +217,21 @@ describe('createRouter', () => {
   });
 
   describe('GET /status/:hubName/:clusterName', () => {
+    it('should deny access when getting all clusters', async () => {
+      mockedAuthorize.mockImplementationOnce(async () => [
+        { result: AuthorizeResult.DENY },
+      ]);
+
+      const result = await request(app).get('/status/foo/cluster1');
+
+      expect(mockedAuthorize).toHaveBeenCalled();
+
+      expect(result.statusCode).toBe(403);
+      expect(result.body.error).toEqual({
+        name: 'NotAllowedError',
+        message: 'Unauthorized',
+      });
+    });
     it('should correctly parse a cluster', async () => {
       const result = await request(app).get('/status/foo/cluster1');
 
diff --git a/plugins/ocm-backend/src/service/router.ts b/plugins/ocm-backend/src/service/router.ts
@@ -14,20 +14,33 @@
  * limitations under the License.
  */
 
-import { errorHandler, loggerToWinstonLogger } from '@backstage/backend-common';
+import {
+  createLegacyAuthAdapters,
+  errorHandler,
+  loggerToWinstonLogger,
+  PluginEndpointDiscovery,
+} from '@backstage/backend-common';
 import {
   coreServices,
   createBackendPlugin,
+  HttpAuthService,
+  PermissionsService,
 } from '@backstage/backend-plugin-api';
 import { Config } from '@backstage/config';
+import { NotAllowedError } from '@backstage/errors';
+import { AuthorizeResult } from '@backstage/plugin-permission-common';
+import { createPermissionIntegrationRouter } from '@backstage/plugin-permission-node';
 
 import express from 'express';
 import Router from 'express-promise-router';
+import { Request } from 'express-serve-static-core';
 import { Logger } from 'winston';
 
 import {
   Cluster,
   ClusterOverview,
+  ocmClusterReadPermission,
+  ocmEntityPermissions,
 } from '@janus-idp/backstage-plugin-ocm-common';
 
 import { readOcmConfigs } from '../helpers/config';
@@ -52,11 +65,25 @@ import { ManagedClusterInfo } from '../types';
 export interface RouterOptions {
   logger: Logger;
   config: Config;
+  discovery: PluginEndpointDiscovery;
+  permissions: PermissionsService;
+  httpAuth?: HttpAuthService;
 }
 
-const buildRouter = (config: Config, logger: Logger) => {
+const buildRouter = (
+  config: Config,
+  logger: Logger,
+  httpAuth: HttpAuthService,
+  permissions: PermissionsService,
+) => {
   const router = Router();
+
+  const permissionsIntegrationRouter = createPermissionIntegrationRouter({
+    permissions: ocmEntityPermissions,
+  });
+
   router.use(express.json());
+  router.use(permissionsIntegrationRouter);
 
   const clients = Object.fromEntries(
     readOcmConfigs(config).map(provider => [
@@ -68,43 +95,63 @@ const buildRouter = (config: Config, logger: Logger) => {
     ]),
   );
 
-  router.get(
-    '/status/:providerId/:clusterName',
-    async ({ params: { clusterName, providerId } }, response) => {
-      logger.debug(
-        `Incoming status request for ${clusterName} cluster on ${providerId} hub`,
-      );
-
-      if (!clients.hasOwnProperty(providerId)) {
-        throw Object.assign(new Error('Hub not found'), {
-          statusCode: 404,
-          name: 'HubNotFound',
-        });
-      }
-
-      const normalizedClusterName = translateResourceToOCM(
-        clusterName,
-        clients[providerId].hubResourceName,
-      );
-
-      const mc = await getManagedCluster(
-        clients[providerId].client,
-        normalizedClusterName,
-      );
-      const mci = await getManagedClusterInfo(
-        clients[providerId].client,
-        normalizedClusterName,
-      );
-
-      response.send({
-        name: clusterName,
-        ...parseManagedCluster(mc),
-        ...parseUpdateInfo(mci),
-      } as Cluster);
-    },
-  );
+  const authorize = async (request: Request) => {
+    const decision = (
+      await permissions.authorize([{ permission: ocmClusterReadPermission }], {
+        credentials: await httpAuth.credentials(request),
+      })
+    )[0];
+
+    return decision;
+  };
+
+  router.get('/status/:providerId/:clusterName', async (request, response) => {
+    const decision = await authorize(request);
+
+    if (decision.result === AuthorizeResult.DENY) {
+      throw new NotAllowedError('Unauthorized');
+    }
+
+    const { clusterName, providerId } = request.params;
+    logger.debug(
+      `Incoming status request for ${clusterName} cluster on ${providerId} hub`,
+    );
+
+    if (!clients.hasOwnProperty(providerId)) {
+      throw Object.assign(new Error('Hub not found'), {
+        statusCode: 404,
+        name: 'HubNotFound',
+      });
+    }
+
+    const normalizedClusterName = translateResourceToOCM(
+      clusterName,
+      clients[providerId].hubResourceName,
+    );
+
+    const mc = await getManagedCluster(
+      clients[providerId].client,
+      normalizedClusterName,
+    );
+    const mci = await getManagedClusterInfo(
+      clients[providerId].client,
+      normalizedClusterName,
+    );
+
+    response.send({
+      name: clusterName,
+      ...parseManagedCluster(mc),
+      ...parseUpdateInfo(mci),
+    } as Cluster);
+  });
+
+  router.get('/status', async (request, response) => {
+    const decision = await authorize(request);
+
+    if (decision.result === AuthorizeResult.DENY) {
+      throw new NotAllowedError('Unauthorized');
+    }
 
-  router.get('/status', async (_, response) => {
     logger.debug(`Incoming status request for all clusters`);
 
     const allClusters = await Promise.all(
@@ -144,8 +191,11 @@ export async function createRouter(
 ): Promise<express.Router> {
   const { logger } = options;
   const { config } = options;
+  const { permissions } = options;
+
+  const { httpAuth } = createLegacyAuthAdapters(options);
 
-  return buildRouter(config, logger);
+  return buildRouter(config, logger, httpAuth, permissions);
 }
 
 export const ocmPlugin = createBackendPlugin({
@@ -156,9 +206,18 @@ export const ocmPlugin = createBackendPlugin({
         logger: coreServices.logger,
         config: coreServices.rootConfig,
         http: coreServices.httpRouter,
+        httpAuth: coreServices.httpAuth,
+        permissions: coreServices.permissions,
       },
-      async init({ config, logger, http }) {
-        http.use(buildRouter(config, loggerToWinstonLogger(logger)));
+      async init({ config, logger, http, httpAuth, permissions }) {
+        http.use(
+          buildRouter(
+            config,
+            loggerToWinstonLogger(logger),
+            httpAuth,
+            permissions,
+          ),
+        );
       },
     });
   },
diff --git a/plugins/ocm-common/package.json b/plugins/ocm-common/package.json
@@ -35,5 +35,8 @@
     "plugin"
   ],
   "homepage": "https://janus-idp.io/",
-  "bugs": "https://github.com/janus-idp/backstage-plugins/issues"
+  "bugs": "https://github.com/janus-idp/backstage-plugins/issues",
+  "dependencies": {
+    "@backstage/plugin-permission-common": "^0.7.13"
+  }
 }
diff --git a/plugins/ocm-common/src/index.ts b/plugins/ocm-common/src/index.ts
@@ -1,60 +1,11 @@
-/***/
 /**
  * Common functionalities for the Open Cluster Management plugin.
  *
  * @packageDocumentation
  */
 
-export type ClusterStatus = {
-  available: boolean;
-  reason?: string;
-};
-
-export type ClusterBase = {
-  name: string;
-};
-
-export type ClusterUpdate = {
-  available?: boolean;
-  version?: string;
-  url?: string;
-};
-
-export type ClusterNodesStatus = {
-  status: string;
-  type: string;
-};
-
-export type ClusterDetails = {
-  consoleUrl?: string;
-  kubernetesVersion?: string;
-  oauthUrl?: string;
-  openshiftId?: string;
-  openshiftVersion?: string;
-  platform?: string;
-  region?: string;
-  allocatableResources?: {
-    cpuCores?: number;
-    memorySize?: string;
-    numberOfPods?: number;
-  };
-  availableResources?: {
-    cpuCores?: number;
-    memorySize?: string;
-    numberOfPods?: number;
-  };
-  update?: ClusterUpdate;
-  status: ClusterStatus;
-};
-
-export type Cluster = ClusterBase & ClusterDetails;
-export type ClusterOverview = ClusterBase & {
-  status: ClusterStatus;
-  update: ClusterUpdate;
-  platform: string;
-  openshiftVersion: string;
-  nodes: Array<ClusterNodesStatus>;
-};
+export * from './types';
+export * from './permissions';
 
 export const ANNOTATION_CLUSTER_ID = 'janus-idp.io/ocm-cluster-id';
 export const ANNOTATION_PROVIDER_ID = 'janus-idp.io/ocm-provider-id';
diff --git a/plugins/ocm-common/src/permissions.ts b/plugins/ocm-common/src/permissions.ts
diff --git a/plugins/ocm-common/src/types.ts b/plugins/ocm-common/src/types.ts
