Skip to content

increaase pipeEstablishingTimeout to 60s #321

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dtrifiro
Copy link

@dtrifiro dtrifiro commented Mar 25, 2025

When using encrypted ssh keys, ssh-agent or an external authenticator (e.g. yubikey), there might be an time interval in which the user has to perform a manual action (typing ssh key password, approving the agent connection, physically touching the yubikey), which will take longer than the 3s timeout that is in place, causing the
connection to be closed with no further explanation:

$ ssh <session_id>@uptermd.upterm.dev
Enter passphrase for key '/home/dtrifiro/.ssh/id_ed25519':
Connection closed by 37.16.25.99 port 22

Increasing the timeout to 60s fixes this issue.

When using encrypted ssh keys, ssh-agent or an external authenticator
(e.g. yubikey), there might be an time interval in which the user
has to perform a manual action (typing ssh key password, approving
the agent connection, physically touching the yubikey), which will
take longer than the 3s timeout that is in place, causing the
connection to be closed with no further explanation:

```
$ ssh <session_id>@uptermd.upterm.dev
Enter passphrase for key '/home/dtrifiro/.ssh/id_ed25519':
Connection closed by 37.16.25.99 port 22
```

Increasing the timeout to 60s fixes this issue.
@dtrifiro dtrifiro force-pushed the increase-pipe-timeout branch from 3f0e5d2 to d160824 Compare March 31, 2025 10:41
@dtrifiro
Copy link
Author

Hey @owenthereal, sorry to bug you again, but would you mind taking a look? Thanks once again.

@dtrifiro dtrifiro changed the title increaase pipeEstablishingTimeout to 30s increaase pipeEstablishingTimeout to 60s Mar 31, 2025
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.

1 participant