Skip to content

Commit 9c77b41

Browse files
committed
Fix how Node options are specified for build step and production
1 parent 813f7dc commit 9c77b41

File tree

6 files changed

+4
-11
lines changed

6 files changed

+4
-11
lines changed

.github/workflows/release-live.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,5 @@ jobs:
125125
app_suffix: ""
126126
version_number: "${{ needs.determine_version.outputs.next_version }}"
127127
vcs_tag_prefix: "SFv"
128-
node_options: "--optimize_for_size --max_old_space_size=4096"
129128
project: "SIL.XForge.Scripture"
130129
server_domain_name: "scriptureforge.org"

.github/workflows/release-qa.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,5 @@ jobs:
146146
app_suffix: "_qa"
147147
version_number: "${{ needs.determine_version.outputs.next_version }}"
148148
vcs_tag_prefix: "SF-QAv"
149-
node_options: "--optimize_for_size --max_old_space_size=4096"
150149
project: "SIL.XForge.Scripture"
151150
server_domain_name: "qa.scriptureforge.org"

.github/workflows/release.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,6 @@ on:
5252
description: "The version number to use for the deployment."
5353
type: string
5454
required: true
55-
node_options:
56-
description: "Node-related options when building RealtimeServer."
57-
type: string
58-
required: true
5955
project:
6056
description: "The project to deploy."
6157
type: string
@@ -174,7 +170,6 @@ jobs:
174170
DEPLOY_CREDENTIALS: ~/.ssh/deploy_key
175171
HOSTNAME: ${{inputs.server_domain_name}}
176172
ALTERNATE_DOMAIN: ${{secrets.alternate_domain}}
177-
NODE_OPTIONS: ${{inputs.node_options}}
178173
PARATEXT_CLIENT_ID: ${{secrets.paratext_client_id}}
179174
PROJECT: ${{inputs.project}}
180175
SERVAL_CLIENT_ID: ${{secrets.serval_client_id}}

scripts/build-and-ship

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,7 @@ cat <<EOF >"$BUILD_OUTPUT/app/secrets.json"
3636
},
3737
"Site": {
3838
"Origin": "https://${HOSTNAME};${ALTERNATE_DOMAIN}"
39-
},
40-
"node-options": "${NODE_OPTIONS}"
39+
}
4140
}
4241
EOF
4342

src/SIL.XForge.Scripture/Startup.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ public IServiceProvider ConfigureServices(IServiceCollection services)
155155

156156
services.AddSignalR();
157157

158-
string? nodeOptions = Configuration.GetValue<string>("node-options");
158+
string? nodeOptions = Configuration.GetValue<string>("Realtime:NodeOptions");
159159
services.AddSFRealtimeServer(LoggerFactory, Configuration, nodeOptions);
160160

161161
services.AddSFServices();

src/SIL.XForge.Scripture/appsettings.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,8 @@
2929
"Prefix": "sf"
3030
},
3131
"Realtime": {
32-
"Port": 5003
32+
"Port": 5003,
33+
"NodeOptions": "--optimize_for_size"
3334
},
3435
"Bugsnag": {
3536
"ApiKey": "b72a46a8924a3cd161d4c5534287923c",

0 commit comments

Comments
 (0)