fix: host must be a valid HTTPS URL when using ElectronPublicUpdateSe…

…rvice (#174)

* test: Add test for only give update type electron and repo but no host

* fix: Set a default host is used without specifying a host

Only happened when using 'ElectronPublicUpdateService'

src/index.ts

@@ -323,6 +323,10 @@ function validateInput(opts: IUpdateElectronAppOptions) {
         'repo is required and should be in the format `owner/repo`',
       );
 
+      if (!updateSource.host) {
+        updateSource.host = host;
+      }
+
       assert(updateSource.host && isHttpsUrl(updateSource.host), 'host must be a valid HTTPS URL');
       break;
     }

test/index.test.ts

@@ -4,7 +4,13 @@ import path from 'node:path';
 
 import { autoUpdater, dialog } from 'electron';
 
-import { updateElectronApp, makeUserNotifier, IUpdateInfo, IUpdateDialogStrings } from '../src';
+import {
+  updateElectronApp,
+  makeUserNotifier,
+  IUpdateInfo,
+  IUpdateDialogStrings,
+  UpdateSourceType,
+} from '../src';
 const repo = 'some-owner/some-repo';
 
 beforeEach(() => {
@@ -49,6 +55,15 @@ describe('updateElectronApp', () => {
         updateElectronApp({ repo, host: 'http://example.com' });
       }).toThrow('host must be a valid HTTPS URL');
     });
+
+    it('from default', () => {
+      updateElectronApp({
+        updateSource: {
+          type: UpdateSourceType.ElectronPublicUpdateService,
+          repo,
+        },
+      });
+    });
   });
 
   describe('updateInterval', () => {