Skip to content

Commit c0a3629

Browse files
Always enable websocket transport (#232)
1 parent 3328340 commit c0a3629

File tree

2 files changed

+2
-14
lines changed

2 files changed

+2
-14
lines changed

startupscript/butane/006-start-proxy-agent.sh

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,6 @@ readonly TERRA_SERVER
4141
if [[ "${TERRA_SERVER}" == "dev-stable" ]]; then
4242
OPTIONS+=("--debug=true")
4343
fi
44-
PROXY_WS_TUNNEL="$(get_metadata_value "proxy-websocket-tunnel-enabled" "")"
45-
if [[ "${PROXY_WS_TUNNEL}" == "TRUE" ]]; then
46-
OPTIONS+=("--websocket-transport=true")
47-
fi
48-
readonly PROXY_WS_TUNNEL
4944
readonly OPTIONS
5045

5146
# Pull the latest proxy agent
@@ -67,4 +62,5 @@ docker run \
6762
--shim-path="${SHIM_PATH}" \
6863
--rewrite-websocket-host="${REWRITE_WEBSOCKET_HOST}" \
6964
--enable-monitoring-script="${ENABLE_MONITORING_SCRIPT:-false}" \
65+
--websocket-transport=true \
7066
"${OPTIONS[@]}"

startupscript/dataproc/startup.sh

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -871,14 +871,6 @@ if [[ "${PROXY_TYPE}" != "${PROXY_TYPE_GOOGLE}" ]]; then
871871
ENABLE_MONITORING_SCRIPT="$(get_metadata_value "instance/attributes/enable-dataproc-monitoring-script")"
872872
readonly ENABLE_MONITORING_SCRIPT
873873

874-
OPTIONS=()
875-
PROXY_WS_TUNNEL="$(get_metadata_value "instance/attributes/proxy-websocket-tunnel-enabled" "")"
876-
if [[ "${PROXY_WS_TUNNEL}" == "TRUE" ]]; then
877-
OPTIONS+=("--websocket-transport=true")
878-
fi
879-
readonly PROXY_WS_TUNNEL
880-
readonly OPTIONS
881-
882874
readonly WORKSPACE_LINK_EL='<a id="workspace" class="forum" target="_blank" href="'"${UI_BASE_URL}/workspaces/${TERRA_WORKSPACE}"'"'">${TERRA_WORKSPACE}</a>"
883875

884876
# Define Workbench Proxy Agent startup script
@@ -937,7 +929,7 @@ docker run \
937929
--session-cookie-name="_xsrf" \
938930
--inject-banner="\$(cat ${PROXY_AGENT_BANNER})" \
939931
--enable-monitoring-script="${ENABLE_MONITORING_SCRIPT:-false}" \
940-
${OPTIONS[@]}
932+
--websocket-transport=true
941933
EOF
942934
# Grant execute permission to the startup script
943935
chmod +x "${WORKBENCH_PROXY_AGENT_STARTUP_SCRIPT}"

0 commit comments

Comments
 (0)