docs: sandbox published ports survive restarts#25386
Draft
dvdksn wants to merge 2 commits into
Draft
Conversation
Published ports are now persisted and re-published when a sandbox or the daemon restarts (docker/sandboxes#3614). Explicit host ports are reused; OS-assigned host ports get a fresh port on each start. Update the port publishing notes in the sandboxes usage guide to reflect persistence and the new wildcard unpublish-by-sandbox-port behavior. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
The changes correctly update the port-publishing behavior documentation to reflect that published ports now persist across sandbox and daemon restarts. The updated content is accurate, clear, and consistent with the existing document style. No high- or medium-severity issues found.
Replace the bulleted 'a few things to keep in mind' list with concise prose paragraphs and drop the bold-term format. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Document persistent port publishing.
The "Accessing services in the sandbox" section of the sandboxes usage guide previously stated that published ports are not persistent and must be re-published after a restart. That's no longer true. This PR updates the port-publishing notes to describe the new behavior:
--unpublish 3000(sandbox-port-only, no host port) is now accepted and removes every host port mapped to that sandbox port — useful for ephemeral ports whose host number you don't know. The previous note said this form was rejected.Related issues
🤖 Generated with Claude Code