feat: improve update logic and error messages

Author: flavioislima

public/locales/en/translation.json

@@ -88,8 +88,25 @@
                 "title": "Ubisoft Connect"
             },
             "update": {
-                "body": "Something went wrong with the update after multiple attempts! Please manually uninstall and reinstall HyperPlay. error: {{error}}",
-                "title": "Update Error"
+                "addressUnreachable": "Address unreachable. Please check your internet connection.",
+                "body": "Something went wrong with the update after multiple attempts! Please check the error message below or reinstall HyperPlay. error: {{error}}",
+                "certAuthorityInvalid": "Certificate authority invalid. Please check your system time and date or open a ticket with HyperPlay support.",
+                "certDateInvalid": "Certificate date invalid. Please check your system time and date or open a ticket with HyperPlay support.",
+                "connectionAborted": "Connection aborted. Please check your internet connection.",
+                "connectionClosed": "Connection closed. Please check your internet connection.",
+                "connectionRefused": "Connection refused. Please check your internet connection.",
+                "connectionReset": "Connection reset. Please check your internet connection.",
+                "connectionTimedOut": "Connection timed out. Please check your internet connection.",
+                "emptyResponse": "Empty response. Please check your internet connection.",
+                "failed": "Download Failed. Please check your internet connection.",
+                "http2ServerRefusedStream": "HTTP2 server refused stream. Please check your internet connection.",
+                "internetDisconnected": "Internet disconnected. Please check your internet connection.",
+                "message": "Error Updating",
+                "nameNotResolved": "Name not resolved. Please check your internet connection.",
+                "networkAccessDenied": "Network access denied. Please check your internet connection.",
+                "networkChanged": "Network changed. Please check your internet connection.",
+                "proxyConnectionFailed": "Proxy connection failed. Please check your proxy settings.",
+                "sslProtocolError": "SSL protocol error. Please check your system time and date or open a ticket with HyperPlay support."
             },
             "winetricks": {
                 "message": "Winetricks returned the following error during execution:{{newLine}}{{error}}",

src/backend/updater.ts

@@ -2,12 +2,15 @@ import { dialog, shell } from 'electron'
 import { autoUpdater } from 'electron-updater'
 import { t } from 'i18next'
 
-import { configStore, icon, isLinux } from './constants'
+import { configStore, icon, isLinux, isMac } from './constants'
 import { logError, logInfo, LogPrefix } from './logger/logger'
 import { captureException } from '@sentry/electron'
 import { getFileSize } from './utils'
 import { ClientUpdateStatuses } from '@hyperplay/utils'
 import { trackEvent } from './metrics/metrics'
+import { homedir } from 'os'
+import { join } from 'path'
+import { rm } from 'fs/promises'
 // to test auto update on windows locally make sure you added the option "verifyUpdateCodeSignature": false
 // under build.win in package.json and also change the app version to an old one there
 
@@ -21,18 +24,32 @@ autoUpdater.autoInstallOnAppQuit = true
 let isAppUpdating = false
 let hasUpdated = false
 let updateAttempts = 0
-const MAX_UPDATE_ATTEMPTS = 3
+const MAX_UPDATE_ATTEMPTS = 5
 
 // check for updates every hour
-const checkUpdateInterval = 1 * 60 * 60 * 1000
-setInterval(() => {
-  if (shouldCheckForUpdates) {
-    autoUpdater.checkForUpdates()
+const checkUpdateInterval = 1 * 1000 * 60 * 60
+setInterval(async () => {
+  if (shouldCheckForUpdates && !hasUpdated && !isAppUpdating) {
+    logInfo('Checking for client updates...', LogPrefix.AutoUpdater)
+    await autoUpdater.checkForUpdates()
   }
 }, checkUpdateInterval)
 
 autoUpdater.on('update-available', async (info) => {
+  if (isAppUpdating && hasUpdated) {
+    logInfo(
+      'New update available, but user has already updated the app',
+      LogPrefix.AutoUpdater
+    )
+    return
+  }
+
   if (!shouldCheckForUpdates) {
+    logInfo(
+      'New update available, but user has disabled auto updates',
+      LogPrefix.AutoUpdater
+    )
+
     return
   }
   newVersion = info.version
@@ -70,6 +87,7 @@ autoUpdater.on('download-progress', (progress) => {
 
 autoUpdater.on('update-downloaded', async () => {
   logInfo('App update is downloaded')
+  hasUpdated = true
 
   trackEvent({
     event: 'Client Update Downloaded',
@@ -92,12 +110,22 @@ autoUpdater.on('update-downloaded', async () => {
   if (response === 1) {
     return autoUpdater.quitAndInstall()
   }
-  hasUpdated = true
+  logInfo('User chose not to update the app for now.', LogPrefix.AutoUpdater)
 })
 
 autoUpdater.on('error', async (error) => {
   isAppUpdating = false
-  logError(`Error updating HyperPlay: ${error.message}`, LogPrefix.AutoUpdater)
+
+  // To avoid false positives, we should not show the error dialog if the app has already updated successfully
+  if (hasUpdated) {
+    return
+  }
+
+  const errorMessage = getErrorMessage(error.message)
+  logError(`Error updating HyperPlay: ${errorMessage}`, LogPrefix.AutoUpdater)
+
+  // will remove cached updates when it fails to avoid corrupted updates
+  await removeCachedUpdatesFolder()
 
   updateAttempts++
 
@@ -133,12 +161,11 @@ autoUpdater.on('error', async (error) => {
   updateAttempts = 0
 
   const { response } = await dialog.showMessageBox({
-    title: t('box.error.update.title', 'Error Updating'),
+    title: t('box.error.update.message', 'Error Updating'),
     message: t(
-      'box.error.update.message',
-      `Something went wrong with the update after multiple attempts! Please manually uninstall and reinstall HyperPlay. error: ${JSON.stringify(
-        error
-      )}`
+      'box.error.update.body',
+      `Something went wrong with the update after multiple attempts! Please check the error message below or reinstall HyperPlay. error: {{error}}`,
+      { error: errorMessage }
     ),
     type: 'error',
     buttons: [t('button.cancel', 'Cancel'), t('button.download', 'Download')]
@@ -155,3 +182,119 @@ export function isClientUpdating(): ClientUpdateStatuses {
   }
   return isAppUpdating ? 'updating' : 'idle'
 }
+
+async function removeCachedUpdatesFolder() {
+  // remove hyperplay-updates folder from cache directory
+  // on macOS: /Users/<username>/Library/Caches/hyperplay-updates
+  // on Windows: C:\Users\<username>\AppData\Local\hyperplay-updates
+  const macOSPath = join(homedir(), 'Library', 'Caches', 'hyperplay-updates')
+  const windowsPath = join(homedir(), 'AppData', 'Local', 'hyperplay-updates')
+
+  try {
+    await rm(isMac ? macOSPath : windowsPath, { recursive: true })
+  } catch (error) {
+    logError(
+      `Error removing cached updates folder: ${error}`,
+      LogPrefix.AutoUpdater
+    )
+  }
+}
+
+const commonDownloadErrors: Record<string, () => string> = {
+  ERR_NETWORK_CHANGED: () =>
+    t(
+      'box.error.update.networkChanged',
+      'Network changed. Please check your internet connection.'
+    ),
+  ERR_INTERNET_DISCONNECTED: () =>
+    t(
+      'box.error.update.internetDisconnected',
+      'Internet disconnected. Please check your internet connection.'
+    ),
+  ERR_CONNECTION_RESET: () =>
+    t(
+      'box.error.update.connectionReset',
+      'Connection reset. Please check your internet connection.'
+    ),
+  ERR_CONNECTION_CLOSED: () =>
+    t(
+      'box.error.update.connectionClosed',
+      'Connection closed. Please check your internet connection.'
+    ),
+  ERR_CONNECTION_TIMED_OUT: () =>
+    t(
+      'box.error.update.connectionTimedOut',
+      'Connection timed out. Please check your internet connection.'
+    ),
+  ERR_NAME_NOT_RESOLVED: () =>
+    t(
+      'box.error.update.nameNotResolved',
+      'Name not resolved. Please check your internet connection.'
+    ),
+  ERR_CONNECTION_REFUSED: () =>
+    t(
+      'box.error.update.connectionRefused',
+      'Connection refused. Please check your internet connection.'
+    ),
+  ERR_SSL_PROTOCOL_ERROR: () =>
+    t(
+      'box.error.update.sslProtocolError',
+      'SSL protocol error. Please check your system time and date or open a ticket with HyperPlay support.'
+    ),
+  ERR_CERT_AUTHORITY_INVALID: () =>
+    t(
+      'box.error.update.certAuthorityInvalid',
+      'Certificate authority invalid. Please check your system time and date or open a ticket with HyperPlay support.'
+    ),
+  ERR_NETWORK_ACCESS_DENIED: () =>
+    t(
+      'box.error.update.networkAccessDenied',
+      'Network access denied. Please check your internet connection.'
+    ),
+  ERR_PROXY_CONNECTION_FAILED: () =>
+    t(
+      'box.error.update.proxyConnectionFailed',
+      'Proxy connection failed. Please check your proxy settings.'
+    ),
+  ERR_CONNECTION_ABORTED: () =>
+    t(
+      'box.error.update.connectionAborted',
+      'Connection aborted. Please check your internet connection.'
+    ),
+  ERR_ADDRESS_UNREACHABLE: () =>
+    t(
+      'box.error.update.addressUnreachable',
+      'Address unreachable. Please check your internet connection.'
+    ),
+  ERR_CERT_DATE_INVALID: () =>
+    t(
+      'box.error.update.certDateInvalid',
+      'Certificate date invalid. Please check your system time and date or open a ticket with HyperPlay support.'
+    ),
+  ERR_HTTP2_SERVER_REFUSED_STREAM: () =>
+    t(
+      'box.error.update.http2ServerRefusedStream',
+      'HTTP2 server refused stream. Please check your internet connection.'
+    ),
+  ERR_EMPTY_RESPONSE: () =>
+    t(
+      'box.error.update.emptyResponse',
+      'Empty response. Please check your internet connection.'
+    ),
+  ERR_FAILED: () =>
+    t(
+      'box.error.update.failed',
+      'Download Failed. Please check your internet connection.'
+    )
+}
+
+function getErrorMessage(error: string): string {
+  const trimmedError = error.replace('net::', '').trim()
+  if (
+    Object.prototype.hasOwnProperty.call(commonDownloadErrors, trimmedError)
+  ) {
+    return commonDownloadErrors[trimmedError]()
+  } else {
+    return error
+  }
+}