Skip to content

cli: off-by-one in dynamic-port-range validation #7759

@adborden

Description

@adborden

Problem

CHANGELOG refers to the minimum port range as 25 ports in agave v3.0.0.

If you run the validator with --dynamic-port-range 8000-8024 (25 ports), it fails with the error:

error: Invalid value for '--dynamic-port-range <MIN_PORT-MAX_PORT>': Port range is too small. Try --dynamic-port-range 8000-8025

The validation is testing for 26 minimum ports, not 25.

Proposed Solution

Use MINIMUM_VALIDATOR_PORT_RANGE_WIDTH - 1 in the validation check and in the error message in port_range_validation.

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions