Skip to content

Commit

Permalink
chore: remove broken default option download.rejectUnauthorized (#1658
Browse files Browse the repository at this point in the history
)
  • Loading branch information
erikian committed Jan 12, 2024
1 parent 55eebc7 commit 10474b0
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 17 deletions.
2 changes: 0 additions & 2 deletions src/cli.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,15 +19,13 @@ export function parseArgs(argv: string[]) {
boolean: [
'all',
'deref-symlinks',
'download.rejectUnauthorized',
'junk',
'overwrite',
'prune',
'quiet'
],
default: {
'deref-symlinks': true,
'download.rejectUnauthorized': true,
junk: true,
prune: true
},
Expand Down
2 changes: 0 additions & 2 deletions src/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -340,8 +340,6 @@ export interface Options {
* include, but are not limited to:
* - `cacheRoot` (*string*): The directory where prebuilt, pre-packaged Electron downloads are cached.
* - `mirrorOptions` (*Object*): Options to override the default Electron download location.
* - `rejectUnauthorized` (*boolean* - default: `true`): Whether SSL certificates are required to be
* valid when downloading Electron.
*
* **Note:** `download` sub-options will have no effect if the {@link electronZipDir} option is set.
*/
Expand Down
10 changes: 0 additions & 10 deletions test/cli.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,6 @@ test('CLI argument: --electron-version populates opts.electronVersion', t => {
t.is(args.electronVersion, '1.2.3')
})

test('CLI argument: --download.rejectUnauthorized default', t => {
const args = parseArgs([])
t.true(args.download.rejectUnauthorized, 'default for --download.rejectUnauthorized is true')
})

test('CLI argument: --no-download.rejectUnauthorized makes rejectUnauthorized false', t => {
const args = parseArgs(['--no-download.rejectUnauthorized'])
t.false(args.download.rejectUnauthorized, 'download.rejectUnauthorized should be false')
})

test('CLI argument: --asar=true', t => {
const args = parseArgs(['--asar=true'])
t.true(args.asar)
Expand Down
3 changes: 0 additions & 3 deletions usage.txt
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,6 @@ download a list of sub-options to pass to @electron/get. They are spec
@electron/get documentation
- mirrorOptions: alternate URL options for downloading Electron zips. See
@electron/get documentation for details
- rejectUnauthorized: whether SSL certs are required to be valid when downloading
Electron. Defaults to true, use --no-download.rejectUnauthorized to disable
checks.
electron-version the version of Electron that is being packaged, see
https://github.com/electron/electron/releases
electron-zip-dir the local path to a directory containing Electron ZIP files
Expand Down

0 comments on commit 10474b0

Please sign in to comment.