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

Fixes the generation of keys on systems with newer OpenSSH versions (>= 7.8) #30

Merged
merged 1 commit into from
Nov 28, 2018

Conversation

danielrs
Copy link

The OpenSSH release:

https://www.openssh.com/txt/release-7.8

Changes the default format when generating private keys, this pull requests fixes that by letting the users on systems with newer OpenSSH versions specify if they want to use the old PEM format by passing the flag --use-pem to the generate-keys.sh script.

#23

users generate the keys using the old PEM style on newer OpenSSH
versions (>= 7.8). Updated the README.md accordingly.
@pivotal-issuemaster
Copy link

@danielrs Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@danielrs Thank you for signing the Contributor License Agreement!

@ahaczewski
Copy link

ahaczewski commented Nov 27, 2018

Wouldn't it be possible to detect ssh version and then check whether it is greater than 7.8?

Like

verle() {
    [  "$1" = "`echo -e "$1\n$2" | sort -V | head -n1`" ]
}
sshver=`ssh -V 2>&1 | cut -d, -f1 | cut -d' ' -f1 | cut -d_ -f2`
param=`verle 7.8 $sshver && echo "-m PEM" || echo ""`

@larssb
Copy link

larssb commented Nov 28, 2018

Let's get this in :-). Would be great as newcomers would then not bump their heads into this issue.

Thnx

@vito
Copy link
Member

vito commented Nov 28, 2018

Happy to merge this in since the problem seems to be particular acute for new users. Auto-detecting it and/or implementing the key generation stuff mentioned in concourse/docs#124 would be better IMO but for now, documenting it is better than nothing.

Thanks!

@vito vito merged commit 25a3567 into concourse:master Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants