From ad81c13ab2b863be46a98803e48754b0e2b3746b Mon Sep 17 00:00:00 2001 From: AJ Ancheta <7781450+ancheetah@users.noreply.github.com> Date: Wed, 11 Feb 2026 10:20:19 -0500 Subject: [PATCH] fix(logger): fix custom logger --- .changeset/happy-actors-learn.md | 5 +++++ packages/sdk-effects/logger/src/lib/logger.effects.ts | 10 ++++++---- 2 files changed, 11 insertions(+), 4 deletions(-) create mode 100644 .changeset/happy-actors-learn.md diff --git a/.changeset/happy-actors-learn.md b/.changeset/happy-actors-learn.md new file mode 100644 index 0000000000..d5963aa9d8 --- /dev/null +++ b/.changeset/happy-actors-learn.md @@ -0,0 +1,5 @@ +--- +'@forgerock/sdk-logger': patch +--- + +Fixes double logging when using custom logger. Custom logger should override default console logs. diff --git a/packages/sdk-effects/logger/src/lib/logger.effects.ts b/packages/sdk-effects/logger/src/lib/logger.effects.ts index fc386133e5..be365c4221 100644 --- a/packages/sdk-effects/logger/src/lib/logger.effects.ts +++ b/packages/sdk-effects/logger/src/lib/logger.effects.ts @@ -13,10 +13,12 @@ export function logger(config: { level: LogLevel; custom?: CustomLogger }) { // Implement log functions const logFunctions = { - error: (...args: LogMessage[]) => custom?.error(...args) || console.error(...args), - warn: (...args: LogMessage[]) => custom?.warn(...args) || console.warn(...args), - info: (...args: LogMessage[]) => custom?.info(...args) || console.info(...args), - debug: (...args: LogMessage[]) => custom?.debug(...args) || console.debug(...args), + error: (...args: LogMessage[]) => + custom?.error ? custom.error(...args) : console.error(...args), + warn: (...args: LogMessage[]) => (custom?.warn ? custom.warn(...args) : console.warn(...args)), + info: (...args: LogMessage[]) => (custom?.info ? custom.info(...args) : console.info(...args)), + debug: (...args: LogMessage[]) => + custom?.debug ? custom.debug(...args) : console.debug(...args), }; // Implement level inclusion