umputun / rlb-stats / 32349662605

88%

master: 88%

push

github

fix: correct /api/candle aggregate/max_points precedence

The README states aggregate overrides max_points, but the handler parsed
max_points last so it actually won. Restructure so an explicit aggregate is
used and max_points is skipped entirely when present, matching the docs.

While here, fix two adjacent max_points bugs surfaced in review:
- max_points=0 divided by zero; now rejected (must be 1..255)
- ParseInt(_, 8) rejected the documented max of 255 and accepted negatives;
  use ParseUint with an explicit 1..255 range check

Document the 1..255 range and the aggregate-overrides note in the README.

6 of 7 new or added lines in 1 file covered. (85.71%)

627 of 713 relevant lines covered (87.94%)

107.69 hits per line