linux/ohmyzsh
Watch
1
0
Fork
You've already forked ohmyzsh
 0
mirror of https://github.com/ohmyzsh/ohmyzsh.git synced 2026-08-07 06:08:39 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(aws): use return instead of exit in state helpers to avoid killing the shell

Browse source 
`_aws_update_state` and `_aws_clear_state` ran `... || exit 1` when the
directory of `$AWS_STATE_FILE` did not exist. These helpers are sourced
functions invoked from `asp`/`asr` (and profile clearing), so `exit 1`
terminates the user's interactive shell (the terminal closes) instead of
just aborting the helper.

Use `return 1` so a missing state directory aborts only the helper,
leaving the interactive shell intact. Only affects users who set
`AWS_PROFILE_STATE_ENABLED=true`.
This commit is contained in:
Your Name 2026-06-12 16:11:22 +01:00
commit d4a9158a5c
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
plugins/aws/aws.plugin.zsh
View file

@ -9,14 +9,14 @@ function agr() {
# Update state file if enabled # Update state file if enabled
function _aws_update_state() { function _aws_update_state() {
if [[ "$AWS_PROFILE_STATE_ENABLED" == true ]]; then if [[ "$AWS_PROFILE_STATE_ENABLED" == true ]]; then
test -d $(dirname ${AWS_STATE_FILE}) || exit 1 test -d $(dirname ${AWS_STATE_FILE}) || return 1
echo "${AWS_PROFILE} ${AWS_REGION}" > "${AWS_STATE_FILE}" echo "${AWS_PROFILE} ${AWS_REGION}" > "${AWS_STATE_FILE}"
fi fi
} }
function _aws_clear_state() { function _aws_clear_state() {
if [[ "$AWS_PROFILE_STATE_ENABLED" == true ]]; then if [[ "$AWS_PROFILE_STATE_ENABLED" == true ]]; then
test -d $(dirname ${AWS_STATE_FILE}) || exit 1 test -d $(dirname ${AWS_STATE_FILE}) || return 1
echo -n > "${AWS_STATE_FILE}" echo -n > "${AWS_STATE_FILE}"
fi fi
} }