backend-plugin-api: moved loggerToWinstonLogger to backend-common

Signed-off-by: Patrik Oldsberg <poldsberg@gmail.com>

packages/backend-app-api/src/services/implementations/configService.ts

@@ -14,11 +14,13 @@
  * limitations under the License.
  */
 
-import { loadBackendConfig } from '@backstage/backend-common';
+import {
+  loadBackendConfig,
+  loggerToWinstonLogger,
+} from '@backstage/backend-common';
 import {
   coreServices,
   createServiceFactory,
-  loggerToWinstonLogger,
 } from '@backstage/backend-plugin-api';
 
 /** @public */

packages/backend-app-api/src/services/implementations/rootLifecycleService.ts

@@ -13,10 +13,10 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+import { loggerToWinstonLogger } from '@backstage/backend-common';
 import {
   createServiceFactory,
   coreServices,
-  loggerToWinstonLogger,
   LifecycleServiceShutdownHook,
   RootLifecycleService,
 } from '@backstage/backend-plugin-api';

packages/backend-app-api/src/services/implementations/tokenManagerService.ts

@@ -17,9 +17,11 @@
 import {
   coreServices,
   createServiceFactory,
-  loggerToWinstonLogger,
 } from '@backstage/backend-plugin-api';
-import { ServerTokenManager } from '@backstage/backend-common';
+import {
+  loggerToWinstonLogger,
+  ServerTokenManager,
+} from '@backstage/backend-common';
 
 /** @public */
 export const tokenManagerFactory = createServiceFactory({

packages/backend-app-api/src/services/implementations/urlReaderService.ts

@@ -14,11 +14,10 @@
  * limitations under the License.
  */
 
-import { UrlReaders } from '@backstage/backend-common';
+import { loggerToWinstonLogger, UrlReaders } from '@backstage/backend-common';
 import {
   coreServices,
   createServiceFactory,
-  loggerToWinstonLogger,
 } from '@backstage/backend-plugin-api';
 
 /** @public */

packages/backend-common/api-report.md

@@ -29,6 +29,7 @@ import { Knex } from 'knex';
 import { KubeConfig } from '@kubernetes/client-node';
 import { LoadConfigOptionsRemote } from '@backstage/config-loader';
 import { Logger } from 'winston';
+import { LoggerService } from '@backstage/backend-plugin-api';
 import { MergeResult } from 'isomorphic-git';
 import { DiscoveryService as PluginEndpointDiscovery } from '@backstage/backend-plugin-api';
 import { PushResult } from 'isomorphic-git';
@@ -46,6 +47,7 @@ import { SearchOptions } from '@backstage/backend-plugin-api';
 import { SearchResponse } from '@backstage/backend-plugin-api';
 import { SearchResponseFile } from '@backstage/backend-plugin-api';
 import { Server } from 'http';
+import { TransportStreamOptions } from 'winston-transport';
 import { UrlReaderService as UrlReader } from '@backstage/backend-plugin-api';
 import { V1PodTemplateSpec } from '@kubernetes/client-node';
 import * as winston from 'winston';
@@ -515,6 +517,12 @@ export function loadBackendConfig(options: {
   argv: string[];
 }): Promise<Config>;
 
+// @public (undocumented)
+export function loggerToWinstonLogger(
+  logger: LoggerService,
+  opts?: TransportStreamOptions,
+): Logger;
+
 // @public
 export function notFoundHandler(): RequestHandler;
 

packages/backend-common/package.json

@@ -83,6 +83,7 @@
     "tar": "^6.1.12",
     "uuid": "^8.3.2",
     "winston": "^3.2.1",
+    "winston-transport": "^4.5.0",
     "yauzl": "^2.10.0",
     "yn": "^4.0.0"
   },

packages/backend-common/src/logging/index.ts

@@ -22,3 +22,4 @@ export {
   redactWinstonLogLine,
 } from './rootLogger';
 export * from './voidLogger';
+export { loggerToWinstonLogger } from './loggerToWinstonLogger';

packages/backend-common/src/logging/loggerToWinstonLogger.ts

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import { LoggerService } from '../definitions';
+import { LoggerService } from '@backstage/backend-plugin-api';
 import { Logger as WinstonLogger, createLogger } from 'winston';
 import Transport, { TransportStreamOptions } from 'winston-transport';
 

packages/backend-plugin-api/api-report.md

@@ -7,14 +7,12 @@
 
 import { Config } from '@backstage/config';
 import { Handler } from 'express';
-import { Logger } from 'winston';
 import { PermissionEvaluator } from '@backstage/plugin-permission-common';
 import { PluginCacheManager } from '@backstage/backend-common';
 import { PluginDatabaseManager } from '@backstage/backend-common';
 import { PluginTaskScheduler } from '@backstage/backend-tasks';
 import { Readable } from 'stream';
 import { TokenManager } from '@backstage/backend-common';
-import { TransportStreamOptions } from 'winston-transport';
 
 // @public (undocumented)
 export interface BackendFeature {
@@ -201,12 +199,6 @@ export interface LoggerService {
   warn(message: string, meta?: Error | LogMeta): void;
 }
 
-// @public (undocumented)
-export function loggerToWinstonLogger(
-  logger: LoggerService,
-  opts?: TransportStreamOptions,
-): Logger;
-
 // @public (undocumented)
 export type LogMeta = {
   [name: string]: unknown;

packages/backend-plugin-api/package.json

@@ -38,9 +38,7 @@
     "@backstage/config": "workspace:^",
     "@backstage/plugin-permission-common": "workspace:^",
     "@types/express": "^4.17.6",
-    "express": "^4.17.1",
-    "winston": "^3.2.1",
-    "winston-transport": "^4.5.0"
+    "express": "^4.17.1"
   },
   "devDependencies": {
     "@backstage/cli": "workspace:^"

packages/backend-plugin-api/src/services/helpers/index.ts

@@ -1,17 +0,0 @@
-/*
- * Copyright 2022 The Backstage Authors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-export { loggerToWinstonLogger } from './loggerToWinstonLogger';

packages/backend-plugin-api/src/services/index.ts

@@ -15,5 +15,4 @@
  */
 
 export * from './definitions';
-export * from './helpers';
 export * from './system';