@verdaccio/logger 9.0.0-next-9.30 → 9.0.0-next-9.32

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # @verdaccio/logger
 
+## 9.0.0-next-9.32
+
+### Patch Changes
+
+- 107c4d3: Avoid creating an unused stdout destination when pretty logging uses its own transport.
+- f3f8976: chore: replace \_\_dirname with import.meta.dirname
+- 89a7362: Prevent file logging failures from unexpectedly terminating the registry.
+  
+  In the 9.x asynchronous file logging path, an unsuccessful initial open could leave a
+  destination registered for process exit and cause `sonic boom is not ready yet`. Setup
+  now reports the original error and removes only the destination that failed to open;
+  healthy destinations continue to flush when the process exits.
+  
+  After a file opens successfully, write and reopen errors are reported as structured
+  JSON errors on standard error. Error handling remains active for repeated failures,
+  including synchronous reopen failures during log rotation. Logging can resume after
+  the destination becomes writable again, and configured redaction also applies to the
+  error reports. No configuration change is required.
+- 107c4d3: Honor the configured `sync` option for JSON logs written to stdout or files, including in production. Keep asynchronous writes as the default.
+- Updated dependencies [59f1350]
+  - @verdaccio/core@9.0.0-next-9.32
+
+## 9.0.0-next-9.31
+
+### Patch Changes
+
+- Updated dependencies [c2b5897]
+- Updated dependencies [68ab0d4]
+- Updated dependencies [c52b632]
+- Updated dependencies [7054084]
+- Updated dependencies [cf15239]
+  - @verdaccio/core@9.0.0-next-9.31
+
 ## 9.0.0-next-9.30
 
 ### Patch Changes

package/build/logger.js

@@ -3,6 +3,7 @@ const require_formatter = require("./formatter.js");
 const require_transport = require("./transport.js");
 let debug = require("debug");
 debug = require_runtime.__toESM(debug);
