Skip to content

Conversation

@zankich
Copy link
Contributor

@zankich zankich commented Apr 18, 2016

While following the AWS guide, we ran into issues with the fingerprint which was generated. The guide currently creates a SHA256 fingerprint, and this pull request changes the ssh-keygen command to generate an md5 fingerprint.

Without this change we were getting the following error message while attempting to SSH into an app deployed onto our CF installation.

○ → cf ssh dora -i 0
FAILED
Error opening SSH connection: ssh: handshake failed: Unsupported host key fingerprint format

As well as the following acceptance-tests failues

[Fail] {NO_DEA_SUPPORT} SSH ssh [It] can execute a remote command in the container
/Users/pivotal/go/src/github.com/cloudfoundry/cf-acceptance-tests/ssh/ssh_test.go:68

[Fail] {NO_DEA_SUPPORT} SSH ssh [It] runs an interactive session when no command is provided
/Users/pivotal/go/src/github.com/cloudfoundry/cf-acceptance-tests/ssh/ssh_test.go:105

[Fail] {NO_DEA_SUPPORT} SSH ssh [It] allows local port forwarding
/Users/pivotal/go/src/github.com/cloudfoundry/cf-acceptance-tests/ssh/ssh_test.go:126

cc @kkallday

- now generates a fingerprint in md5 format

Signed-off-by: Kevin Kelani <[email protected]>
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/117855121.

@emalm
Copy link
Contributor

emalm commented Apr 18, 2016

Thanks, @zankich. This seems to be a behavioral change in ssh-keygen: on OS X 10.10 Yosemite and Ubuntu Trusty, ssh-keygen always produces the colon-separated MD5 hash and has no -E flag. On OS X 10.11 El Capitan and Ubuntu Xenial, it behaves as documented. Would you mind updating the documentation to cover both behavioral cases? I unfortunately don't think that there's a single command that works for both, so maybe present the -E-flag command as the primary case and the older, non--E case as the fallback?

Thanks again,
Eric, CF Runtime Diego PM

@zankich
Copy link
Contributor Author

zankich commented Apr 18, 2016

@ematpl we've updated the instructions for older OS version. Let us know if you'd like anything else changed!

@emalm
Copy link
Contributor

emalm commented Apr 18, 2016

Thanks, @zankich! I'll merge it once the diego-release can-i-bump light goes green.

Best,
Eric

@emalm emalm merged commit 19db249 into cloudfoundry:develop Apr 19, 2016
luan added a commit that referenced this pull request Oct 26, 2016
Submodule src/github.com/onsi/gomega 7ce781e..16f4cfd:
  > Merge pull request #178 from ebabani/master
  > Merge pull request #152 from tinygrasshopper/kill-rogue-processes
  > Merge pull request #171 from timstclair/gstruct
  > Update README.md
  > Merge pull request #148 from farjump/fix-byte-slice-formatter
  > Merge pull request #166 from trayo/patch-2
  > Merge pull request #164 from wendorf/assert_typo
  > Merge remote-tracking branch 'origin/pr/163'
  > Merge pull request #160 from tinygrasshopper/fix_failing_close_ghttp
  > Merge pull request #150 from tinygrasshopper/build-fix
  > Merge pull request #159 from WesleyJeanette/patch-1
  > Merge pull request #157 from kwadrat/master
  > Merge pull request #141 from mariantalla/gomega-yaml-matcher
  > Reset tmpDir in gexec.CleanupBuildArtifacts

Signed-off-by: Bernerd Schaefer <[email protected]>
anoop2811 pushed a commit that referenced this pull request Nov 2, 2016
Submodule src/github.com/onsi/gomega 7ce781e..16f4cfd:
  > Merge pull request #178 from ebabani/master
  > Merge pull request #152 from tinygrasshopper/kill-rogue-processes
  > Merge pull request #171 from timstclair/gstruct
  > Update README.md
  > Merge pull request #148 from farjump/fix-byte-slice-formatter
  > Merge pull request #166 from trayo/patch-2
  > Merge pull request #164 from wendorf/assert_typo
  > Merge remote-tracking branch 'origin/pr/163'
  > Merge pull request #160 from tinygrasshopper/fix_failing_close_ghttp
  > Merge pull request #150 from tinygrasshopper/build-fix
  > Merge pull request #159 from WesleyJeanette/patch-1
  > Merge pull request #157 from kwadrat/master
  > Merge pull request #141 from mariantalla/gomega-yaml-matcher
  > Reset tmpDir in gexec.CleanupBuildArtifacts

Signed-off-by: Bernerd Schaefer <[email protected]>
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.

3 participants