Skip to content

Commit

Permalink
[BUG]: fixup enable-ssh-askpass-require
Browse files Browse the repository at this point in the history
Fix typo in ssh-askpass-require->enable-ssh-askpass-require.
The fragment file name and the function check for that file name didn't
match so it would never ask for the passphrase on the cli in Linux.

This is now working as expected.

Fixes #229, #233
  • Loading branch information
dglynn authored and Darren Glynn committed Apr 2, 2024
1 parent 70a356e commit 2c6747d
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions zsh/.zshrc
Original file line number Diff line number Diff line change
Expand Up @@ -222,9 +222,7 @@ function zsh-quickstart-enable-omz-plugins() {
}

function zsh-quickstart-set-ssh-askpass-require() {
if [[ $(_zqs-get-setting ssh-askpass-require) == 'true' ]]; then
export SSH_ASKPASS_REQUIRE=never
fi
export SSH_ASKPASS_REQUIRE=never
}

function zsh-quickstart-enable-ssh-askpass-require() {
Expand Down Expand Up @@ -385,7 +383,7 @@ load-our-ssh-keys() {

if [[ -z "$SSH_CLIENT" ]] || can_haz keychain; then
# We're not on a remote machine, so load keys
if [[ "$(_zqs-get-setting ssh-askpass-require)" == 'true' ]]; then
if [[ "$(_zqs-get-setting enable-ssh-askpass-require)" == 'true' ]]; then
zsh-quickstart-set-ssh-askpass-require
fi
load_ssh_keys="$(_zqs-get-setting load-ssh-keys true)"
Expand Down

0 comments on commit 2c6747d

Please sign in to comment.