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 Incorrect Socket Timeout #28

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

bakertim
Copy link

self.timeout is already converted to seconds so the extra /1000 changes the default 10 second timeout to 0.01 seconds. That's long enough that 1 out of every 3 or 4 attempts will connect correctly but the rest fail with the output:

DEBUG: Timeout set to 10.0 seconds
INFO: Libssh2 v1.8.0
DEBUG: <URL> resolved. 1 addresses
WARN: Connection to <IP_ADDRESS> on port 22 failed
INFO: Bye bye
DEBUG: Disconnected

This should resolve #24.

@jandolina
Copy link

This fixed my issues! Thank you.

@nmingotti
Copy link

I confirm, removing / 1000 is fundamental or running a ssh comman will rarely work. I installed via pod a few days ago, the bug is still there. The easiest is to correct it straight in XCode, for now.

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.

Authenticating with Public/Private key
3 participants