npm / download-counts

Background jobs and a minimal service for collecting and delivering download counts
329 stars 27 forks source link

Not handling outOfRange flag for range/ endpoint #25

Closed mrjoelkemp closed 7 years ago

mrjoelkemp commented 8 years ago

It's currently possible to retrieve data for a range much wider than a month.

  1. This should likely be patched if the max range should still be enforced
  2. If you're okay with allowing a wider range, there is an issue with an incomplete json response for the payload from the following request.

The incomplete response is likely a non-issue once the range is capped though.

seldo commented 7 years ago

Ranges and size of bulk queries are now correctly caped.