fix(port): raise default base port to 60000, validate full block - #12
Merged
Conversation
macOS launchd occupies port 8021, causing collisions by the 3rd workspace set. Move default base_port from 8000 to 60000 to avoid system-reserved ports entirely. Port allocation now checks every port in a candidate block (not just the first) via BlockFree(), skipping blocks where any port is occupied. Also guards against allocating above port 65535.
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
1 participant
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.
What
Raises the default
base_portfrom8000to60000and validates every port in a candidate block before allocating it.Why
macOS
launchdoccupies port 8021, causing collisions by the 3rd workspace set with the old default. The 60000 range avoids system-reserved ports entirely.How
base_portchanged from8000to60000inapplyDefaults()BlockFree(startPort, count)function checks all ports in a block, not just the first — if any port is occupied, the block is skipped