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

Provide an alternative mechanism for extracting the server IP address #417

Closed
wants to merge 1 commit into from

Conversation

pcc
Copy link

@pcc pcc commented Apr 15, 2013

On some installations of ssh, overriding ProxyCommand is forbidden,
which means that the fake proxy won't work. Instead, have mosh-server
report the local address in SSH_CONNECTION, which will be used if
the fake proxy doesn't report an address.

On some installations of ssh, overriding ProxyCommand is forbidden,
which means that the fake proxy won't work.   Instead, have mosh-server
report the local address in SSH_CONNECTION, which will be used if
the fake proxy doesn't report an address.
@andersk
Copy link
Member

andersk commented Apr 26, 2013

This will result in a MOSH SSHIP line spewed to the console when an old client connects to a new server.

(It might be wise for us to develop a framework for adding extensions to the startup protocol without this kind of breakage.)

@cgull
Copy link
Member

cgull commented May 20, 2015

There's a somewhat hackish way around this: mosh could do ssh $host 'echo MOSH SSHIP SSH_CONNECTION; mosh ...'

cgull added a commit to cgull/mosh that referenced this pull request Jan 17, 2016
This is an alternate way to resolve mobile-shell#417 without affecting
backward compatibility.
cgull added a commit to cgull/mosh that referenced this pull request Mar 20, 2016
This is an alternate way to resolve mobile-shell#417 without affecting
backward compatibility.
@cgull cgull closed this in dbbed24 Mar 31, 2016
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.

3 participants