-
-
Notifications
You must be signed in to change notification settings - Fork 317
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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: