From b5d2c013a18257f433b112d6557384bf41e9a789 Mon Sep 17 00:00:00 2001 From: Sasha <118575614+weboko@users.noreply.github.com> Date: Thu, 16 Jan 2025 13:32:40 +0100 Subject: [PATCH] fix access to process variable (#2201) --- packages/sdk/src/create/libp2p.ts | 5 +++-- packages/sdk/src/env.ts | 8 ++++++++ 2 files changed, 11 insertions(+), 2 deletions(-) create mode 100644 packages/sdk/src/env.ts diff --git a/packages/sdk/src/create/libp2p.ts b/packages/sdk/src/create/libp2p.ts index e979487f9b..7aca089b2e 100644 --- a/packages/sdk/src/create/libp2p.ts +++ b/packages/sdk/src/create/libp2p.ts @@ -17,6 +17,7 @@ import { import { derivePubsubTopicsFromNetworkConfig, Logger } from "@waku/utils"; import { createLibp2p } from "libp2p"; +import { isTestEnvironment } from "../env.js"; import { CreateWakuNodeOptions, DefaultPingMaxInboundStreams, @@ -36,7 +37,7 @@ export async function defaultLibp2p( options?: Partial, userAgent?: string ): Promise { - if (!options?.hideWebSocketInfo && process?.env?.NODE_ENV !== "test") { + if (!options?.hideWebSocketInfo && !isTestEnvironment()) { /* eslint-disable no-console */ console.info( "%cIgnore WebSocket connection failures", @@ -54,7 +55,7 @@ export async function defaultLibp2p( : {}; const filter = - options?.filterMultiaddrs === false || process?.env?.NODE_ENV === "test" + options?.filterMultiaddrs === false || isTestEnvironment() ? filterAll : wss; diff --git a/packages/sdk/src/env.ts b/packages/sdk/src/env.ts new file mode 100644 index 0000000000..c134d1bbcc --- /dev/null +++ b/packages/sdk/src/env.ts @@ -0,0 +1,8 @@ +export function isTestEnvironment(): boolean { + try { + return process?.env?.NODE_ENV === "test"; + } catch (_e) { + // process variable is not defined in PROD environment + return false; + } +}