Don't allocate a ptty on remote end

Bug #1127546 reported by Brendan Kidwell
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Tunnel Manager
Fix Released
Medium
Brandon Williams

Bug Description

I may not be understanding this correctly, but it looks in like in source/tunnelrunner, you're explicitly allocating a ptty on the remote end when you invoke 'ssh', when you probably meant to do the opposite.

In the ssh manual, the '-T' option is to disable allocating a ptty, and '-t' is for forcing it.

Revision history for this message
Brendan Kidwell (bkidwell) wrote :
Revision history for this message
Brandon Williams (opensource-subakutty) wrote :

I find opening a pseudo-TTY to be a reliable way to keep a long-running SSH tunnel open on some SSH servers -- more reliable than keep-alives. For this reason, I consider -t to be a good default for this program.

I will consider adding a hidden config option to allow the user to modify the default tunnel arguments. This will allow you to change -t to -T if you desire, while not exposing this "expert" level option to regular users for whom the current default command line is working just fine.

Hopefully I'll be able to get to this in a couple of weeks time.

If enough other people indicate that they would prefer -T to be the default, then I will consider making that the default and override it locally for my own use.

Changed in tunnelmanager:
importance: Undecided → Medium
status: New → In Progress
assignee: nobody → Brandon Williams (opensource-subakutty)
Revision history for this message
Brandon Williams (opensource-subakutty) wrote :

Revision 73 adds a new hidden configuration variable for advanced users. Add the following to your ~/.TunnelManager/properties.conf:

[BASE]
sshbaseargs = ['-N', '-T', '-x', '-o', 'ExitOnForwardFailure=yes']

This will change the '-t' to '-T' when executing ssh. If you prefer this to be server specific, remove '-t' from the list entirely and add server specific settings to your ~/.ssh/config file.

Changed in tunnelmanager:
status: In Progress → Fix Committed
Changed in tunnelmanager:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.