+let on_exit_leak_free = require("on-exit-leak-free");
 //#region src/logger.ts
 var debug$1 = (0, debug.default)("verdaccio:logger");
 function isProd() {
@@ -68,17 +69,82 @@ async function prepareSetup(options = DEFAULT_LOGGER_CONF, pino) {
 	if (loggerConfig.type === "file") {
 		debug$1("logging file enabled");
 		if (willUseTransport(loggerConfig.format)) return createLogger(loggerConfig, void 0, loggerConfig.format, pino);
-		const destination = pino.destination(loggerConfig.path);
+		const destination = pino.destination({
+			dest: loggerConfig.path,
+			sync: loggerConfig.sync ?? false,
+			minLength: 0
+		});
+		let errorLogger;
+		let reopening = false;
+		let reopenListeners = [];
+		const onReopenReady = () => {
+			reopening = false;
+			reopenListeners = [];
+		};
+		const reportedErrors = /* @__PURE__ */ new WeakSet();
+		const onRuntimeError = (error) => {
+			if (reportedErrors.has(error)) return;
+			reportedErrors.add(error);
+			for (const listener of reopenListeners) destination.removeListener("ready", listener);
+			destination.removeListener("ready", onReopenReady);
+			reopening = false;
+			reopenListeners = [];
+			(errorLogger ??= createLogger({
+				level: "error",
+				redact: loggerConfig.redact
+			}, pino.destination({
+				dest: 2,
+				sync: true,
+				minLength: 0
+			}), "json", pino)).error({
+				err: error,
+				path: loggerConfig.path
+			}, "file log destination error");
+		};
 		await new Promise((resolve, reject) => {
-			destination.once("ready", resolve);
-			destination.once("error", reject);
+			const onReady = () => {
+				destination.removeListener("error", onError);
+				destination.on("error", onRuntimeError);
+				resolve();
+			};
+			const onError = (error) => {
+				destination.removeListener("ready", onReady);
+				(0, on_exit_leak_free.unregister)(destination);
+				reject(error);
+			};
+			destination.once("ready", onReady);
+			destination.once("error", onError);
 		});
 		debug$1("file destination ready: %s", loggerConfig.path);
-		process.on("SIGUSR2", () => destination.reopen());
+		const reopen = destination.reopen.bind(destination);
+		destination.reopen = (file) => {
+			if (!reopening) {
+				reopening = true;
+				destination.once("ready", onReopenReady);
+			}
+			const previousListeners = new Set(destination.rawListeners("ready"));
+			let reopenError;
+			try {
+				reopen(file);
+			} catch (error) {
+				reopenError = error;
+			} finally {
+				reopenListeners.push(...destination.rawListeners("ready").filter((listener) => !previousListeners.has(listener)));
+			}
+			if (reopenError) onRuntimeError(reopenError);
+		};
+		process.on("SIGUSR2", () => {
+			if (!reopening) destination.reopen();
+		});
 		return createLogger(loggerConfig, destination, loggerConfig.format, pino);
 	}
 	debug$1("logging stdout enabled");
-	return createLogger(loggerConfig, pino.destination(1), loggerConfig.format, pino);
+	const destination = willUseTransport(loggerConfig.format) ? void 0 : pino.destination({
+		dest: 1,
+		sync: loggerConfig.sync ?? false,
+		minLength: 0
+	});
+	return createLogger(loggerConfig, destination, loggerConfig.format, pino);
 }
 //#endregion
 exports.createLogger = createLogger;

package/build/logger.js.map

@@ -1 +1 @@
-{"version":3,"file":"logger.js","names":[],"sources":["../src/logger.ts"],"sourcesContent":["// <reference types=\"node\" />\nimport buildDebug from 'debug';\nimport type { LoggerOptions } from 'pino';\n\nimport type { Logger, LoggerConfigItem, LoggerFormat } from '@verdaccio/types';\n\nimport { fillInMsgTemplate } from './formatter';\nimport { createPrettyTransport, isPrettyFormat } from './transport';\n\nconst debug = buildDebug('verdaccio:logger');\n\nfunction isProd() {\n  return process.env.NODE_ENV === 'production';\n}\n\nconst DEFAULT_LOG_FORMAT = isProd() ? 'json' : 'pretty';\ndebug('default log format: %s', DEFAULT_LOG_FORMAT);\n\nexport type LogPlugin = {\n  dest: string;\n  options?: any[];\n};\n\nexport function createLogger(\n  options: LoggerConfigItem = { level: 'http' },\n  destination: NodeJS.WritableStream | undefined,\n  format: LoggerFormat = DEFAULT_LOG_FORMAT,\n  pino\n): any {\n  debug('setup logger');\n  let pinoConfig: LoggerOptions = {\n    customLevels: {\n      http: 25,\n    },\n    level: options.level,\n    serializers: {\n      err: pino.stdSerializers.err,\n      req: pino.stdSerializers.req,\n      res: pino.stdSerializers.res,\n    },\n    redact: options.redact,\n  };\n\n  debug('has prettifier? %o', !isProd());\n  let logger;\n  // pretty logs are not allowed in production for performance reasons\n  if (isPrettyFormat(format) && isProd() === false) {\n    const transport = createPrettyTransport(pino, options, format);\n    logger = pino(pinoConfig, transport);\n  } else {\n    pinoConfig = {\n      ...pinoConfig,\n      // https://getpino.io/#/docs/api?id=hooks-object\n      hooks: {\n        logMethod(args: [obj: unknown, msg?: string, ...rest: unknown[]], method, _level) {\n          const [templateObject, message, ...otherArgs] = args;\n          const templateVars =\n            !!templateObject && typeof templateObject === 'object'\n              ? Object.getOwnPropertyNames(templateObject)\n              : [];\n          if (!message || !templateVars.length) return method.apply(this, args);\n          const hydratedMessage = fillInMsgTemplate(message, templateObject as any, false);\n          return method.apply(this, [templateObject, hydratedMessage, ...otherArgs] as any);\n        },\n      },\n    };\n    logger = pino(pinoConfig, destination);\n  }\n\n  if (process.env.DEBUG) {\n    logger.on('level-change', (lvl, val, prevLvl, prevVal, instance) => {\n      if (logger !== instance) {\n        return;\n      }\n      debug('%s (%d) was changed to %s (%d)', lvl, val, prevLvl, prevVal);\n    });\n  }\n\n  return logger;\n}\n\nconst DEFAULT_LOGGER_CONF: LoggerConfigItem = {\n  type: 'stdout',\n  format: 'pretty',\n  level: 'http',\n};\n\nexport type LoggerConfig = LoggerConfigItem;\n\nexport function willUseTransport(format: LoggerFormat | undefined): boolean {\n  const resolvedFormat = format ?? (isProd() ? 'json' : 'pretty');\n  return isPrettyFormat(resolvedFormat) && isProd() === false;\n}\n\nexport async function prepareSetup(\n  options: LoggerConfigItem = DEFAULT_LOGGER_CONF,\n  pino\n): Promise<Logger> {\n  let loggerConfig = options;\n  if (!loggerConfig?.level) {\n    loggerConfig = {\n      ...loggerConfig,\n      level: 'http',\n    };\n  }\n  if (loggerConfig.type === 'file') {\n    debug('logging file enabled');\n    // Pretty format uses a pino transport that creates its own destination stream,\n    // so no destination is needed here.\n    if (willUseTransport(loggerConfig.format)) {\n      return createLogger(loggerConfig, undefined, loggerConfig.format, pino);\n    }\n    // For file destinations (json format), wait for the fd to be ready\n    // so we fail fast on bad paths / permissions instead of losing early logs\n    const destination = pino.destination(loggerConfig.path);\n    await new Promise<void>((resolve, reject) => {\n      destination.once('ready', resolve);\n      destination.once('error', reject);\n    });\n    debug('file destination ready: %s', loggerConfig.path);\n    process.on('SIGUSR2', () => destination.reopen());\n    return createLogger(loggerConfig, destination, loggerConfig.format, pino);\n  }\n  debug('logging stdout enabled');\n  return createLogger(loggerConfig, pino.destination(1), loggerConfig.format, pino);\n}\n"],"mappings":";;;;;;AASA,IAAM,WAAA,GAAQ,MAAA,QAAA,CAAW,kBAAkB;AAE3C,SAAS,SAAS;CAChB,OAAA,QAAA,IAAA,aAAgC;AAClC;AAEA,IAAM,qBAAqB,OAAO,IAAI,SAAS;AAC/C,QAAM,0BAA0B,kBAAkB;AAOlD,SAAgB,aACd,UAA4B,EAAE,OAAO,OAAO,GAC5C,aACA,SAAuB,oBACvB,MACK;CACL,QAAM,cAAc;CACpB,IAAI,aAA4B;EAC9B,cAAc,EACZ,MAAM,GACR;EACA,OAAO,QAAQ;EACf,aAAa;GACX,KAAK,KAAK,eAAe;GACzB,KAAK,KAAK,eAAe;GACzB,KAAK,KAAK,eAAe;EAC3B;EACA,QAAQ,QAAQ;CAClB;CAEA,QAAM,sBAAsB,CAAC,OAAO,CAAC;CACrC,IAAI;CAEJ,IAAI,kBAAA,eAAe,MAAM,KAAK,OAAO,MAAM,OAAO;EAChD,MAAM,YAAY,kBAAA,sBAAsB,MAAM,SAAS,MAAM;EAC7D,SAAS,KAAK,YAAY,SAAS;CACrC,OAAO;EACL,aAAa;GACX,GAAG;GAEH,OAAO,EACL,UAAU,MAAwD,QAAQ,QAAQ;IAChF,MAAM,CAAC,gBAAgB,SAAS,GAAG,aAAa;IAChD,MAAM,eACJ,CAAC,CAAC,kBAAkB,OAAO,mBAAmB,WAC1C,OAAO,oBAAoB,cAAc,IACzC,CAAC;IACP,IAAI,CAAC,WAAW,CAAC,aAAa,QAAQ,OAAO,OAAO,MAAM,MAAM,IAAI;IACpE,MAAM,kBAAkB,kBAAA,kBAAkB,SAAS,gBAAuB,KAAK;IAC/E,OAAO,OAAO,MAAM,MAAM;KAAC;KAAgB;KAAiB,GAAG;IAAS,CAAQ;GAClF,EACF;EACF;EACA,SAAS,KAAK,YAAY,WAAW;CACvC;CAEA,IAAI,QAAQ,IAAI,OACd,OAAO,GAAG,iBAAiB,KAAK,KAAK,SAAS,SAAS,aAAa;EAClE,IAAI,WAAW,UACb;EAEF,QAAM,kCAAkC,KAAK,KAAK,SAAS,OAAO;CACpE,CAAC;CAGH,OAAO;AACT;AAEA,IAAM,sBAAwC;CAC5C,MAAM;CACN,QAAQ;CACR,OAAO;AACT;AAIA,SAAgB,iBAAiB,QAA2C;CAC1E,MAAM,iBAAiB,WAAW,OAAO,IAAI,SAAS;CACtD,OAAO,kBAAA,eAAe,cAAc,KAAK,OAAO,MAAM;AACxD;AAEA,eAAsB,aACpB,UAA4B,qBAC5B,MACiB;CACjB,IAAI,eAAe;CACnB,IAAI,CAAC,cAAc,OACjB,eAAe;EACb,GAAG;EACH,OAAO;CACT;CAEF,IAAI,aAAa,SAAS,QAAQ;EAChC,QAAM,sBAAsB;EAG5B,IAAI,iBAAiB,aAAa,MAAM,GACtC,OAAO,aAAa,cAAc,KAAA,GAAW,aAAa,QAAQ,IAAI;EAIxE,MAAM,cAAc,KAAK,YAAY,aAAa,IAAI;EACtD,MAAM,IAAI,SAAe,SAAS,WAAW;GAC3C,YAAY,KAAK,SAAS,OAAO;GACjC,YAAY,KAAK,SAAS,MAAM;EAClC,CAAC;EACD,QAAM,8BAA8B,aAAa,IAAI;EACrD,QAAQ,GAAG,iBAAiB,YAAY,OAAO,CAAC;EAChD,OAAO,aAAa,cAAc,aAAa,aAAa,QAAQ,IAAI;CAC1E;CACA,QAAM,wBAAwB;CAC9B,OAAO,aAAa,cAAc,KAAK,YAAY,CAAC,GAAG,aAAa,QAAQ,IAAI;AAClF"}
+{"version":3,"file":"logger.js","names":[],"sources":["../src/logger.ts"],"sourcesContent":["// <reference types=\"node\" />\nimport buildDebug from 'debug';\nimport { unregister as onExitUnregister } from 'on-exit-leak-free';\nimport type { LoggerOptions } from 'pino';\n\nimport type { Logger, LoggerConfigItem, LoggerFormat } from '@verdaccio/types';\n\nimport { fillInMsgTemplate } from './formatter';\nimport { createPrettyTransport, isPrettyFormat } from './transport';\n\nconst debug = buildDebug('verdaccio:logger');\n\nfunction isProd() {\n  return process.env.NODE_ENV === 'production';\n}\n\nconst DEFAULT_LOG_FORMAT = isProd() ? 'json' : 'pretty';\ndebug('default log format: %s', DEFAULT_LOG_FORMAT);\n\nexport type LogPlugin = {\n  dest: string;\n  options?: any[];\n};\n\nexport function createLogger(\n  options: LoggerConfigItem = { level: 'http' },\n  destination: NodeJS.WritableStream | undefined,\n  format: LoggerFormat = DEFAULT_LOG_FORMAT,\n  pino\n): any {\n  debug('setup logger');\n  let pinoConfig: LoggerOptions = {\n    customLevels: {\n      http: 25,\n    },\n    level: options.level,\n    serializers: {\n      err: pino.stdSerializers.err,\n      req: pino.stdSerializers.req,\n      res: pino.stdSerializers.res,\n    },\n    redact: options.redact,\n  };\n\n  debug('has prettifier? %o', !isProd());\n  let logger;\n  // pretty logs are not allowed in production for performance reasons\n  if (isPrettyFormat(format) && isProd() === false) {\n    const transport = createPrettyTransport(pino, options, format);\n    logger = pino(pinoConfig, transport);\n  } else {\n    pinoConfig = {\n      ...pinoConfig,\n      // https://getpino.io/#/docs/api?id=hooks-object\n      hooks: {\n        logMethod(args: [obj: unknown, msg?: string, ...rest: unknown[]], method, _level) {\n          const [templateObject, message, ...otherArgs] = args;\n          const templateVars =\n            !!templateObject && typeof templateObject === 'object'\n              ? Object.getOwnPropertyNames(templateObject)\n              : [];\n          if (!message || !templateVars.length) return method.apply(this, args);\n          const hydratedMessage = fillInMsgTemplate(message, templateObject as any, false);\n          return method.apply(this, [templateObject, hydratedMessage, ...otherArgs] as any);\n        },\n      },\n    };\n    logger = pino(pinoConfig, destination);\n  }\n\n  if (process.env.DEBUG) {\n    logger.on('level-change', (lvl, val, prevLvl, prevVal, instance) => {\n      if (logger !== instance) {\n        return;\n      }\n      debug('%s (%d) was changed to %s (%d)', lvl, val, prevLvl, prevVal);\n    });\n  }\n\n  return logger;\n}\n\nconst DEFAULT_LOGGER_CONF: LoggerConfigItem = {\n  type: 'stdout',\n  format: 'pretty',\n  level: 'http',\n};\n\nexport type LoggerConfig = LoggerConfigItem;\n\nexport function willUseTransport(format: LoggerFormat | undefined): boolean {\n  const resolvedFormat = format ?? (isProd() ? 'json' : 'pretty');\n  return isPrettyFormat(resolvedFormat) && isProd() === false;\n}\n\nexport async function prepareSetup(\n  options: LoggerConfigItem = DEFAULT_LOGGER_CONF,\n  pino\n): Promise<Logger> {\n  let loggerConfig = options;\n  if (!loggerConfig?.level) {\n    loggerConfig = {\n      ...loggerConfig,\n      level: 'http',\n    };\n  }\n  if (loggerConfig.type === 'file') {\n    debug('logging file enabled');\n    // Pretty format uses a pino transport that creates its own destination stream,\n    // so no destination is needed here.\n    if (willUseTransport(loggerConfig.format)) {\n      return createLogger(loggerConfig, undefined, loggerConfig.format, pino);\n    }\n    // For file destinations (json format), wait for the fd to be ready\n    // so we fail fast on bad paths / permissions instead of losing early logs\n    const destination = pino.destination({\n      dest: loggerConfig.path,\n      sync: loggerConfig.sync ?? false,\n      minLength: 0,\n    });\n    let errorLogger: Logger | undefined;\n    let reopening = false;\n    let reopenListeners: (() => void)[] = [];\n    const onReopenReady = () => {\n      reopening = false;\n      reopenListeners = [];\n    };\n    const reportedErrors = new WeakSet<Error>();\n    const onRuntimeError = (error: Error) => {\n      // Pino re-emits errors; synchronous reopen failures also throw the same error.\n      if (reportedErrors.has(error)) return;\n      reportedErrors.add(error);\n      // Failed reopens otherwise leave callbacks that close the old fd again on recovery.\n      for (const listener of reopenListeners) destination.removeListener('ready', listener);\n      destination.removeListener('ready', onReopenReady);\n      reopening = false;\n      reopenListeners = [];\n      const reporter: Logger = (errorLogger ??= createLogger(\n        { level: 'error', redact: loggerConfig.redact },\n        pino.destination({ dest: 2, sync: true, minLength: 0 }),\n        'json',\n        pino\n      ));\n      reporter.error({ err: error, path: loggerConfig.path }, 'file log destination error');\n    };\n    await new Promise<void>((resolve, reject) => {\n      const onReady = () => {\n        destination.removeListener('error', onError);\n        destination.on('error', onRuntimeError);\n        resolve();\n      };\n      const onError = (error: Error) => {\n        destination.removeListener('ready', onReady);\n        // A failed open leaves fd=-1, so Pino's exit flush would throw.\n        // SonicBoom.destroy() waits for ready here; unregister the failed stream directly.\n        onExitUnregister(destination);\n        reject(error);\n      };\n      destination.once('ready', onReady);\n      destination.once('error', onError);\n    });\n    debug('file destination ready: %s', loggerConfig.path);\n    const reopen = destination.reopen.bind(destination);\n    // SonicBoom can defer reopening until a pending write finishes.\n    destination.reopen = (file?: string) => {\n      if (!reopening) {\n        reopening = true;\n        destination.once('ready', onReopenReady);\n      }\n      const previousListeners = new Set(destination.rawListeners('ready'));\n      let reopenError: Error | undefined;\n      try {\n        reopen(file);\n      } catch (error) {\n        reopenError = error as Error;\n      } finally {\n        reopenListeners.push(\n          ...destination\n            .rawListeners('ready')\n            .filter((listener) => !previousListeners.has(listener))\n        );\n      }\n      if (reopenError) onRuntimeError(reopenError);\n    };\n    process.on('SIGUSR2', () => {\n      if (!reopening) destination.reopen();\n    });\n    return createLogger(loggerConfig, destination, loggerConfig.format, pino);\n  }\n  debug('logging stdout enabled');\n  const destination = willUseTransport(loggerConfig.format)\n    ? undefined\n    : pino.destination({ dest: 1, sync: loggerConfig.sync ?? false, minLength: 0 });\n  return createLogger(loggerConfig, destination, loggerConfig.format, pino);\n}\n"],"mappings":";;;;;;;AAUA,IAAM,WAAA,GAAQ,MAAA,QAAA,CAAW,kBAAkB;AAE3C,SAAS,SAAS;CAChB,OAAA,QAAA,IAAA,aAAgC;AAClC;AAEA,IAAM,qBAAqB,OAAO,IAAI,SAAS;AAC/C,QAAM,0BAA0B,kBAAkB;AAOlD,SAAgB,aACd,UAA4B,EAAE,OAAO,OAAO,GAC5C,aACA,SAAuB,oBACvB,MACK;CACL,QAAM,cAAc;CACpB,IAAI,aAA4B;EAC9B,cAAc,EACZ,MAAM,GACR;EACA,OAAO,QAAQ;EACf,aAAa;GACX,KAAK,KAAK,eAAe;GACzB,KAAK,KAAK,eAAe;GACzB,KAAK,KAAK,eAAe;EAC3B;EACA,QAAQ,QAAQ;CAClB;CAEA,QAAM,sBAAsB,CAAC,OAAO,CAAC;CACrC,IAAI;CAEJ,IAAI,kBAAA,eAAe,MAAM,KAAK,OAAO,MAAM,OAAO;EAChD,MAAM,YAAY,kBAAA,sBAAsB,MAAM,SAAS,MAAM;EAC7D,SAAS,KAAK,YAAY,SAAS;CACrC,OAAO;EACL,aAAa;GACX,GAAG;GAEH,OAAO,EACL,UAAU,MAAwD,QAAQ,QAAQ;IAChF,MAAM,CAAC,gBAAgB,SAAS,GAAG,aAAa;IAChD,MAAM,eACJ,CAAC,CAAC,kBAAkB,OAAO,mBAAmB,WAC1C,OAAO,oBAAoB,cAAc,IACzC,CAAC;IACP,IAAI,CAAC,WAAW,CAAC,aAAa,QAAQ,OAAO,OAAO,MAAM,MAAM,IAAI;IACpE,MAAM,kBAAkB,kBAAA,kBAAkB,SAAS,gBAAuB,KAAK;IAC/E,OAAO,OAAO,MAAM,MAAM;KAAC;KAAgB;KAAiB,GAAG;IAAS,CAAQ;GAClF,EACF;EACF;EACA,SAAS,KAAK,YAAY,WAAW;CACvC;CAEA,IAAI,QAAQ,IAAI,OACd,OAAO,GAAG,iBAAiB,KAAK,KAAK,SAAS,SAAS,aAAa;EAClE,IAAI,WAAW,UACb;EAEF,QAAM,kCAAkC,KAAK,KAAK,SAAS,OAAO;CACpE,CAAC;CAGH,OAAO;AACT;AAEA,IAAM,sBAAwC;CAC5C,MAAM;CACN,QAAQ;CACR,OAAO;AACT;AAIA,SAAgB,iBAAiB,QAA2C;CAC1E,MAAM,iBAAiB,WAAW,OAAO,IAAI,SAAS;CACtD,OAAO,kBAAA,eAAe,cAAc,KAAK,OAAO,MAAM;AACxD;AAEA,eAAsB,aACpB,UAA4B,qBAC5B,MACiB;CACjB,IAAI,eAAe;CACnB,IAAI,CAAC,cAAc,OACjB,eAAe;EACb,GAAG;EACH,OAAO;CACT;CAEF,IAAI,aAAa,SAAS,QAAQ;EAChC,QAAM,sBAAsB;EAG5B,IAAI,iBAAiB,aAAa,MAAM,GACtC,OAAO,aAAa,cAAc,KAAA,GAAW,aAAa,QAAQ,IAAI;EAIxE,MAAM,cAAc,KAAK,YAAY;GACnC,MAAM,aAAa;GACnB,MAAM,aAAa,QAAQ;GAC3B,WAAW;EACb,CAAC;EACD,IAAI;EACJ,IAAI,YAAY;EAChB,IAAI,kBAAkC,CAAC;EACvC,MAAM,sBAAsB;GAC1B,YAAY;GACZ,kBAAkB,CAAC;EACrB;EACA,MAAM,iCAAiB,IAAI,QAAe;EAC1C,MAAM,kBAAkB,UAAiB;GAEvC,IAAI,eAAe,IAAI,KAAK,GAAG;GAC/B,eAAe,IAAI,KAAK;GAExB,KAAK,MAAM,YAAY,iBAAiB,YAAY,eAAe,SAAS,QAAQ;GACpF,YAAY,eAAe,SAAS,aAAa;GACjD,YAAY;GACZ,kBAAkB,CAAC;GAOnB,CAN0B,gBAAgB,aACxC;IAAE,OAAO;IAAS,QAAQ,aAAa;GAAO,GAC9C,KAAK,YAAY;IAAE,MAAM;IAAG,MAAM;IAAM,WAAW;GAAE,CAAC,GACtD,QACA,IACF,EAAA,CACS,MAAM;IAAE,KAAK;IAAO,MAAM,aAAa;GAAK,GAAG,4BAA4B;EACtF;EACA,MAAM,IAAI,SAAe,SAAS,WAAW;GAC3C,MAAM,gBAAgB;IACpB,YAAY,eAAe,SAAS,OAAO;IAC3C,YAAY,GAAG,SAAS,cAAc;IACtC,QAAQ;GACV;GACA,MAAM,WAAW,UAAiB;IAChC,YAAY,eAAe,SAAS,OAAO;IAG3C,CAAA,GAAA,kBAAA,WAAA,CAAiB,WAAW;IAC5B,OAAO,KAAK;GACd;GACA,YAAY,KAAK,SAAS,OAAO;GACjC,YAAY,KAAK,SAAS,OAAO;EACnC,CAAC;EACD,QAAM,8BAA8B,aAAa,IAAI;EACrD,MAAM,SAAS,YAAY,OAAO,KAAK,WAAW;EAElD,YAAY,UAAU,SAAkB;GACtC,IAAI,CAAC,WAAW;IACd,YAAY;IACZ,YAAY,KAAK,SAAS,aAAa;GACzC;GACA,MAAM,oBAAoB,IAAI,IAAI,YAAY,aAAa,OAAO,CAAC;GACnE,IAAI;GACJ,IAAI;IACF,OAAO,IAAI;GACb,SAAS,OAAO;IACd,cAAc;GAChB,UAAU;IACR,gBAAgB,KACd,GAAG,YACA,aAAa,OAAO,CAAC,CACrB,QAAQ,aAAa,CAAC,kBAAkB,IAAI,QAAQ,CAAC,CAC1D;GACF;GACA,IAAI,aAAa,eAAe,WAAW;EAC7C;EACA,QAAQ,GAAG,iBAAiB;GAC1B,IAAI,CAAC,WAAW,YAAY,OAAO;EACrC,CAAC;EACD,OAAO,aAAa,cAAc,aAAa,aAAa,QAAQ,IAAI;CAC1E;CACA,QAAM,wBAAwB;CAC9B,MAAM,cAAc,iBAAiB,aAAa,MAAM,IACpD,KAAA,IACA,KAAK,YAAY;EAAE,MAAM;EAAG,MAAM,aAAa,QAAQ;EAAO,WAAW;CAAE,CAAC;CAChF,OAAO,aAAa,cAAc,aAAa,aAAa,QAAQ,IAAI;AAC1E"}

package/build/logger.mjs

@@ -1,6 +1,7 @@
 import { fillInMsgTemplate } from "./formatter.mjs";
 import { createPrettyTransport, isPrettyFormat } from "./transport.mjs";
 import buildDebug from "debug";
+import { unregister } from "on-exit-leak-free";
 //#region src/logger.ts
 var debug = buildDebug("verdaccio:logger");
 function isProd() {
@@ -66,17 +67,82 @@ async function prepareSetup(options = DEFAULT_LOGGER_CONF, pino) {
 	if (loggerConfig.type === "file") {
 		debug("logging file enabled");
 		if (willUseTransport(loggerConfig.format)) return createLogger(loggerConfig, void 0, loggerConfig.format, pino);
-		const destination = pino.destination(loggerConfig.path);
+		const destination = pino.destination({
+			dest: loggerConfig.path,
+			sync: loggerConfig.sync ?? false,
+			minLength: 0
+		});
+		let errorLogger;
+		let reopening = false;
+		let reopenListeners = [];
+		const onReopenReady = () => {
+			reopening = false;
+			reopenListeners = [];
+		};
+		const reportedErrors = /* @__PURE__ */ new WeakSet();
+		const onRuntimeError = (error) => {
+			if (reportedErrors.has(error)) return;
+			reportedErrors.add(error);
+			for (const listener of reopenListeners) destination.removeListener("ready", listener);
+			destination.removeListener("ready", onReopenReady);
+			reopening = false;
+			reopenListeners = [];
+			(errorLogger ??= createLogger({
+				level: "error",
+				redact: loggerConfig.redact
+			}, pino.destination({
+				dest: 2,
+				sync: true,
+				minLength: 0
+			}), "json", pino)).error({
+				err: error,
+				path: loggerConfig.path
+			}, "file log destination error");
+		};
 		await new Promise((resolve, reject) => {
-			destination.once("ready", resolve);
-			destination.once("error", reject);
+			const onReady = () => {
+				destination.removeListener("error", onError);
+				destination.on("error", onRuntimeError);
+				resolve();
+			};
+			const onError = (error) => {
+				destination.removeListener("ready", onReady);
+				unregister(destination);
+				reject(error);
+			};
+			destination.once("ready", onReady);
+			destination.once("error", onError);
 		});
 		debug("file destination ready: %s", loggerConfig.path);
-		process.on("SIGUSR2", () => destination.reopen());
+		const reopen = destination.reopen.bind(destination);
+		destination.reopen = (file) => {
+			if (!reopening) {
+				reopening = true;
+				destination.once("ready", onReopenReady);
+			}
+			const previousListeners = new Set(destination.rawListeners("ready"));
+			let reopenError;
+			try {
+				reopen(file);
+			} catch (error) {
+				reopenError = error;
+			} finally {
+				reopenListeners.push(...destination.rawListeners("ready").filter((listener) => !previousListeners.has(listener)));
+			}
+			if (reopenError) onRuntimeError(reopenError);
+		};
+		process.on("SIGUSR2", () => {
+			if (!reopening) destination.reopen();
+		});
 		return createLogger(loggerConfig, destination, loggerConfig.format, pino);
 	}
 	debug("logging stdout enabled");
-	return createLogger(loggerConfig, pino.destination(1), loggerConfig.format, pino);
+	const destination = willUseTransport(loggerConfig.format) ? void 0 : pino.destination({
+		dest: 1,
+		sync: loggerConfig.sync ?? false,
+		minLength: 0
+	});
+	return createLogger(loggerConfig, destination, loggerConfig.format, pino);
 }
 //#endregion
 export { createLogger, prepareSetup, willUseTransport };

package/build/logger.mjs.map

@@ -1 +1 @@
-{"version":3,"file":"logger.mjs","names":[],"sources":["../src/logger.ts"],"sourcesContent":["// <reference types=\"node\" />\nimport buildDebug from 'debug';\nimport type { LoggerOptions } from 'pino';\n\nimport type { Logger, LoggerConfigItem, LoggerFormat } from '@verdaccio/types';\n\nimport { fillInMsgTemplate } from './formatter';\nimport { createPrettyTransport, isPrettyFormat } from './transport';\n\nconst debug = buildDebug('verdaccio:logger');\n\nfunction isProd() {\n  return process.env.NODE_ENV === 'production';\n}\n\nconst DEFAULT_LOG_FORMAT = isProd() ? 'json' : 'pretty';\ndebug('default log format: %s', DEFAULT_LOG_FORMAT);\n\nexport type LogPlugin = {\n  dest: string;\n  options?: any[];\n};\n\nexport function createLogger(\n  options: LoggerConfigItem = { level: 'http' },\n  destination: NodeJS.WritableStream | undefined,\n  format: LoggerFormat = DEFAULT_LOG_FORMAT,\n  pino\n): any {\n  debug('setup logger');\n  let pinoConfig: LoggerOptions = {\n    customLevels: {\n      http: 25,\n    },\n    level: options.level,\n    serializers: {\n      err: pino.stdSerializers.err,\n      req: pino.stdSerializers.req,\n      res: pino.stdSerializers.res,\n    },\n    redact: options.redact,\n  };\n\n  debug('has prettifier? %o', !isProd());\n  let logger;\n  // pretty logs are not allowed in production for performance reasons\n  if (isPrettyFormat(format) && isProd() === false) {\n    const transport = createPrettyTransport(pino, options, format);\n    logger = pino(pinoConfig, transport);\n  } else {\n    pinoConfig = {\n      ...pinoConfig,\n      // https://getpino.io/#/docs/api?id=hooks-object\n      hooks: {\n        logMethod(args: [obj: unknown, msg?: string, ...rest: unknown[]], method, _level) {\n          const [templateObject, message, ...otherArgs] = args;\n          const templateVars =\n            !!templateObject && typeof templateObject === 'object'\n              ? Object.getOwnPropertyNames(templateObject)\n              : [];\n          if (!message || !templateVars.length) return method.apply(this, args);\n          const hydratedMessage = fillInMsgTemplate(message, templateObject as any, false);\n          return method.apply(this, [templateObject, hydratedMessage, ...otherArgs] as any);\n        },\n      },\n    };\n    logger = pino(pinoConfig, destination);\n  }\n\n  if (process.env.DEBUG) {\n    logger.on('level-change', (lvl, val, prevLvl, prevVal, instance) => {\n      if (logger !== instance) {\n        return;\n      }\n      debug('%s (%d) was changed to %s (%d)', lvl, val, prevLvl, prevVal);\n    });\n  }\n\n  return logger;\n}\n\nconst DEFAULT_LOGGER_CONF: LoggerConfigItem = {\n  type: 'stdout',\n  format: 'pretty',\n  level: 'http',\n};\n\nexport type LoggerConfig = LoggerConfigItem;\n\nexport function willUseTransport(format: LoggerFormat | undefined): boolean {\n  const resolvedFormat = format ?? (isProd() ? 'json' : 'pretty');\n  return isPrettyFormat(resolvedFormat) && isProd() === false;\n}\n\nexport async function prepareSetup(\n  options: LoggerConfigItem = DEFAULT_LOGGER_CONF,\n  pino\n): Promise<Logger> {\n  let loggerConfig = options;\n  if (!loggerConfig?.level) {\n    loggerConfig = {\n      ...loggerConfig,\n      level: 'http',\n    };\n  }\n  if (loggerConfig.type === 'file') {\n    debug('logging file enabled');\n    // Pretty format uses a pino transport that creates its own destination stream,\n    // so no destination is needed here.\n    if (willUseTransport(loggerConfig.format)) {\n      return createLogger(loggerConfig, undefined, loggerConfig.format, pino);\n    }\n    // For file destinations (json format), wait for the fd to be ready\n    // so we fail fast on bad paths / permissions instead of losing early logs\n    const destination = pino.destination(loggerConfig.path);\n    await new Promise<void>((resolve, reject) => {\n      destination.once('ready', resolve);\n      destination.once('error', reject);\n    });\n    debug('file destination ready: %s', loggerConfig.path);\n    process.on('SIGUSR2', () => destination.reopen());\n    return createLogger(loggerConfig, destination, loggerConfig.format, pino);\n  }\n  debug('logging stdout enabled');\n  return createLogger(loggerConfig, pino.destination(1), loggerConfig.format, pino);\n}\n"],"mappings":";;;;AASA,IAAM,QAAQ,WAAW,kBAAkB;AAE3C,SAAS,SAAS;CAChB,OAAA,QAAA,IAAA,aAAgC;AAClC;AAEA,IAAM,qBAAqB,OAAO,IAAI,SAAS;AAC/C,MAAM,0BAA0B,kBAAkB;AAOlD,SAAgB,aACd,UAA4B,EAAE,OAAO,OAAO,GAC5C,aACA,SAAuB,oBACvB,MACK;CACL,MAAM,cAAc;CACpB,IAAI,aAA4B;EAC9B,cAAc,EACZ,MAAM,GACR;EACA,OAAO,QAAQ;EACf,aAAa;GACX,KAAK,KAAK,eAAe;GACzB,KAAK,KAAK,eAAe;GACzB,KAAK,KAAK,eAAe;EAC3B;EACA,QAAQ,QAAQ;CAClB;CAEA,MAAM,sBAAsB,CAAC,OAAO,CAAC;CACrC,IAAI;CAEJ,IAAI,eAAe,MAAM,KAAK,OAAO,MAAM,OAAO;EAChD,MAAM,YAAY,sBAAsB,MAAM,SAAS,MAAM;EAC7D,SAAS,KAAK,YAAY,SAAS;CACrC,OAAO;EACL,aAAa;GACX,GAAG;GAEH,OAAO,EACL,UAAU,MAAwD,QAAQ,QAAQ;IAChF,MAAM,CAAC,gBAAgB,SAAS,GAAG,aAAa;IAChD,MAAM,eACJ,CAAC,CAAC,kBAAkB,OAAO,mBAAmB,WAC1C,OAAO,oBAAoB,cAAc,IACzC,CAAC;IACP,IAAI,CAAC,WAAW,CAAC,aAAa,QAAQ,OAAO,OAAO,MAAM,MAAM,IAAI;IACpE,MAAM,kBAAkB,kBAAkB,SAAS,gBAAuB,KAAK;IAC/E,OAAO,OAAO,MAAM,MAAM;KAAC;KAAgB;KAAiB,GAAG;IAAS,CAAQ;GAClF,EACF;EACF;EACA,SAAS,KAAK,YAAY,WAAW;CACvC;CAEA,IAAI,QAAQ,IAAI,OACd,OAAO,GAAG,iBAAiB,KAAK,KAAK,SAAS,SAAS,aAAa;EAClE,IAAI,WAAW,UACb;EAEF,MAAM,kCAAkC,KAAK,KAAK,SAAS,OAAO;CACpE,CAAC;CAGH,OAAO;AACT;AAEA,IAAM,sBAAwC;CAC5C,MAAM;CACN,QAAQ;CACR,OAAO;AACT;AAIA,SAAgB,iBAAiB,QAA2C;CAC1E,MAAM,iBAAiB,WAAW,OAAO,IAAI,SAAS;CACtD,OAAO,eAAe,cAAc,KAAK,OAAO,MAAM;AACxD;AAEA,eAAsB,aACpB,UAA4B,qBAC5B,MACiB;CACjB,IAAI,eAAe;CACnB,IAAI,CAAC,cAAc,OACjB,eAAe;EACb,GAAG;EACH,OAAO;CACT;CAEF,IAAI,aAAa,SAAS,QAAQ;EAChC,MAAM,sBAAsB;EAG5B,IAAI,iBAAiB,aAAa,MAAM,GACtC,OAAO,aAAa,cAAc,KAAA,GAAW,aAAa,QAAQ,IAAI;EAIxE,MAAM,cAAc,KAAK,YAAY,aAAa,IAAI;EACtD,MAAM,IAAI,SAAe,SAAS,WAAW;GAC3C,YAAY,KAAK,SAAS,OAAO;GACjC,YAAY,KAAK,SAAS,MAAM;EAClC,CAAC;EACD,MAAM,8BAA8B,aAAa,IAAI;EACrD,QAAQ,GAAG,iBAAiB,YAAY,OAAO,CAAC;EAChD,OAAO,aAAa,cAAc,aAAa,aAAa,QAAQ,IAAI;CAC1E;CACA,MAAM,wBAAwB;CAC9B,OAAO,aAAa,cAAc,KAAK,YAAY,CAAC,GAAG,aAAa,QAAQ,IAAI;AAClF"}
+{"version":3,"file":"logger.mjs","names":[],"sources":["../src/logger.ts"],"sourcesContent":["// <reference types=\"node\" />\nimport buildDebug from 'debug';\nimport { unregister as onExitUnregister } from 'on-exit-leak-free';\nimport type { LoggerOptions } from 'pino';\n\nimport type { Logger, LoggerConfigItem, LoggerFormat } from '@verdaccio/types';\n\nimport { fillInMsgTemplate } from './formatter';\nimport { createPrettyTransport, isPrettyFormat } from './transport';\n\nconst debug = buildDebug('verdaccio:logger');\n\nfunction isProd() {\n  return process.env.NODE_ENV === 'production';\n}\n\nconst DEFAULT_LOG_FORMAT = isProd() ? 'json' : 'pretty';\ndebug('default log format: %s', DEFAULT_LOG_FORMAT);\n\nexport type LogPlugin = {\n  dest: string;\n  options?: any[];\n};\n\nexport function createLogger(\n  options: LoggerConfigItem = { level: 'http' },\n  destination: NodeJS.WritableStream | undefined,\n  format: LoggerFormat = DEFAULT_LOG_FORMAT,\n  pino\n): any {\n  debug('setup logger');\n  let pinoConfig: LoggerOptions = {\n    customLevels: {\n      http: 25,\n    },\n    level: options.level,\n    serializers: {\n      err: pino.stdSerializers.err,\n      req: pino.stdSerializers.req,\n      res: pino.stdSerializers.res,\n    },\n    redact: options.redact,\n  };\n\n  debug('has prettifier? %o', !isProd());\n  let logger;\n  // pretty logs are not allowed in production for performance reasons\n  if (isPrettyFormat(format) && isProd() === false) {\n    const transport = createPrettyTransport(pino, options, format);\n    logger = pino(pinoConfig, transport);\n  } else {\n    pinoConfig = {\n      ...pinoConfig,\n      // https://getpino.io/#/docs/api?id=hooks-object\n      hooks: {\n        logMethod(args: [obj: unknown, msg?: string, ...rest: unknown[]], method, _level) {\n          const [templateObject, message, ...otherArgs] = args;\n          const templateVars =\n            !!templateObject && typeof templateObject === 'object'\n              ? Object.getOwnPropertyNames(templateObject)\n              : [];\n          if (!message || !templateVars.length) return method.apply(this, args);\n          const hydratedMessage = fillInMsgTemplate(message, templateObject as any, false);\n          return method.apply(this, [templateObject, hydratedMessage, ...otherArgs] as any);\n        },\n      },\n    };\n    logger = pino(pinoConfig, destination);\n  }\n\n  if (process.env.DEBUG) {\n    logger.on('level-change', (lvl, val, prevLvl, prevVal, instance) => {\n      if (logger !== instance) {\n        return;\n      }\n      debug('%s (%d) was changed to %s (%d)', lvl, val, prevLvl, prevVal);\n    });\n  }\n\n  return logger;\n}\n\nconst DEFAULT_LOGGER_CONF: LoggerConfigItem = {\n  type: 'stdout',\n  format: 'pretty',\n  level: 'http',\n};\n\nexport type LoggerConfig = LoggerConfigItem;\n\nexport function willUseTransport(format: LoggerFormat | undefined): boolean {\n  const resolvedFormat = format ?? (isProd() ? 'json' : 'pretty');\n  return isPrettyFormat(resolvedFormat) && isProd() === false;\n}\n\nexport async function prepareSetup(\n  options: LoggerConfigItem = DEFAULT_LOGGER_CONF,\n  pino\n): Promise<Logger> {\n  let loggerConfig = options;\n  if (!loggerConfig?.level) {\n    loggerConfig = {\n      ...loggerConfig,\n      level: 'http',\n    };\n  }\n  if (loggerConfig.type === 'file') {\n    debug('logging file enabled');\n    // Pretty format uses a pino transport that creates its own destination stream,\n    // so no destination is needed here.\n    if (willUseTransport(loggerConfig.format)) {\n      return createLogger(loggerConfig, undefined, loggerConfig.format, pino);\n    }\n    // For file destinations (json format), wait for the fd to be ready\n    // so we fail fast on bad paths / permissions instead of losing early logs\n    const destination = pino.destination({\n      dest: loggerConfig.path,\n      sync: loggerConfig.sync ?? false,\n      minLength: 0,\n    });\n    let errorLogger: Logger | undefined;\n    let reopening = false;\n    let reopenListeners: (() => void)[] = [];\n    const onReopenReady = () => {\n      reopening = false;\n      reopenListeners = [];\n    };\n    const reportedErrors = new WeakSet<Error>();\n    const onRuntimeError = (error: Error) => {\n      // Pino re-emits errors; synchronous reopen failures also throw the same error.\n      if (reportedErrors.has(error)) return;\n      reportedErrors.add(error);\n      // Failed reopens otherwise leave callbacks that close the old fd again on recovery.\n      for (const listener of reopenListeners) destination.removeListener('ready', listener);\n      destination.removeListener('ready', onReopenReady);\n      reopening = false;\n      reopenListeners = [];\n      const reporter: Logger = (errorLogger ??= createLogger(\n        { level: 'error', redact: loggerConfig.redact },\n        pino.destination({ dest: 2, sync: true, minLength: 0 }),\n        'json',\n        pino\n      ));\n      reporter.error({ err: error, path: loggerConfig.path }, 'file log destination error');\n    };\n    await new Promise<void>((resolve, reject) => {\n      const onReady = () => {\n        destination.removeListener('error', onError);\n        destination.on('error', onRuntimeError);\n        resolve();\n      };\n      const onError = (error: Error) => {\n        destination.removeListener('ready', onReady);\n        // A failed open leaves fd=-1, so Pino's exit flush would throw.\n        // SonicBoom.destroy() waits for ready here; unregister the failed stream directly.\n        onExitUnregister(destination);\n        reject(error);\n      };\n      destination.once('ready', onReady);\n      destination.once('error', onError);\n    });\n    debug('file destination ready: %s', loggerConfig.path);\n    const reopen = destination.reopen.bind(destination);\n    // SonicBoom can defer reopening until a pending write finishes.\n    destination.reopen = (file?: string) => {\n      if (!reopening) {\n        reopening = true;\n        destination.once('ready', onReopenReady);\n      }\n      const previousListeners = new Set(destination.rawListeners('ready'));\n      let reopenError: Error | undefined;\n      try {\n        reopen(file);\n      } catch (error) {\n        reopenError = error as Error;\n      } finally {\n        reopenListeners.push(\n          ...destination\n            .rawListeners('ready')\n            .filter((listener) => !previousListeners.has(listener))\n        );\n      }\n      if (reopenError) onRuntimeError(reopenError);\n    };\n    process.on('SIGUSR2', () => {\n      if (!reopening) destination.reopen();\n    });\n    return createLogger(loggerConfig, destination, loggerConfig.format, pino);\n  }\n  debug('logging stdout enabled');\n  const destination = willUseTransport(loggerConfig.format)\n    ? undefined\n    : pino.destination({ dest: 1, sync: loggerConfig.sync ?? false, minLength: 0 });\n  return createLogger(loggerConfig, destination, loggerConfig.format, pino);\n}\n"],"mappings":";;;;;AAUA,IAAM,QAAQ,WAAW,kBAAkB;AAE3C,SAAS,SAAS;CAChB,OAAA,QAAA,IAAA,aAAgC;AAClC;AAEA,IAAM,qBAAqB,OAAO,IAAI,SAAS;AAC/C,MAAM,0BAA0B,kBAAkB;AAOlD,SAAgB,aACd,UAA4B,EAAE,OAAO,OAAO,GAC5C,aACA,SAAuB,oBACvB,MACK;CACL,MAAM,cAAc;CACpB,IAAI,aAA4B;EAC9B,cAAc,EACZ,MAAM,GACR;EACA,OAAO,QAAQ;EACf,aAAa;GACX,KAAK,KAAK,eAAe;GACzB,KAAK,KAAK,eAAe;GACzB,KAAK,KAAK,eAAe;EAC3B;EACA,QAAQ,QAAQ;CAClB;CAEA,MAAM,sBAAsB,CAAC,OAAO,CAAC;CACrC,IAAI;CAEJ,IAAI,eAAe,MAAM,KAAK,OAAO,MAAM,OAAO;EAChD,MAAM,YAAY,sBAAsB,MAAM,SAAS,MAAM;EAC7D,SAAS,KAAK,YAAY,SAAS;CACrC,OAAO;EACL,aAAa;GACX,GAAG;GAEH,OAAO,EACL,UAAU,MAAwD,QAAQ,QAAQ;IAChF,MAAM,CAAC,gBAAgB,SAAS,GAAG,aAAa;IAChD,MAAM,eACJ,CAAC,CAAC,kBAAkB,OAAO,mBAAmB,WAC1C,OAAO,oBAAoB,cAAc,IACzC,CAAC;IACP,IAAI,CAAC,WAAW,CAAC,aAAa,QAAQ,OAAO,OAAO,MAAM,MAAM,IAAI;IACpE,MAAM,kBAAkB,kBAAkB,SAAS,gBAAuB,KAAK;IAC/E,OAAO,OAAO,MAAM,MAAM;KAAC;KAAgB;KAAiB,GAAG;IAAS,CAAQ;GAClF,EACF;EACF;EACA,SAAS,KAAK,YAAY,WAAW;CACvC;CAEA,IAAI,QAAQ,IAAI,OACd,OAAO,GAAG,iBAAiB,KAAK,KAAK,SAAS,SAAS,aAAa;EAClE,IAAI,WAAW,UACb;EAEF,MAAM,kCAAkC,KAAK,KAAK,SAAS,OAAO;CACpE,CAAC;CAGH,OAAO;AACT;AAEA,IAAM,sBAAwC;CAC5C,MAAM;CACN,QAAQ;CACR,OAAO;AACT;AAIA,SAAgB,iBAAiB,QAA2C;CAC1E,MAAM,iBAAiB,WAAW,OAAO,IAAI,SAAS;CACtD,OAAO,eAAe,cAAc,KAAK,OAAO,MAAM;AACxD;AAEA,eAAsB,aACpB,UAA4B,qBAC5B,MACiB;CACjB,IAAI,eAAe;CACnB,IAAI,CAAC,cAAc,OACjB,eAAe;EACb,GAAG;EACH,OAAO;CACT;CAEF,IAAI,aAAa,SAAS,QAAQ;EAChC,MAAM,sBAAsB;EAG5B,IAAI,iBAAiB,aAAa,MAAM,GACtC,OAAO,aAAa,cAAc,KAAA,GAAW,aAAa,QAAQ,IAAI;EAIxE,MAAM,cAAc,KAAK,YAAY;GACnC,MAAM,aAAa;GACnB,MAAM,aAAa,QAAQ;GAC3B,WAAW;EACb,CAAC;EACD,IAAI;EACJ,IAAI,YAAY;EAChB,IAAI,kBAAkC,CAAC;EACvC,MAAM,sBAAsB;GAC1B,YAAY;GACZ,kBAAkB,CAAC;EACrB;EACA,MAAM,iCAAiB,IAAI,QAAe;EAC1C,MAAM,kBAAkB,UAAiB;GAEvC,IAAI,eAAe,IAAI,KAAK,GAAG;GAC/B,eAAe,IAAI,KAAK;GAExB,KAAK,MAAM,YAAY,iBAAiB,YAAY,eAAe,SAAS,QAAQ;GACpF,YAAY,eAAe,SAAS,aAAa;GACjD,YAAY;GACZ,kBAAkB,CAAC;GAOnB,CAN0B,gBAAgB,aACxC;IAAE,OAAO;IAAS,QAAQ,aAAa;GAAO,GAC9C,KAAK,YAAY;IAAE,MAAM;IAAG,MAAM;IAAM,WAAW;GAAE,CAAC,GACtD,QACA,IACF,EAAA,CACS,MAAM;IAAE,KAAK;IAAO,MAAM,aAAa;GAAK,GAAG,4BAA4B;EACtF;EACA,MAAM,IAAI,SAAe,SAAS,WAAW;GAC3C,MAAM,gBAAgB;IACpB,YAAY,eAAe,SAAS,OAAO;IAC3C,YAAY,GAAG,SAAS,cAAc;IACtC,QAAQ;GACV;GACA,MAAM,WAAW,UAAiB;IAChC,YAAY,eAAe,SAAS,OAAO;IAG3C,WAAiB,WAAW;IAC5B,OAAO,KAAK;GACd;GACA,YAAY,KAAK,SAAS,OAAO;GACjC,YAAY,KAAK,SAAS,OAAO;EACnC,CAAC;EACD,MAAM,8BAA8B,aAAa,IAAI;EACrD,MAAM,SAAS,YAAY,OAAO,KAAK,WAAW;EAElD,YAAY,UAAU,SAAkB;GACtC,IAAI,CAAC,WAAW;IACd,YAAY;IACZ,YAAY,KAAK,SAAS,aAAa;GACzC;GACA,MAAM,oBAAoB,IAAI,IAAI,YAAY,aAAa,OAAO,CAAC;GACnE,IAAI;GACJ,IAAI;IACF,OAAO,IAAI;GACb,SAAS,OAAO;IACd,cAAc;GAChB,UAAU;IACR,gBAAgB,KACd,GAAG,YACA,aAAa,OAAO,CAAC,CACrB,QAAQ,aAAa,CAAC,kBAAkB,IAAI,QAAQ,CAAC,CAC1D;GACF;GACA,IAAI,aAAa,eAAe,WAAW;EAC7C;EACA,QAAQ,GAAG,iBAAiB;GAC1B,IAAI,CAAC,WAAW,YAAY,OAAO;EACrC,CAAC;EACD,OAAO,aAAa,cAAc,aAAa,aAAa,QAAQ,IAAI;CAC1E;CACA,MAAM,wBAAwB;CAC9B,MAAM,cAAc,iBAAiB,aAAa,MAAM,IACpD,KAAA,IACA,KAAK,YAAY;EAAE,MAAM;EAAG,MAAM,aAAa,QAAQ;EAAO,WAAW;CAAE,CAAC;CAChF,OAAO,aAAa,cAAc,aAAa,aAAa,QAAQ,IAAI;AAC1E"}

package/build/prettify.js

@@ -1,9 +1,9 @@
 const require_runtime = require("./_virtual/_rolldown/runtime.js");
 const require_formatter = require("./formatter.js");
 const require_colors = require("./colors.js");
+let on_exit_leak_free = require("on-exit-leak-free");
 let node_stream = require("node:stream");
 let node_worker_threads = require("node:worker_threads");
-let on_exit_leak_free = require("on-exit-leak-free");
 let pino_abstract_transport = require("pino-abstract-transport");
 pino_abstract_transport = require_runtime.__toESM(pino_abstract_transport);
 let sonic_boom = require("sonic-boom");

package/build/prettify.mjs

@@ -1,8 +1,8 @@
 import { printMessage } from "./formatter.mjs";
 import { hasColors } from "./colors.mjs";
+import { register, unregister } from "on-exit-leak-free";
 import { Transform, pipeline } from "node:stream";
 import { isMainThread } from "node:worker_threads";
-import { register, unregister } from "on-exit-leak-free";
 import build from "pino-abstract-transport";
 import SonicBoom from "sonic-boom";
 //#region src/prettify.ts

package/build/transport.js

@@ -1,8 +1,7 @@
 const require_colors = require("./colors.js");
 let node_path = require("node:path");
-let node_url = require("node:url");
 //#region src/transport.ts
-var currentDir = {}.url ? (0, node_path.dirname)((0, node_url.fileURLToPath)({}.url)) : __dirname;
+var currentDir = __dirname;
 var prettifyPath = (0, node_path.join)(currentDir, "..", "build", "prettify.js");
 function isPrettyFormat(format) {
 	return ["pretty-timestamped", "pretty"].includes(format ?? "pretty");

package/build/transport.js.map

@@ -1 +1 @@
-{"version":3,"file":"transport.js","names":[],"sources":["../src/transport.ts"],"sourcesContent":["import { dirname, join } from 'node:path';\nimport { fileURLToPath } from 'node:url';\n\nimport type { LoggerConfigItem, LoggerFormat } from '@verdaccio/types';\n\nimport { hasColors } from './colors';\n\n// Pino transports run in a worker thread via require(), so CJS output must work.\n// rolldown lowers `import.meta` to `{}` in the CJS output, so `import.meta.url`\n// is only truthy in the ESM build; module-scoped __dirname covers the CJS build\n// (checking `typeof __dirname` instead is unsafe: node -e and the REPL leak it\n// as a global into ES modules)\nconst currentDir = import.meta.url ? dirname(fileURLToPath(import.meta.url)) : __dirname;\nconst prettifyPath = join(currentDir, '..', 'build', 'prettify.js');\n\nexport function isPrettyFormat(format: LoggerFormat | undefined): boolean {\n  return ['pretty-timestamped', 'pretty'].includes(format ?? 'pretty');\n}\n\n/**\n * Create a pino pretty transport for non-production environments.\n */\nexport function createPrettyTransport(pino: any, options: LoggerConfigItem, format: LoggerFormat) {\n  return pino.transport({\n    target: prettifyPath,\n    options: {\n      destination: options.path || 1,\n      colors: hasColors(options.colors),\n      prettyStamp: format === 'pretty-timestamped',\n      sync: options.sync ?? false,\n    },\n    worker: {\n      name: 'verdaccio-logger-prettify',\n    },\n  });\n}\n"],"mappings":";;;;AAYA,IAAM,aAAA,CAAA,EAAyB,OAAA,GAAM,UAAA,QAAA,EAAA,GAAQ,SAAA,cAAA,CAAA,CAAA,EAA0B,GAAG,CAAC,IAAI;AAC/E,IAAM,gBAAA,GAAe,UAAA,KAAA,CAAK,YAAY,MAAM,SAAS,aAAa;AAElE,SAAgB,eAAe,QAA2C;CACxE,OAAO,CAAC,sBAAsB,QAAQ,CAAC,CAAC,SAAS,UAAU,QAAQ;AACrE;;;;AAKA,SAAgB,sBAAsB,MAAW,SAA2B,QAAsB;CAChG,OAAO,KAAK,UAAU;EACpB,QAAQ;EACR,SAAS;GACP,aAAa,QAAQ,QAAQ;GAC7B,QAAQ,eAAA,UAAU,QAAQ,MAAM;GAChC,aAAa,WAAW;GACxB,MAAM,QAAQ,QAAQ;EACxB;EACA,QAAQ,EACN,MAAM,4BACR;CACF,CAAC;AACH"}
+{"version":3,"file":"transport.js","names":[],"sources":["../src/transport.ts"],"sourcesContent":["import { join } from 'node:path';\n\nimport type { LoggerConfigItem, LoggerFormat } from '@verdaccio/types';\n\nimport { hasColors } from './colors';\n\n// Pino transports run in a worker thread via require(), so CJS output must work.\nconst currentDir = import.meta.dirname;\nconst prettifyPath = join(currentDir, '..', 'build', 'prettify.js');\n\nexport function isPrettyFormat(format: LoggerFormat | undefined): boolean {\n  return ['pretty-timestamped', 'pretty'].includes(format ?? 'pretty');\n}\n\n/**\n * Create a pino pretty transport for non-production environments.\n */\nexport function createPrettyTransport(pino: any, options: LoggerConfigItem, format: LoggerFormat) {\n  return pino.transport({\n    target: prettifyPath,\n    options: {\n      destination: options.path || 1,\n      colors: hasColors(options.colors),\n      prettyStamp: format === 'pretty-timestamped',\n      sync: options.sync ?? false,\n    },\n    worker: {\n      name: 'verdaccio-logger-prettify',\n    },\n  });\n}\n"],"mappings":";;;AAOA,IAAM,aAAA;AACN,IAAM,gBAAA,GAAe,UAAA,KAAA,CAAK,YAAY,MAAM,SAAS,aAAa;AAElE,SAAgB,eAAe,QAA2C;CACxE,OAAO,CAAC,sBAAsB,QAAQ,CAAC,CAAC,SAAS,UAAU,QAAQ;AACrE;;;;AAKA,SAAgB,sBAAsB,MAAW,SAA2B,QAAsB;CAChG,OAAO,KAAK,UAAU;EACpB,QAAQ;EACR,SAAS;GACP,aAAa,QAAQ,QAAQ;GAC7B,QAAQ,eAAA,UAAU,QAAQ,MAAM;GAChC,aAAa,WAAW;GACxB,MAAM,QAAQ,QAAQ;EACxB;EACA,QAAQ,EACN,MAAM,4BACR;CACF,CAAC;AACH"}

package/build/transport.mjs

@@ -1,8 +1,7 @@
 import { hasColors } from "./colors.mjs";
-import { dirname, join } from "node:path";
-import { fileURLToPath } from "node:url";
+import { join } from "node:path";
 //#region src/transport.ts
-var currentDir = import.meta.url ? dirname(fileURLToPath(import.meta.url)) : __dirname;
+var currentDir = import.meta.dirname;
 var prettifyPath = join(currentDir, "..", "build", "prettify.js");
 function isPrettyFormat(format) {
 	return ["pretty-timestamped", "pretty"].includes(format ?? "pretty");

package/build/transport.mjs.map

@@ -1 +1 @@
-{"version":3,"file":"transport.mjs","names":[],"sources":["../src/transport.ts"],"sourcesContent":["import { dirname, join } from 'node:path';\nimport { fileURLToPath } from 'node:url';\n\nimport type { LoggerConfigItem, LoggerFormat } from '@verdaccio/types';\n\nimport { hasColors } from './colors';\n\n// Pino transports run in a worker thread via require(), so CJS output must work.\n// rolldown lowers `import.meta` to `{}` in the CJS output, so `import.meta.url`\n// is only truthy in the ESM build; module-scoped __dirname covers the CJS build\n// (checking `typeof __dirname` instead is unsafe: node -e and the REPL leak it\n// as a global into ES modules)\nconst currentDir = import.meta.url ? dirname(fileURLToPath(import.meta.url)) : __dirname;\nconst prettifyPath = join(currentDir, '..', 'build', 'prettify.js');\n\nexport function isPrettyFormat(format: LoggerFormat | undefined): boolean {\n  return ['pretty-timestamped', 'pretty'].includes(format ?? 'pretty');\n}\n\n/**\n * Create a pino pretty transport for non-production environments.\n */\nexport function createPrettyTransport(pino: any, options: LoggerConfigItem, format: LoggerFormat) {\n  return pino.transport({\n    target: prettifyPath,\n    options: {\n      destination: options.path || 1,\n      colors: hasColors(options.colors),\n      prettyStamp: format === 'pretty-timestamped',\n      sync: options.sync ?? false,\n    },\n    worker: {\n      name: 'verdaccio-logger-prettify',\n    },\n  });\n}\n"],"mappings":";;;;AAYA,IAAM,aAAa,YAAY,MAAM,QAAQ,cAAc,YAAY,GAAG,CAAC,IAAI;AAC/E,IAAM,eAAe,KAAK,YAAY,MAAM,SAAS,aAAa;AAElE,SAAgB,eAAe,QAA2C;CACxE,OAAO,CAAC,sBAAsB,QAAQ,CAAC,CAAC,SAAS,UAAU,QAAQ;AACrE;;;;AAKA,SAAgB,sBAAsB,MAAW,SAA2B,QAAsB;CAChG,OAAO,KAAK,UAAU;EACpB,QAAQ;EACR,SAAS;GACP,aAAa,QAAQ,QAAQ;GAC7B,QAAQ,UAAU,QAAQ,MAAM;GAChC,aAAa,WAAW;GACxB,MAAM,QAAQ,QAAQ;EACxB;EACA,QAAQ,EACN,MAAM,4BACR;CACF,CAAC;AACH"}
+{"version":3,"file":"transport.mjs","names":[],"sources":["../src/transport.ts"],"sourcesContent":["import { join } from 'node:path';\n\nimport type { LoggerConfigItem, LoggerFormat } from '@verdaccio/types';\n\nimport { hasColors } from './colors';\n\n// Pino transports run in a worker thread via require(), so CJS output must work.\nconst currentDir = import.meta.dirname;\nconst prettifyPath = join(currentDir, '..', 'build', 'prettify.js');\n\nexport function isPrettyFormat(format: LoggerFormat | undefined): boolean {\n  return ['pretty-timestamped', 'pretty'].includes(format ?? 'pretty');\n}\n\n/**\n * Create a pino pretty transport for non-production environments.\n */\nexport function createPrettyTransport(pino: any, options: LoggerConfigItem, format: LoggerFormat) {\n  return pino.transport({\n    target: prettifyPath,\n    options: {\n      destination: options.path || 1,\n      colors: hasColors(options.colors),\n      prettyStamp: format === 'pretty-timestamped',\n      sync: options.sync ?? false,\n    },\n    worker: {\n      name: 'verdaccio-logger-prettify',\n    },\n  });\n}\n"],"mappings":";;;AAOA,IAAM,aAAa,YAAY;AAC/B,IAAM,eAAe,KAAK,YAAY,MAAM,SAAS,aAAa;AAElE,SAAgB,eAAe,QAA2C;CACxE,OAAO,CAAC,sBAAsB,QAAQ,CAAC,CAAC,SAAS,UAAU,QAAQ;AACrE;;;;AAKA,SAAgB,sBAAsB,MAAW,SAA2B,QAAsB;CAChG,OAAO,KAAK,UAAU;EACpB,QAAQ;EACR,SAAS;GACP,aAAa,QAAQ,QAAQ;GAC7B,QAAQ,UAAU,QAAQ,MAAM;GAChC,aAAa,WAAW;GACxB,MAAM,QAAQ,QAAQ;EACxB;EACA,QAAQ,EACN,MAAM,4BACR;CACF,CAAC;AACH"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@verdaccio/logger",
-  "version": "9.0.0-next-9.30",
+  "version": "9.0.0-next-9.32",
   "description": "Verdaccio Logger",
   "main": "./build/index.js",
   "types": "./build/index.d.ts",
@@ -33,7 +33,7 @@
     "node": ">=24"
   },
   "dependencies": {
-    "@verdaccio/core": "9.0.0-next-9.30",
+    "@verdaccio/core": "9.0.0-next-9.32",
     "colorette": "2.0.20",
     "dayjs": "1.11.23",
     "debug": "4.4.3",
@@ -43,11 +43,11 @@
     "sonic-boom": "4.2.1"
   },
   "devDependencies": {
-    "@verdaccio/types": "14.0.0-next-9.13",
+    "@verdaccio/types": "14.0.0-next-9.15",
     "cross-env": "10.1.0",
     "rimraf": "6.1.3",
-    "vite": "8.2.1",
-    "vitest": "4.1.10"
+    "vite": "8.2.2",
+    "vitest": "4.1.11"
   },
   "funding": {
     "type": "opencollective",

package/src/colors.ts

@@ -1,8 +0,0 @@
-import { isColorSupported } from 'colorette';
-
-export function hasColors(colors: boolean | undefined) {
-  if (colors) {
-    return isColorSupported;
-  }
-  return typeof colors === 'undefined' ? true : colors;
-}

package/src/formatter.ts

@@ -1,90 +0,0 @@
-import { green, red, white } from 'colorette';
-import { inspect } from 'node:util';
-
-import type { LevelCode } from './levels';
-import { calculateLevel, levelsColors, subSystemLevels } from './levels';
-import type { PrettyOptionsExtended } from './types';
-import { formatLoggingDate, isObject, padRight } from './utils';
-
-let LEVEL_VALUE_MAX = 0;
-for (const l of Object.keys(levelsColors)) {
-  LEVEL_VALUE_MAX = Math.max(LEVEL_VALUE_MAX, l.length);
-}
-
-const ERROR_FLAG = '!';
-
-export interface ObjectTemplate {
-  level: LevelCode;
-  msg: string;
-  sub?: string;
-  [key: string]: string | number | object | null | void;
-}
-
-export function fillInMsgTemplate(
-  msg: string,
-  templateOptions: ObjectTemplate,
-  colors: boolean
-): string {
-  const templateRegex = /@{(!?[$A-Za-z_][$0-9A-Za-z\._]*)}/g;
-
-  return msg.replace(templateRegex, (_, name): string => {
-    let str = templateOptions;
-    let isError;
-    if (name[0] === ERROR_FLAG) {
-      name = name.slice(1);
-      isError = true;
-    }
-
-    // object can be @{foo.bar.}
-    const listAccessors = name.split('.');
-    for (let property = 0; property < listAccessors.length; property++) {
-      const id = listAccessors[property];
-      if (isObject(str)) {
-        str = (str as object)[id];
-      }
-    }
-
-    if (typeof str === 'string') {
-      if (colors === false || (str as string).includes('\n')) {
-        return str;
-      } else if (isError) {
-        return red(str);
-      }
-      return green(str);
-    }
-
-    // object, showHidden, depth, colors
-    return inspect(str, undefined, null, colors);
-  });
-}
-
-function getMessage(debugLevel, msg, sub, templateObjects, hasColors: boolean) {
-  const finalMessage = fillInMsgTemplate(msg, templateObjects, hasColors);
-
-  const subSystemType = hasColors
-    ? subSystemLevels.color[sub ?? 'default']
-    : subSystemLevels.white[sub ?? 'default'];
-  if (hasColors) {
-    const logString = `${levelsColors[debugLevel](padRight(debugLevel, LEVEL_VALUE_MAX))}${white(
-      `${subSystemType} ${finalMessage}`
-    )}`;
-
-    return logString;
-  }
-  const logString = `${padRight(debugLevel, LEVEL_VALUE_MAX)}${subSystemType} ${finalMessage}`;
-
-  return logString;
-}
-
-export function printMessage(
-  templateObjects: ObjectTemplate,
-  options: Pick<PrettyOptionsExtended, 'prettyStamp'>,
-  hasColors: boolean
-): string {
-  const { prettyStamp } = options;
-  const { level, msg, sub } = templateObjects;
-  const debugLevel = calculateLevel(level);
-  const logMessage = getMessage(debugLevel, msg, sub, templateObjects, hasColors);
-
-  return prettyStamp ? formatLoggingDate(templateObjects.time as number, logMessage) : logMessage;
-}

package/src/index.ts

@@ -1,28 +0,0 @@
-import pino from 'pino';
-
-import type { Logger, LoggerConfigItem } from '@verdaccio/types';
-
-import { prepareSetup } from './logger';
-
-export { createLogger, prepareSetup, willUseTransport } from './logger';
-export type { LogPlugin, LoggerConfig } from './logger';
-export { default as prettifyTransport, autoEnd, buildPretty, buildSafeSonicBoom } from './prettify';
-export { hasColors } from './colors';
-export { fillInMsgTemplate, printMessage } from './formatter';
-export { createPrettyTransport, isPrettyFormat } from './transport';
-export type { LevelCode } from './levels';
-export { formatLoggingDate, padRight } from './utils';
-
-/**
- * Initialize the logger singleton.
- */
-export async function setup(options: LoggerConfigItem): Promise<Logger> {
-  if (typeof logger !== 'undefined') {
-    return logger;
-  }
-
-  logger = await prepareSetup(options, pino);
-  return logger;
-}
-
-export let logger: Logger;

package/src/levels.ts

@@ -1,60 +0,0 @@
-import { black, blue, cyan, green, magenta, red, white, yellow } from 'colorette';
-
-export type LogLevel = 'trace' | 'debug' | 'info' | 'http' | 'warn' | 'error' | 'fatal';
-
-export type LevelCode = number;
-
-export function calculateLevel(levelCode: LevelCode): LogLevel {
-  switch (true) {
-    case levelCode === 10:
-      return 'trace';
-    case levelCode === 20:
-      return 'debug';
-    case levelCode === 25:
-      return 'http';
-    case levelCode === 30:
-      return 'info';
-    case levelCode === 40:
-      return 'warn';
-    case levelCode === 50:
-      return 'error';
-    case levelCode === 60:
-      return 'fatal';
-    default:
-      return 'fatal';
-  }
-}
-
-export const levelsColors = {
-  fatal: red,
-  error: red,
-  warn: yellow,
-  http: magenta,
-  info: cyan,
-  debug: green,
-  trace: white,
-};
-
-enum ARROWS {
-  LEFT = '<--',
-  RIGHT = '-->',
-  EQUAL = '-=-',
-  NEUTRAL = '---',
-}
-
-export const subSystemLevels = {
-  color: {
-    in: green(ARROWS.LEFT),
-    out: yellow(ARROWS.RIGHT),
-    auth: blue(ARROWS.NEUTRAL),
-    fs: black(ARROWS.EQUAL),
-    default: blue(ARROWS.NEUTRAL),
-  },
-  white: {
-    in: ARROWS.LEFT,
-    out: ARROWS.RIGHT,
-    auth: ARROWS.NEUTRAL,
-    fs: ARROWS.EQUAL,
-    default: ARROWS.NEUTRAL,
-  },
-};