Skip to content

fix(clients): wait for connection before accepting commands - #2740

Merged
bmorcelli merged 1 commit into
BruceDevices:devfrom
Pedro-Jesus-Fuentes-Morcillo:fix/ssh-telnet-connect-ux
Aug 2, 2026
Merged

fix(clients): wait for connection before accepting commands#2740
bmorcelli merged 1 commit into
BruceDevices:devfrom
Pedro-Jesus-Fuentes-Morcillo:fix/ssh-telnet-connect-ux

Conversation

@Pedro-Jesus-Fuentes-Morcillo

Copy link
Copy Markdown
Contributor

Proposed Changes

Show a "Connecting..." message and block command input until the SSH or Telnet worker reports the session as ready or closed. Before this change the terminal opened right away as if connected, so a failed connection or a wrong login was only shown after the user had already typed one or more commands, and what they typed was lost.

The wait runs before the input loop in runSessionUiLoop, which is shared by SSH and Telnet, so both get the same behavior. You can still cancel with the back button while it is connecting.

Types of Changes

Enhancement.

Verification

Tested on a LilyGo T-Embed CC1101 Plus against both a Linux host and a Windows 11 host with OpenSSH.

  • Wrong host or wrong password: the error is shown right away and no command prompt is offered.
  • Correct credentials: a short "Connecting..." message is shown, then the terminal opens ready for input.

Testing

This was verified manually on hardware. Compiles across all board envs.

Linked Issues

Closes #2739

User-Facing Change

SSH and Telnet now show a "Connecting..." message and wait for the connection before accepting commands, so a failed connection or login is reported right away instead of after typing

Further Comments

The "Connecting..." message uses displayTextLine so it follows the active theme, like the other status messages in the UI.

Show a "Connecting..." message and block command input until the SSH
or Telnet worker reports the session as ready or closed. A failed
connection or login is now surfaced right away instead of after the
user has already typed one or more commands.
@bmorcelli
bmorcelli merged commit 1326430 into BruceDevices:dev Aug 2, 2026
7 checks passed
0Zane pushed a commit to 0Zane/firmware that referenced this pull request Aug 3, 2026
…ices#2740)

Show a "Connecting..." message and block command input until the SSH
or Telnet worker reports the session as ready or closed. A failed
connection or login is now surfaced right away instead of after the
user has already typed one or more commands.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants