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

Multiple tests fail with latest upstream ssh package #155

Closed
eikenb opened this issue Feb 1, 2017 · 4 comments
Closed

Multiple tests fail with latest upstream ssh package #155

eikenb opened this issue Feb 1, 2017 · 4 comments
Assignees

Comments

@eikenb
Copy link
Member

eikenb commented Feb 1, 2017

They fail on the same thing. Here is an example.

$ go test -integration -v -run TestServerCompareSubsystems
=== RUN TestServerCompareSubsystems
--- FAIL: TestServerCompareSubsystems (0.11s)
server_integration_test.go:348: EOF
server_integration_test.go:407: exit status 255
FAIL
exit status 1
FAIL github.com/pkg/sftp 0.117s

I bisected it down to commit a59c127441a8ae2ad9b0fb300ab36a6558bba697 in the golang.org/x/crypto package. I think I found an upstream bug about it. I'm testing the fix they have for review now.

@eikenb
Copy link
Member Author

eikenb commented Feb 1, 2017

Forgot to link to the upstream bug.

golang/go#18861

And the fix under review.

https://go-review.googlesource.com/#/c/36055/1

@eikenb
Copy link
Member Author

eikenb commented Feb 1, 2017

I have verified that the fix for the upstream issue fixes these failing tests. I'll leave this ticket open and run tests locally until this is fixed.

@eikenb
Copy link
Member Author

eikenb commented Feb 5, 2017

Implemented a hack in for travis so I could start merging without travis failing constantly. Will leave this open thought until I can revert the hack.

@eikenb
Copy link
Member Author

eikenb commented Feb 6, 2017

Fixed upstream and workaround removed.

@eikenb eikenb closed this as completed Feb 6, 2017
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

No branches or pull requests

1 participant