umputun / rlb-stats / 28681985834

84%

master: 83%

Build Type

Pull #60

github

Committed by paskal

Commit Message

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.

Pull Request Pull Request #60: fix: correct /api/candle aggregate/max_points precedence

Coverage Stats

8 of 8 new or added lines in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

319 of 383 relevant lines covered (83.29%)

10.58 hits per line