fix(notifications): use baseUrl from configuration (#1092)

janus-idp · Jan 21, 2024 · eed63a9 · eed63a9
1 parent b7a9dc0
commit eed63a9
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/plugins/notifications/src/api/NotificationsApiImpl.ts b/plugins/notifications/src/api/NotificationsApiImpl.ts
@@ -1,4 +1,4 @@
-import { FetchApi } from '@backstage/core-plugin-api';
+import { ConfigApi, FetchApi } from '@backstage/core-plugin-api';
 
 import {
   Configuration,
@@ -15,14 +15,19 @@ import {
 
 export type NotificationsApiOptions = {
   fetchApi: FetchApi;
+  configApi: ConfigApi;
 };
 
 export class NotificationsApiImpl implements NotificationsApi {
   private readonly backendRestApi: NotificationsOpenApi;
 
   constructor(options: NotificationsApiOptions) {
+    const backendUrl = options.configApi.getString('backend.baseUrl');
+    const basePath = `${backendUrl}/api/notifications`;
+
     const configuration = new Configuration({
       fetchApi: options.fetchApi.fetch,
+      basePath,
     });
     this.backendRestApi = new NotificationsOpenApi(configuration);
   }

diff --git a/plugins/notifications/src/plugin.ts b/plugins/notifications/src/plugin.ts
@@ -1,9 +1,9 @@
 import {
+  configApiRef,
   createApiFactory,
   createPlugin,
   createRoutableExtension,
   fetchApiRef,
-  // identityApiRef,
 } from '@backstage/core-plugin-api';
 
 import { NotificationsApiImpl, notificationsApiRef } from './api';
@@ -17,10 +17,11 @@ export const notificationsPlugin = createPlugin({
   apis: [
     createApiFactory({
       api: notificationsApiRef,
-      deps: { fetchApi: fetchApiRef },
-      factory({ fetchApi }) {
+      deps: { fetchApi: fetchApiRef, configApi: configApiRef },
+      factory({ fetchApi, configApi }) {
         return new NotificationsApiImpl({
           fetchApi,
+          configApi,
         });
       },
     }),