Add checks aggregate route

[maudetes]

It allows to group_by on a column and count the check occurrences for a selected date.
Useful for debugging current checks trends, ex:

• http://localhost:8000/api/checks/aggregate?group_by=domain&created_at=today
• http://localhost:8000/api/checks/aggregate?group_by=status&created_at=today
• http://localhost:8000/api/checks/aggregate?group_by=headers-%3E%3E%27content-type%27&created_at=2024-09-05

Notes

• This is probably not the most REST logic route, so I would be interested in alternative implementations.
• Param parsing is very naive currently and would fail on non valid values
• I was wondering if we should follow these group_by indicators in a Munin plugin instead?

[bolinocroustibat]

LGTM, great job! A few optional questions/remark if we want to go further.

[bolinocroustibat]

What about /api/checks?group_by={column}&created_at={date} instead? Maybe it is as explicit?
Isn't the aggregation an internal mecanism which doesn't need to be explicited in the URL?
Or, do we need to have an aggregate in the URL to differentiate other future type of search parameters?
Just raising the question, but might be more difficult to separate the routes in the code that way.

[maudetes]

I wanted to start with this approach, but I had some considerations:

• the payload would vary greatly depending on group_by being used or not.
• It also does not actually return checks objects per se, it returns count aggregation.
• Since we did not have any existing GET /api/checks, it seemed weird to require a group_by parameter on this route. It made me think that /api/checks/latest and /api/checks/all should probably be /api/checks with additional relevant params (pagination would do actually). Thus going with /aggregate path for now. A refactoring may follow to standardize /api/checks routes maybe.

[bolinocroustibat]

Fair enough then, let's keep it that way. Thanks for the explanation!

[maudetes]

I am leaving it unresolved as it may be useful reference for future readers.

[bolinocroustibat]

By the way, it's much better than this:

https://x.com/excid3/status/1836042970472075673