Skip to content

Commit

Permalink
deps: Upgrade @electron/notarize to v2.1.0 (#2331)
Browse files Browse the repository at this point in the history
This package replaces the older `electron-notarize`.
This version uses Apple's `notarytool` executable by default to sign
code which we'll need as the legacy executable will stop working by
Nov. 1 2023.
  • Loading branch information
taratatach authored Oct 25, 2023
1 parent 6d75a1f commit 7eb9b23
Show file tree
Hide file tree
Showing 5 changed files with 19 additions and 14 deletions.
4 changes: 4 additions & 0 deletions .github/actions/build-and-publish/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,9 @@ inputs:
apple-id-password:
description: 'The Apple notarization account password.'
required: false
apple-team-id:
description: 'Id of the team the Apple notarization account if part of.'
required: false
runs:
using: composite
steps:
Expand All @@ -28,6 +31,7 @@ runs:
CSC_KEY_PASSWORD: "${{ inputs.mac-cert-password }}"
APPLE_ID: "${{ inputs.apple-id }}"
APPLE_ID_PASSWORD: "${{ inputs.apple-id-password }}"
APPLE_TEAM_ID: "${{ inputs.apple-team-id }}"
run: |
if [ "${{ runner.os }}" == "Linux" ]; then
# Install build dependencies:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/macos.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -273,6 +273,7 @@ jobs:
mac-cert-password: "${{ secrets.mac_cert_password }}"
apple-id: "${{ secrets.apple_id }}"
apple-id-password: "${{ secrets.apple_id_password }}"
apple-team-id: "${{ secrets.apple_team_id }}"
- name: Save artifacts
if: ${{ github.event_name == 'pull_request' }}
uses: actions/upload-artifact@v3
Expand All @@ -283,4 +284,3 @@ jobs:
${{ github.workspace }}/dist/Cozy[- ]Drive[- ]*.dmg
${{ github.workspace }}/dist/Cozy[- ]Drive[- ]*-mac.zip
retention-days: 5

6 changes: 3 additions & 3 deletions build/afterSignHook.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
const fs = require('fs')
const path = require('path')
// eslint-disable-next-line node/no-unpublished-require
const electron_notarize = require('electron-notarize')
const electron_notarize = require('@electron/notarize')

module.exports = async function (params) {
// Only notarize the app on Mac OS only.
Expand All @@ -28,10 +28,10 @@ module.exports = async function (params) {
console.log(`Notarizing ${appId} found at ${appPath}`)

await electron_notarize.notarize({
appBundleId: appId,
appPath: appPath,
appleId: process.env.APPLE_ID,
appleIdPassword: process.env.APPLE_ID_PASSWORD
appleIdPassword: process.env.APPLE_ID_PASSWORD,
teamId: process.env.APPLE_TEAM_ID
})

// eslint-disable-next-line no-console
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,7 @@
},
"devDependencies": {
"@babel/core": "7.16.7",
"@electron/notarize": "^2.1.0",
"@sentry/cli": "^2.3.0",
"babel-preset-cozy-app": "^2.0.1",
"chai": "4.3.4",
Expand All @@ -151,7 +152,6 @@
"electron-builder": "^23.6.0",
"electron-download": "^4.1.1",
"electron-mocha": "11.0.2",
"electron-notarize": "^0.1.1",
"elm": "^0.19.1",
"elm-format": "0.8.5",
"elm-test": "^0.19.1",
Expand Down
19 changes: 10 additions & 9 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -1070,6 +1070,15 @@
optionalDependencies:
global-agent "^3.0.0"

"@electron/notarize@^2.1.0":
version "2.1.0"
resolved "https://registry.yarnpkg.com/@electron/notarize/-/notarize-2.1.0.tgz#76aaec10c8687225e8d0a427cc9df67611c46ff3"
integrity sha512-Q02xem1D0sg4v437xHgmBLxI2iz/fc0D4K7fiVWHa/AnW8o7D751xyKNXgziA6HrTOme9ul1JfWN5ark8WH1xA==
dependencies:
debug "^4.1.1"
fs-extra "^9.0.1"
promise-retry "^2.0.1"

"@electron/[email protected]":
version "2.0.8"
resolved "https://registry.yarnpkg.com/@electron/remote/-/remote-2.0.8.tgz#85ff321f0490222993207106e2f720273bb1a5c3"
Expand Down Expand Up @@ -3753,14 +3762,6 @@ [email protected]:
which "^2.0.2"
yargs "^16.2.0"

electron-notarize@^0.1.1:
version "0.1.1"
resolved "https://registry.yarnpkg.com/electron-notarize/-/electron-notarize-0.1.1.tgz#c3563d70c5e7b3315f44e8495b30050a8c408b91"
integrity sha512-TpKfJcz4LXl5jiGvZTs5fbEx+wUFXV5u8voeG5WCHWfY/cdgdD8lDZIZRqLVOtR3VO+drgJ9aiSHIO9TYn/fKg==
dependencies:
debug "^4.1.1"
fs-extra "^8.0.1"

electron-osx-sign@^0.6.0:
version "0.6.0"
resolved "https://registry.yarnpkg.com/electron-osx-sign/-/electron-osx-sign-0.6.0.tgz#9b69c191d471d9458ef5b1e4fdd52baa059f1bb8"
Expand Down Expand Up @@ -4604,7 +4605,7 @@ [email protected]:
jsonfile "^6.0.1"
universalify "^2.0.0"

[email protected], fs-extra@^8.0.1, fs-extra@^8.1.0:
[email protected], fs-extra@^8.1.0:
version "8.1.0"
resolved "https://registry.yarnpkg.com/fs-extra/-/fs-extra-8.1.0.tgz#49d43c45a88cd9677668cb7be1b46efdb8d2e1c0"
integrity sha512-yhlQgA6mnOJUKOsRUFsgJdQCvkKhcz8tlZG5HBQfReYZy46OwLcY+Zia0mtdHsOo9y/hP+CxMN0TU9QxoOtG4g==
Expand Down

0 comments on commit 7eb9b23

Please sign in to comment.