feat(ssh): make inactivity timeout configurable

[NefixEstrada]

With this change, the SSH inactivity timeout is now configurable. By default, is set at 5 minutes (300 seconds), which was the already existing value.

Solves #943

[Eugeny]

Thank you!

[przemekjagielski]

Are you sure, this change is workly? I tested this change and session was killed after 5 minute.

[przemekjagielski]

@Eugeny

[NefixEstrada]

Are you sure, this change is workly? I tested this change and session was killed after 5 minute.

We've had this in production for some weeks and is working correctly. Did you change the configuration value in the YAML file?

[przemekjagielski]

ssh: enable: true listen: "0.0.0.0:2222" keys: /data/ssh-keys host_key_verification: prompt inactivity_timeout: 86400

[NefixEstrada]

ssh: enable: true listen: "0.0.0.0:2222" keys: /data/ssh-keys host_key_verification: prompt inactivity_timeout: 86400

It needs to be in a valid humantime format: https://docs.rs/humantime/latest/humantime/fn.parse_duration.html

For example, we've set it up as inactivity_timeout: 30m

[przemekjagielski]

I changed this but again it not works. I see, PR have status failed. Maybe this change is not in main branch?

[przemekjagielski]

Any update? @NefixEstrada @Eugeny

[Eugeny]

@przemekjagielski post your config file

[przemekjagielski]

`

sso_providers: []
recordings:
enable: true
path: /data/recordings
external_host: ~
database_url: "mysql://warpgate:PASSWORD@mariadb/warpgate"
ssh:
enable: true
listen: "0.0.0.0:2222"
keys: /data/ssh-keys
host_key_verification: prompt
inactivity_timeout: 30m
http:
enable: true
listen: "0.0.0.0:8888"
certificate: /data/tls.certificate.pem
key: /data/tls.key.pem
trust_x_forwarded_headers: false
mysql:
enable: true
listen: "0.0.0.0:33306"
certificate: /data/tls.certificate.pem
key: /data/tls.key.pem
log:
retention: 7days
send_to: ~
config_provider: database`

[NefixEstrada]

Here's my configuration file for ssh:

ssh:
  enable: true
  listen: "0.0.0.0:2222"
  keys: /var/lib/warpgate/ssh-keys
  host_key_verification: prompt
  inactivity_timeout: 30m

Which version are you using? Did you compile from the main branch? Because there hasn't been a release since december, which doesn't have the changes in this PR