Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

Add a reminder to ssh-add #38

Merged
merged 4 commits into from Feb 13, 2018
Merged

Conversation

dryewo
Copy link
Contributor

@dryewo dryewo commented Feb 13, 2018

I tripped on this multiple times, without adding the key to SSH Agent ssh just says "Permission denied (public key)".

This additional message would have helped.

@aermakov-zalando
Copy link
Contributor

Can you make it print the warning only if the SSH agent reports no installed keys? It should be easy to check with ssh-add -l.

@aermakov-zalando
Copy link
Contributor

It might make sense to also print it as a warning.

@dryewo
Copy link
Contributor Author

dryewo commented Feb 13, 2018

is it safe to assume that every computer has ssh-add installed?

@aermakov-zalando
Copy link
Contributor

@dryewo you can just ignore it if it's not. but if it's installed and reports some keys, there's no point in printing the warning.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.5%) to 70.0% when pulling ba33c94 on dryewo:patch-1 into 117db8a on zalando-stups:master.

@aermakov-zalando
Copy link
Contributor

👍

1 similar comment
@dryewo
Copy link
Contributor Author

dryewo commented Feb 13, 2018

👍

@aermakov-zalando aermakov-zalando merged commit 9092f07 into zalando-stups:master Feb 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants