Skip to content

Validate proxy and gateway port settings - #23

Open
rissrice2105-agent wants to merge 1 commit into
profullstack:mainfrom
rissrice2105-agent:fix/validate-port-settings
Open

Validate proxy and gateway port settings#23
rissrice2105-agent wants to merge 1 commit into
profullstack:mainfrom
rissrice2105-agent:fix/validate-port-settings

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Closes #22

Summary

  • require complete decimal port strings instead of accepting parseInt prefixes
  • reject zero, negative, fractional, non-numeric, and values above 65535
  • preserve whitespace trimming and documented defaults
  • cover malformed inputs and both valid range boundaries

Validation

  • node --test tests/config.test.ts (2/2)
  • focused TypeScript check for lib/config.ts and tests/config.test.ts
  • git diff --check

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.

Port settings accept malformed and out-of-range values

1 participant