Skip to content

Commit f05a841

Browse files
authored
Disable Code Signing for MacOS (#2292)
* Update electron.yaml * disable code signing for macos * fix: small issue with jwt creation
1 parent 7e753b6 commit f05a841

File tree

3 files changed

+20
-11
lines changed

3 files changed

+20
-11
lines changed

.github/workflows/electron.yaml

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -70,25 +70,33 @@ jobs:
7070
run: |
7171
cd launcher && npm i
7272
73-
- name: Build Electron app
73+
- name: Build Electron app (windows and linux)
7474
uses: samuelmeuli/action-electron-builder@v1
7575
with:
7676
github_token: ${{ secrets.github_token }}
7777
package_root: launcher
7878
use_vue_cli: true
7979
release: never
8080
build_script_name: dist
81-
mac_certs: ${{ secrets.mac_certs }}
82-
mac_certs_password: ${{ secrets.mac_certs_password }}
8381
args: ${{ env.BUILD_ARGS }}
8482
env:
85-
APPLE_ID: ${{ secrets.apple_id }}
86-
APPLE_APP_SPECIFIC_PASSWORD: ${{ secrets.apple_app_specific_password }}
87-
TEAM_SHORT_NAME: ${{ secrets.team_short_name }}
88-
APP_ID: "com.stereum.launcher"
89-
APPLE_TEAM_ID: ${{ secrets.team_id }}
83+
NOTARIZE: "true"
9084
KEYPAIR_ALIAS: ${{ secrets.KEYPAIR_ALIAS }}
91-
if: ${{ !contains(github.event.head_commit.message, '[NOCI]') }}
85+
if: ${{ !contains(github.event.head_commit.message, '[NOCI]') && matrix.os != 'macos-latest' }}
86+
87+
- name: Build Electron app (macOS, unsigned)
88+
uses: samuelmeuli/action-electron-builder@v1
89+
with:
90+
github_token: ${{ secrets.github_token }}
91+
package_root: launcher
92+
use_vue_cli: true
93+
release: never
94+
build_script_name: dist
95+
args: ${{ env.BUILD_ARGS }}
96+
env:
97+
NOTARIZE: "false"
98+
CSC_IDENTITY_AUTO_DISCOVERY: "false"
99+
if: ${{ !contains(github.event.head_commit.message, '[NOCI]') && matrix.os == 'macos-latest'}}
92100

93101
- name: Verify Signing using Signtool on windows-latest
94102
run: |

controls/roles/manage-service/tasks/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@
109109
hex_chars: "0123456789abcdef"
110110
become: yes
111111
when:
112-
- stereum_service_configuration.service in ['BesuService', 'GethService', 'RethService', 'NethermindService', 'ErigonService', 'LighthouseBeaconService', 'NimbusBeaconService', 'PrysmBeaconService', 'TekuBeaconService', 'LodestarBeaconService', 'OpGethService', 'OpErigonService', 'OpRethService']
112+
- stereum_service_configuration.service in ['BesuService', 'GethService', 'RethService', 'NethermindService', 'ErigonService', 'LighthouseBeaconService', 'NimbusBeaconService', 'PrysmBeaconService', 'TekuBeaconService', 'LodestarBeaconService', 'OpGethService', 'OpErigonService', 'OpRethService', 'GrandineBeaconService', 'EthrexService']
113113
- stereum_service_configuration.volumes | select('search', ':/engine.jwt|/execution/engine.jwt|:/op-engine.jwt') | length > 0
114114

115115
- include_tasks: write-graffiti-files.yml

launcher/vue.config.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
const shouldNotarize = process.env.NOTARIZE === "true";
12
module.exports = {
23
parallel: false,
34
pluginOptions: {
@@ -10,7 +11,7 @@ module.exports = {
1011
},
1112
appId: "com.stereum.launcher",
1213
productName: "Stereum-Launcher",
13-
afterSign: "@sapien99/vue-cli-plugin-electron-builder-notarize",
14+
...(shouldNotarize ? { afterSign: "@sapien99/vue-cli-plugin-electron-builder-notarize" } : {}),
1415
buildDependenciesFromSource: false,
1516
nodeGypRebuild: false,
1617
npmRebuild: false,

0 commit comments

Comments
 (0)