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

tiny change, fix incorrect reference to SSH config variable "ClientAlive... #1065

Closed
wants to merge 1 commit into from

Conversation

@hjwp
Copy link

@hjwp hjwp commented Feb 10, 2014

...Interval"

Confusingly, the SSH client's keepalive parameter is "ServerAliveInterval" (ie, how often to check the server is alive), whereas "ClientAliveInterval" is something you would set server-side.

https://unix.stackexchange.com/questions/3026/what-do-the-options-serveraliveinterval-and-clientaliveinterval-in-sshd-conf

http://unixhelp.ed.ac.uk/CGI/man-cgi?ssh_config+5

http://unixhelp.ed.ac.uk/CGI/man-cgi?sshd_config+5

…iveInterval"

Confusingly, the SSH client's keepalive parameter is "ServerAliveInterval" (ie, how often to check the server is alive), whereas "ClientAliveInterval" is something you would set server-side.

https://unix.stackexchange.com/questions/3026/what-do-the-options-serveraliveinterval-and-clientaliveinterval-in-sshd-conf

http://unixhelp.ed.ac.uk/CGI/man-cgi?ssh_config+5

http://unixhelp.ed.ac.uk/CGI/man-cgi?sshd_config+5
@bitprophet
Copy link
Member

@bitprophet bitprophet commented Feb 28, 2017

Wants backport to 1.10.x or thereabouts (I can do this next bugfix pass), otherwise looks good - thanks!

@bitprophet
Copy link
Member

@bitprophet bitprophet commented Apr 24, 2017

Unclear why I only replied to this in 2017 when original was 2014...presumably somebody linked me to it or something 😊

Been so long the original patch predates the site split; manually applied, will credit.

bitprophet added a commit that referenced this pull request Apr 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants