Skip to content

fix: ssh connection to a workspace is established only once #76

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

Merged
merged 1 commit into from
Apr 10, 2025

Conversation

fioan89
Copy link
Collaborator

@fioan89 fioan89 commented Apr 9, 2025

  • right now ssh automatically connects to the workspace when a user selects it from the main env. page. However, if the user manually disconnects from the hamburger menu, the SSH is no longer automatically established afterward.

  • the reason for this behavior was that the connection request flag that triggered the ssh connection behind the scenes was never reset. In order to reset we have to know when a user hits the disconnect button.

  • fortunately Toolbox provides two callbacks, one before establishing the ssh connection and another one right after disconnect. Even though we don't know the reason for disconnect it's still a good opportunity to reset the request flag.

  • resolves SSH connection to a Workspace is established only once #38

- right now ssh automatically connects to the workspace when a user selects it from the main env. page.
  However, if the user manually disconnects from the hamburger menu, the SSH is no longer automatically
  established afterward.
- the reason for this behavior was that the connection request flag that triggered the ssh connection behind
  the scenes was never reset. In order to reset we have to know when a user hits the disconnect button.
- fortunately Toolbox provides two callbacks, one before establishing the ssh connection and another one right
  after disconnect. Even though we don't know the reason for disconnect it's still a good opportunity to
  reset the request flag.

- resolves #38
@fioan89 fioan89 requested review from matifali and bcpeinhardt April 9, 2025 19:55
@fioan89 fioan89 merged commit a509615 into main Apr 10, 2025
5 checks passed
@fioan89 fioan89 deleted the fix-ssh-connection branch April 10, 2025 18:14
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.

SSH connection to a Workspace is established only once
2 participants