-
-
Notifications
You must be signed in to change notification settings - Fork 312
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
Autossh reconnection issue with sish #309
Comments
Hi,
Step to reproduce the issue:
I hope it helps you. |
sish will close unused connections (include closed ones) by default, unless you mess something up with your configuration. Otherwise, you can try to set:
|
Agree with setting I’m still looking into this as I’m wondering if maybe it has something to do with autossh’s monitor port. |
Hi, sish_issue.mp4Best, |
Hi @antoniomika In the past weeks I did some tests and with the help of a developer friend of mine, we found a way to solve the issue. We added, to line 64 of sshmuxer/channel.go and to line 87 of utils/conn.go a sleep of 3 seconds. In this way, the issue disappears: reconnections and stop/start tunnel with autossh work correctly. I attach the modified files. I did not make a pull request, because I think the fix is dirty, surely you will have a better and more elegant way to solve the problem :D Best, |
Hello, |
Hi @antoniomika,
I think I found an issue when using sish with autossh.
After starting sish with autossh with the following command, for example:
When you want to stop the tunnel, since autossh automatically reconnects the tunnel, you have to press CTRL+C several times in rapid succession to stop sish session and exit autossh.
if you try to reconnect to the same domain sish says:
In order to reuse the blocked domain, sish must be restarted
the problem is probably due to the fact that sish is very quick to reconnect.
I managed to replicate the issue several time.
A possible solution to mitigate the problem could be a parameter to delay the ssh connection when ssh is invoked? (--ssh-connection-delay=5s)
If you need to do further tests I can help you.
Best,
Fabio.
The text was updated successfully, but these errors were encountered: