Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Script execution pauses for passphrase protected keys #174

Open
srm09 opened this issue Sep 11, 2020 · 3 comments
Open

Script execution pauses for passphrase protected keys #174

srm09 opened this issue Sep 11, 2020 · 3 comments
Labels
api-update Updates to the programmatic API

Comments

@srm09
Copy link
Contributor

srm09 commented Sep 11, 2020

Problem

For passphrase protected keys, the script execution pauses for the script operator to enter the passphrase. The only way to avoid this is to add the key to the default ssh-agent prior to the script execution.

If the user wants to use the Internal Crashd Agent, there is no way to bypass the passphrase prompt. It would be helpful to add a way to provide the passphrase in the key.

Possible Solution

Introduce a new directive ssh_key which has the following:

  • path ==> points to the path of the key file
  • passphrase ==> string input for the passphrase of the key
  • passphrase-file ==> path of the file containing the passphrase

This ssh_key struct can then be passed to the ssh_config instead of just passing the private_key_pathto the ssh_config.

@srm09 srm09 added the api-update Updates to the programmatic API label Sep 11, 2020
@vladimirvivien
Copy link
Contributor

@srm09 I don't think it would be good to (encourage) store passphrase in a file. There is no way around it, key requires a passphrase the user must enter it. It probably is a good idea to enter in at standard input masked. That way crashd does not have to handle it. I think good documentation explaining this is enough.

@srm09
Copy link
Contributor Author

srm09 commented Sep 11, 2020

The idea was to introduce a non-interactive way of running crashd with passphrase protected keys.

@vladimirvivien
Copy link
Contributor

Yes, I dont know if there is a safe around it. And definitely storing of passphrase is a no no.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-update Updates to the programmatic API
Projects
None yet
Development

No branches or pull requests

2 participants