search

palantir / giraffe

Gracefully Integrated Remote Access For Files and Execution
https://palantir.github.io/giraffe/
Apache License 2.0
51 stars 16 forks source link

fix thread leak #84

Open rzpt opened 5 years ago

rzpt commented 5 years ago

If the SshHostControlSystem constructor throws, then it will not be closed.

palantirtech commented 5 years ago

Thanks for your interest in palantir/giraffe, @rzpt! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.