pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-08-07 15:52:52 +03:00
Code Issues Packages Projects Releases Wiki Activity

chore: remove unused shared logger default export (#5380)

Browse Source 
Integrated into release/v3.8.42
This commit is contained in:
Jan Leon
2026-06-30 00:15:10 +02:00
committed by GitHub
parent 8e8d08ea15
commit f2d7aa9303
2 changed files with 11 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/shared/utils/logger.ts
View File

@@ -171,5 +171,3 @@ export const logger = buildLogger();
export function createLogger(module: string) {
return logger.child({ module });
}
export default logger;

13
tests/unit/logger-redaction-wiring.test.ts
View File

@@ -12,7 +12,8 @@ process.env.APP_LOG_TO_FILE = "true";
process.env.APP_LOG_FILE_PATH = logFile;
process.env.APP_LOG_LEVEL = "debug";
const { createLogger } = await import("../../src/shared/utils/logger.ts");
const loggerModule = await import("../../src/shared/utils/logger.ts");
const { createLogger } = loggerModule;
/** Poll the (worker-thread-written) log file until the predicate holds or timeout. */
async function readLogWhen(
@@ -38,10 +39,18 @@ test("logger redacts a Bearer secret in a free-form message and an error stack (
log.error({ err }, "request failed");
const contents = await readLogWhen(
(c) => c.includes("[REDACTED]") && !c.includes("sk-superSecretKey") && !c.includes("sk-anotherSecret")
(c) =>
c.includes("[REDACTED]") &&
!c.includes("sk-superSecretKey") &&
!c.includes("sk-anotherSecret")
);
assert.match(contents, /\[REDACTED\]/, "redaction marker must appear in the log output");
assert.doesNotMatch(contents, /sk-superSecretKey/, "message secret must be redacted");
assert.doesNotMatch(contents, /sk-anotherSecret/, "error-stack secret must be redacted");
});
test("shared logger module exposes named logger helpers", () => {
assert.equal(typeof loggerModule.logger.info, "function");
assert.equal(typeof loggerModule.createLogger, "function");
});