From 958928865d75c414ac765d77533141d9dbb5022e Mon Sep 17 00:00:00 2001 From: Marvin Liu Date: Tue, 14 Jan 2025 13:28:19 -0800 Subject: [PATCH] fix: add safe access to loggerProvider in session replay plugin --- .../plugin-session-replay-browser/src/session-replay.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/plugin-session-replay-browser/src/session-replay.ts b/packages/plugin-session-replay-browser/src/session-replay.ts index 60d55afe0..8f5e60563 100644 --- a/packages/plugin-session-replay-browser/src/session-replay.ts +++ b/packages/plugin-session-replay-browser/src/session-replay.ts @@ -18,7 +18,7 @@ export class SessionReplayPlugin implements EnrichmentPlugin { async setup(config: BrowserConfig) { try { - config.loggerProvider.log(`Installing @amplitude/plugin-session-replay, version ${VERSION}.`); + config?.loggerProvider.log(`Installing @amplitude/plugin-session-replay, version ${VERSION}.`); this.config = config; @@ -66,7 +66,7 @@ export class SessionReplayPlugin implements EnrichmentPlugin { experimental: this.options.experimental, }).promise; } catch (error) { - config.loggerProvider.error(`Session Replay: Failed to initialize due to ${(error as Error).message}`); + config?.loggerProvider.error(`Session Replay: Failed to initialize due to ${(error as Error).message}`); } } @@ -109,7 +109,7 @@ export class SessionReplayPlugin implements EnrichmentPlugin { return Promise.resolve(event); } catch (error) { - this.config.loggerProvider.error(`Session Replay: Failed to enrich event due to ${(error as Error).message}`); + this.config?.loggerProvider.error(`Session Replay: Failed to enrich event due to ${(error as Error).message}`); return Promise.resolve(event); } } @@ -123,7 +123,7 @@ export class SessionReplayPlugin implements EnrichmentPlugin { // @ts-ignore this.config = null; } catch (error) { - this.config.loggerProvider.error(`Session Replay: teardown failed due to ${(error as Error).message}`); + this.config?.loggerProvider.error(`Session Replay: teardown failed due to ${(error as Error).message}`); } }