Skip to content

Conversation

@kcoopermiller
Copy link
Member

@kcoopermiller kcoopermiller commented Dec 10, 2025

Note

Introduces SSH connectivity and protocol-aware port exposure across SDK and CLI.

  • CLI: New prime sandbox ssh command to create, authorize, and clean up SSH sessions; expose now supports --protocol (HTTP/TCP/UDP) and shows external endpoints; list-ports can list all exposures across sandboxes.
  • SDK: ExposePortRequest accepts protocol; ExposedPort includes external_port and external_endpoint; new SSHSession model; SandboxClient/AsyncSandboxClient add list_all_exposed_ports, create_ssh_session, and close_ssh_session; expose accepts protocol.
  • Examples: Adds sandbox_port_expose_demo.py demonstrating HTTP and TCP exposure with verification.
  • Exports: Exposes SSHSession in package __init__.

Written by Cursor Bugbot for commit 175fd61. This will update automatically on new commits. Configure here.

@kcoopermiller kcoopermiller changed the base branch from main to feature/tcp December 10, 2025 17:29
@kcoopermiller kcoopermiller changed the base branch from feature/tcp to main December 10, 2025 19:53
@kcoopermiller kcoopermiller changed the base branch from main to feature/tcp December 10, 2025 19:53
@kcoopermiller kcoopermiller changed the base branch from feature/tcp to main December 10, 2025 19:53
@kcoopermiller kcoopermiller marked this pull request as ready for review December 30, 2025 15:51
print(f" TLS Socket: {tcp_exposure.tls_socket}")
if tcp_exposure.external_port:
print(f" External Port: {tcp_exposure.external_port}")
time.sleep(120)
Copy link

Choose a reason for hiding this comment

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

Excessive 120-second delay in demo file

The demo has a hardcoded time.sleep(120) (2 minutes) after exposing the TCP port, which is 12x longer than the 10-second sleep used after HTTP port exposure. This appears to be leftover debug or testing code that makes the demo unnecessarily slow to run. A shorter delay (similar to the HTTP case) would be more appropriate for a demo file.

Fix in Cursor Fix in Web

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