-
-
Notifications
You must be signed in to change notification settings - Fork 144
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
timeout in waiting for rekeying process. #439
Comments
Can you help me figure out how to reproduce it? |
Since I can't reproduce the issue, I need the following information:
|
@gdevenyi Please answer my questions. I need to be able to reproduce the issue so I can determine whether the patch actually fixes it. Since TurboVNC 3.2 beta1 is imminent, a timely response is appreciated. |
This is due to specifics of my ssh config I have
To make bad connections timeout sooner and this appears to be too aggressive somewhere. But, the failure happens way earlier than 10 seconds. Furthermore If change to
It still fails exactly the same way. If I comment out these lines, the config works. |
I will always assume and test the default configuration unless you specify that your configuration is not the default. In the future, please include that information in the original bug report. This appears to be a similar issue to #161, in that JSch incorrectly treats the |
JSch expects ConnectTimeout to be in milliseconds, but OpenSSH specifies that it should be in seconds, so we need to convert the value after it is read from the OpenSSH config file. Fixes #439
JSch expects ConnectTimeout to be in milliseconds, but OpenSSH specifies that it should be in seconds, so we need to convert the value after it is read from the OpenSSH config file. Fixes #439
Will definitely be much more specific in the future. This is a non-OpenSSH implementation so eveything is up for grabs. |
Did the patch fix the issue on your end? The issue I observed was subtly different (different error message), so I'm not sure if it was the same issue. |
The timeout bug is fixed in 2025-01-22. |
Trying to test the new implementation from #393 (from today's dev build)
I get the following error:
Which leads me to this reasonably recent bug from jsch
mwiede/jsch#604
The text was updated successfully, but these errors were encountered: