Skip to content
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

refactor(ssh): use auth handlers to start SSH connection #3583

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

henrybarreto
Copy link
Member

The SSH's ShellHub server is the gateway between the user and its device. Each device added to ShellHub, connects to the gateway, allowing ShellHub's SSH server to be it manager. When a client wants to connect to them device, this request comes first into our SSH server and, after ward, to the device itself.

As our server is also an SSH server, what makes the SSH client understands that the user first connects to a server and is redirected to another one, causing problems with some SSH clients' authentication detection.

In order to solve it, we, now, connect to the device before starting the session itself, what makes this redirect seamless to the SSH clients.

Refs to #3240

The SSH's ShellHub server is the gateway between the user and its
device. Each device added to ShellHub, connects to the gateway, allowing
ShellHub's SSH server to be it manager. When a client wants to connect
to them device, this request comes first into our SSH server and, after
ward, to the device itself.

As our server is also an SSH server, what makes the SSH client
understands that the user first connects to a server and is redirected
to another one, causing problems with some SSH clients' authentication
detection.

In order to solve it, we, now, connect to the device before starting the
session itself, what makes this redirect seamless to the SSH clients.

Refs to #3240
@gustavosbarreto gustavosbarreto merged commit 70f3eca into master Mar 6, 2024
8 checks passed
@gustavosbarreto gustavosbarreto deleted the fix/ssh_auth_on_handler branch March 6, 2024 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants