fix: use normal port ranges by Flo4604 · Pull Request #5271 · unkeyed/unkey
What does this PR do?
Expands the port range validation for runtime settings to allow the full valid TCP port range (1-65535) instead of the previously restricted range (2000-54000). This change affects both the schema validation and the form input constraints.
Fixes # (issue)
If there is not an issue for this, please create one first. This is used to tracking purposes and also helps us understand why this PR exists
Type of change
- Bug fix (non-breaking change which fixes an issue)
- Chore (refactoring code, technical debt, workflow improvements)
- Enhancement (small improvements)
- New feature (non-breaking change which adds functionality)
- Breaking change (fix or feature that would cause existing functionality to not work as expected)
- This change requires a documentation update
How should this be tested?
- Test that port values from 1-65535 are now accepted in the port settings form
- Test that port values outside this range (0, 65536+) are properly rejected
- Verify that the form validation displays appropriate error messages for invalid ports
- Test that existing port configurations within the old range (2000-54000) continue to work
Checklist
Required
- Filled out the "How to test" section in this PR
- Read Contributing Guide
- Self-reviewed my own code
- Commented on my code in hard-to-understand areas
- Ran
pnpm build - Ran
pnpm fmt - Ran
make fmton/godirectory - Checked for warnings, there are none
- Removed all
console.logs - Merged the latest changes from main onto my branch with
git pull origin main - My changes don't cause any responsiveness issues
Appreciated
- If a UI change was made: Added a screen recording or screenshots to this PR
- Updated the Unkey Docs if changes were necessary