Skip to content

add sf vm ssh#111

Merged
coffinsfcompute merged 1 commit into
mainfrom
coffin/ssh
Apr 11, 2025
Merged

add sf vm ssh#111
coffinsfcompute merged 1 commit into
mainfrom
coffin/ssh

Conversation

@coffinsfcompute
Copy link
Copy Markdown
Contributor

No description provided.

@semanticdiff-com
Copy link
Copy Markdown

semanticdiff-com Bot commented Apr 11, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  deno.lock Unsupported file format
  package.json  0% smaller
  src/helpers/urls.ts  0% smaller
  src/lib/ssh.ts  0% smaller
  src/lib/vm.ts  0% smaller

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

This PR adds SSH functionality to the CLI, allowing users to connect directly to virtual machines using the sf vm ssh command.

  • Added new ssh.ts module implementing SSH connection handling with proper command escaping via the shescape library
  • Added vms_ssh_get endpoint in urls.ts to fetch SSH connection details from the API
  • Integrated SSH functionality into VM management through registerSsh() in vm.ts
  • Implemented secure handling of SSH host keys with optional verification
  • Added support for username specification in the format USERNAME@VM_ID for SSH connections

4 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

Comment thread src/lib/ssh.ts
@coffinsfcompute coffinsfcompute merged commit f34e2c8 into main Apr 11, 2025
2 checks passed
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