fix: logging uses window.log and doesn't print object

packages/session-desktop/preload.js

@@ -222,8 +222,8 @@ ipc.on('get-ready-for-shutdown', async () => {
 
 // We pull these dependencies in now, from here, because they have Node.js dependencies
 
-const logging = require('./ts/util/logging');
-window.log = { ...logging.default };
+const { loggingFunctions } = require('./ts/util/logging');
+window.log = loggingFunctions;
 
 if (config.proxyUrl) {
   window.log.info('Using provided proxy url');

packages/session-desktop/ts/mains/main_node.ts

@@ -192,7 +192,7 @@ function prepareURL(pathSegments: Array<string>, moreKeys?: { theme: any }) {
       locale: locale.name,
       version: app.getVersion(),
       commitHash,
-      environment: (config as any).environment,
+      environment: 'production',
       node_version: process.versions.node,
       hostname: os.hostname(),
       appInstance: process.env.NODE_APP_INSTANCE,

packages/session-desktop/ts/session/apis/seed_node_api/SeedNodeAPI.ts

@@ -257,10 +257,7 @@ async function getSnodesFromSeedUrl(urlObj: URL): Promise<Array<any>> {
     },
     agent: sslAgent,
   };
-  window?.log?.info(
-    `insecureNodeFetch => plaintext for getSnodesFromSeedUrl  ${url}`,
-    fetchOptions
-  );
+  window?.log?.info(`insecureNodeFetch => plaintext for getSnodesFromSeedUrl  ${url}`);
 
   const response = await window.fetch(url, {
     method: fetchOptions.method,

packages/session-desktop/ts/util/logging.ts

@@ -49,15 +49,24 @@ function log(...args: any) {
   logAtLevel('info', 'INFO ', ...args);
 }
 
+const loggingFunctions = {
+  fatal: _.partial(logAtLevel, 'fatal', 'FATAL'),
+  error: _.partial(logAtLevel, 'error', 'ERROR'),
+  warn: _.partial(logAtLevel, 'warn', 'WARN '),
+  info: _.partial(logAtLevel, 'info', 'INFO '),
+  debug: _.partial(logAtLevel, 'debug', 'DEBUG'),
+  trace: _.partial(logAtLevel, 'trace', 'TRACE'),
+};
+
 if (window.console) {
-  (console as any)._log = (console as any).log;
+  (console as any)._log = console.log;
   (console as any).log = log;
-  (console as any)._trace = (console as any).trace;
-  (console as any)._debug = (console as any).debug;
-  (console as any)._info = (console as any).info;
-  (console as any)._warn = (console as any).warn;
-  (console as any)._error = (console as any).error;
-  (console as any)._fatal = (console as any).error;
+  (console as any)._trace = console.trace;
+  (console as any)._debug = console.debug;
+  (console as any)._info = console.info;
+  (console as any)._warn = console.warn;
+  (console as any)._error = console.error;
+  (console as any)._fatal = console.error;
 }
 
 // The mechanics of preparing a log for publish
@@ -125,15 +134,6 @@ function logAtLevel(level: string, prefix: string, ...args: any) {
   ipc.send(`log-${level}`, logText);
 }
 
-export default {
-  fatal: _.partial(logAtLevel, 'fatal', 'FATAL'),
-  error: _.partial(logAtLevel, 'error', 'ERROR'),
-  warn: _.partial(logAtLevel, 'warn', 'WARN '),
-  info: _.partial(logAtLevel, 'info', 'INFO '),
-  debug: _.partial(logAtLevel, 'debug', 'DEBUG'),
-  trace: _.partial(logAtLevel, 'trace', 'TRACE'),
-};
-
 window.onerror = (_message, _script, _line, _col, error) => {
   const errorInfo = JSON.stringify(error);
 
@@ -148,3 +148,5 @@ window.addEventListener('unhandledrejection', rejectionEvent => {
   const errorInfo = error && error.stack ? error.stack : error;
   window.log.error('Top-level unhandled promise rejection:', errorInfo);
 });
+
+export { loggingFunctions };