fix: add check for process global to exist (#1195)

near · Oct 30, 2023 · 695220e · 695220e
1 parent 038b9b9
commit 695220e
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 6 deletions.
diff --git a/.changeset/funny-jars-camp.md b/.changeset/funny-jars-camp.md
@@ -0,0 +1,8 @@
+---
+"@near-js/accounts": patch
+"near-api-js": patch
+"@near-js/providers": patch
+"@near-js/utils": patch
+---
+
+add check for global 'process' object
diff --git a/packages/accounts/src/account.ts b/packages/accounts/src/account.ts
@@ -363,7 +363,7 @@ export class Account {
      * @param beneficiaryId The NEAR account that will receive the remaining Ⓝ balance from the account being deleted
      */
     async deleteAccount(beneficiaryId: string) {
-        if (!process.env['NEAR_NO_LOGS']) {
+        if (!(typeof process === 'object' && process.env['NEAR_NO_LOGS'])) {
             console.log('Deleting an account does not automatically transfer NFTs and FTs to the beneficiary address. Ensure to transfer assets before deleting.');
         }
         return this.signAndSendTransaction({

diff --git a/packages/near-api-js/src/connect.ts b/packages/near-api-js/src/connect.ts
@@ -54,7 +54,7 @@ export async function connect(config: ConnectConfig): Promise<Near> {
                     keyPathStore,
                     config.keyStore || config.deps?.keyStore
                 ], { writeKeyStoreIndex: 1 });
-                if (!process.env['NEAR_NO_LOGS']) {
+                if (!(typeof process === 'object' && process.env['NEAR_NO_LOGS'])) {
                     console.log(`Loaded master account ${accountKeyFile[0]} key from ${config.keyPath} with public key = ${keyPair.getPublicKey()}`);
                 }
             }

diff --git a/packages/providers/src/fetch_json.ts b/packages/providers/src/fetch_json.ts
@@ -16,7 +16,7 @@ export interface ConnectionInfo {
     headers?: { [key: string]: string | number };
 }
 
-const logWarning = (...args) => !process.env['NEAR_NO_LOGS'] && console.warn(...args);
+const logWarning = (...args) => !(typeof process === 'object' && process.env['NEAR_NO_LOGS']) && console.warn(...args);
 
 export async function fetchJson(connectionInfoOrUrl: string | ConnectionInfo, json?: string): Promise<any> {
     let connectionInfo: ConnectionInfo = { url: null };

diff --git a/packages/providers/src/json-rpc-provider.ts b/packages/providers/src/json-rpc-provider.ts
@@ -372,7 +372,7 @@ export class JsonRpcProvider extends Provider {
                 return response;
             } catch (error) {
                 if (error.type === 'TimeoutError') {
-                    if (!process.env['NEAR_NO_LOGS']) {
+                    if (!(typeof process === 'object' && process.env['NEAR_NO_LOGS'])) {
                         console.warn(`Retrying request to ${method} as it has timed out`, params);
                     }
                     return null;

diff --git a/packages/utils/src/errors/errors.ts b/packages/utils/src/errors/errors.ts
@@ -1,5 +1,5 @@
 export function logWarning(...args: any[]): void {
-    if (!process.env['NEAR_NO_LOGS']){
+    if (!(typeof process === 'object' && process.env['NEAR_NO_LOGS'])){
         console.warn(...args);
     }
 }
diff --git a/packages/utils/src/logging.ts b/packages/utils/src/logging.ts
@@ -2,7 +2,7 @@ import { FinalExecutionOutcome } from '@near-js/types';
 
 import { parseRpcError } from './errors';
 
-const SUPPRESS_LOGGING = !!process.env.NEAR_NO_LOGS;
+const SUPPRESS_LOGGING = !!(typeof process === 'object' && process.env.NEAR_NO_LOGS);
 
 /**
  * Parse and print details from a query execution response