Disable tty pass through to talisman binary in pre-push hook mode

thoughtworks · Sep 7, 2020 · df8a1cc · df8a1cc
1 parent 648e456
commit df8a1cc
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/global_install_scripts/talisman_hook_script.bash b/global_install_scripts/talisman_hook_script.bash
@@ -75,7 +75,10 @@ fi
 DEBUG_OPTS=""
 [[ -n "${TALISMAN_DEBUG}" ]] && DEBUG_OPTS="-d"
 INTERACTIVE=""
-[[ -n "${TALISMAN_INTERACTIVE}" ]] && INTERACTIVE="-i"
+if [ -n "${TALISMAN_INTERACTIVE}" ]; then
+    INTERACTIVE="-i"
+	[[ "${HOOKNAME}" == "pre-commit" ]] && exec < /dev/tty || echo_warning "talisman pre-push hook cannot be invoked in interactive mode currently"
+fi
 
 CMD="${TALISMAN_BINARY} ${DEBUG_OPTS} --githook ${HOOKNAME} ${INTERACTIVE}"
 echo_debug "ARGS are $@"