Skip to content

Commit

Permalink
ssh domains: use default local echo threshol
Browse files Browse the repository at this point in the history
The implementation of this is a bit gnarly, because we have
an optional value with a default that *is* populated by the
dynamic config stuff, but that isn't part of the derive(Debug)
impl.

closes: #5547
  • Loading branch information
wez committed Jul 15, 2024
1 parent fe17be2 commit ae9cdad
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions config/src/ssh.rs
Original file line number Diff line number Diff line change
Expand Up @@ -119,13 +119,15 @@ impl SshDomain {
name: format!("SSH:{host}"),
remote_address: host.to_string(),
multiplexing: SshMultiplexing::None,
local_echo_threshold_ms: default_local_echo_threshold_ms(),
..SshDomain::default()
});

mux_ssh.push(Self {
name: format!("SSHMUX:{host}"),
remote_address: host.to_string(),
multiplexing: SshMultiplexing::WezTerm,
local_echo_threshold_ms: default_local_echo_threshold_ms(),
..SshDomain::default()
});
}
Expand Down
2 changes: 2 additions & 0 deletions docs/changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,8 @@ As features stabilize some brief notes about them will accumulate here.
`pixel_height`.
* ssh: we now explicitly kill and reap the `ProxyCommand` associated
with an ssh session. Thanks to @daaku! #5494 #5479
* `default_ssh_domains()` didn't use the default local echo threshold
for ssh domains. #5547

#### Updated
* Bundled conpty.dll and OpenConsole.exe to build 1.19.240130002.nupkg
Expand Down

0 comments on commit ae9cdad

Please sign in to comment.