Skip to content
This repository was archived by the owner on Aug 30, 2024. It is now read-only.

Conversation

sreya
Copy link
Collaborator

@sreya sreya commented May 19, 2021

  • Removed ServerAliveInterval and ServerAliveCountMax from config-ssh
    settings. These settings are opinionated about when idle sessions should disconnect
    and should be inherited from the user's global settings instead. 'coder
    config-ssh' should only write configuration options that are necessary
    for 'coder sh' and its variants to succeed.

@sreya sreya requested review from deansheather and kylecarbs May 19, 2021 19:25
Copy link
Member

@kylecarbs kylecarbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems fine to me.

sreya added 2 commits May 19, 2021 19:45
- Removed ServerAliveInterval and ServerAliveCountMax from config-ssh
  settings. These settings are opinionated about when idle sessions
  should disconnect and should be inherited from the user's global settings.
  'coder config-ssh' should only write configuration options that are
  necessary for 'coder sh' and its variants to succeed.
@sreya sreya merged commit bf7f71a into master May 20, 2021
@sreya sreya deleted the fix-sshconfig branch May 20, 2021 15:10
@jawnsy
Copy link
Contributor

jawnsy commented May 21, 2021

[ch12778]

@shortcut-integration
Copy link

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants