fix: add logger to electron-updater (#2552)

* tmp: debugging autoupdater

* tmp: debugging auto-update

* tmp: random dir in tmp dir

* tmp: more debugging and tweaks

* chore: cleanup

* chore: cleanup

src/auto-updater/index.js

@@ -20,6 +20,7 @@ function setup (ctx) {
   // we download manually in 'update-available'
   autoUpdater.autoDownload = false
   autoUpdater.autoInstallOnAppQuit = true
+  autoUpdater.logger = logger
 
   autoUpdater.on('error', err => {
     logger.error(`[updater] ${err.toString()}`)
@@ -88,6 +89,21 @@ function setup (ctx) {
     })
   })
 
+  let progressPercentTimeout = null
+  autoUpdater.on('download-progress', ({ percent, bytesPerSecond }) => {
+    const logDownloadProgress = () => {
+      logger.info(`[updater] download progress is ${percent.toFixed(2)}% at ${bytesPerSecond} bps.`)
+    }
+    // log the percent, but not too often to avoid spamming the logs, but we should
+    // be sure we're logging at what percent any hiccup is occurring.
+    clearTimeout(progressPercentTimeout)
+    if (percent === 100) {
+      logDownloadProgress()
+      return
+    }
+    progressPercentTimeout = setTimeout(logDownloadProgress, 2000)
+  })
+
   autoUpdater.on('update-downloaded', ({ version }) => {
     logger.info(`[updater] update to ${version} downloaded`)
 
@@ -151,7 +167,7 @@ async function checkForUpdates () {
 }
 
 module.exports = async function (ctx) {
-  if (['test', 'development'].includes(process.env.NODE_ENV)) {
+  if (['test', 'development'].includes(process.env.NODE_ENV ?? '')) {
     ctx.manualCheckForUpdates = () => {
       showDialog({
         title: 'Not available in development',

src/common/logger.js

@@ -121,6 +121,10 @@ module.exports = Object.freeze({
     logger.error(err)
   },
 
+  warn: (msg, meta) => {
+    logger.warn(msg, meta)
+  },
+
   logsPath,
   addAnalyticsEvent
 })