Fix crash with update check on Linux

Fix #3907 Auditors: @aekeus Test Plan: Open the browser and wait 3 minutes. Check updateLog.log and it should have a message about not scheduling updates
brave · Sep 12, 2016 · dcc4a8f · dcc4a8f · aekeus · Sep 12, 2016
1 parent 47bf893
commit dcc4a8f
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/app/updater.js b/app/updater.js
@@ -89,15 +89,18 @@ exports.init = (platform, arch, ver) => {
   version = ver
 
   var baseUrl = exports.updateUrl(appConfig.updates, platform, arch)
-  debug(`updateUrl = ${baseUrl}`)
 
-  scheduleUpdates()
-
-  // This will fail if we are in dev
-  try {
-    autoUpdater.setFeedURL(baseUrl)
-  } catch (err) {
-    console.log(err)
+  if (baseUrl) {
+    debug(`updateUrl = ${baseUrl}`)
+    scheduleUpdates()
+    // This will fail if we are in dev
+    try {
+      autoUpdater.setFeedURL(baseUrl)
+    } catch (err) {
+      console.log(err)
+    }
+  } else {
+    debug('No updateUrl, not scheduling updates.')
   }
 }