Skip to content

feat(vercel-sandbox,sandbox): fix ssh connection hanging#98

Merged
marc-vercel merged 1 commit intonamed-sandboxesfrom
fix-ssh-connection-hanging
Mar 17, 2026
Merged

feat(vercel-sandbox,sandbox): fix ssh connection hanging#98
marc-vercel merged 1 commit intonamed-sandboxesfrom
fix-ssh-connection-hanging

Conversation

@marc-vercel
Copy link
Collaborator

@marc-vercel marc-vercel commented Mar 17, 2026

When we do a ssh to a resumed sandbox, it works. But it does not work when trying to do a ssh into a new sandbox and it hanged.

There were two errors:

  1. We hanged forever without showing the error. This has been fixed.
  2. We could not connect because we did not pass the internal parameters (routes)

Testing

We have performed these tests:

❯ sandbox create
✅ Sandbox beige-leading-flea-4dExSp created.
   │ team: marc-codina-enhanced-vtest314
   ╰ project: my-sandbox-app

❯ sandbox ssh beige-leading-flea-4dExSp
$ sh
▲ /vercel/sandbox/ echo 'hi'
hi
▲ /vercel/sandbox/ exit
exit
╰▶ connection to ▲ beige-leading-flea-4dExSp closed.

❯ sandbox stop beige-leading-flea-4dExSp
✔ Stopping active session from beige-leading-flea-4dExSp

❯ sandbox ssh beige-leading-flea-4dExSp
$ sh
▲ /vercel/sandbox/ exit
exit
╰▶ connection to ▲ beige-leading-flea-4dExSp closed.

@vercel
Copy link

vercel bot commented Mar 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sandbox-sdk Ready Ready Preview Mar 17, 2026 3:49pm

@marc-vercel marc-vercel marked this pull request as ready for review March 17, 2026 15:56
@marc-vercel marc-vercel requested review from AndyW22 and QuiiBz March 17, 2026 15:57
@marc-vercel marc-vercel merged commit d4ac7da into named-sandboxes Mar 17, 2026
10 of 11 checks passed
@marc-vercel marc-vercel deleted the fix-ssh-connection-hanging branch March 17, 2026 16:29
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.

2 participants