minvws / nl-kat-coordination

Repo nl-kat-coordination for minvws
European Union Public License 1.2
123 stars 55 forks source link

Refactor `/task` and `/queues` endpoint to support `GET` parameter instead of `POST` body in scheduler #3161

Open jpbruinsslot opened 3 months ago

jpbruinsslot commented 3 months ago

Now the scheduler endpoint for /tasks allows for certain query parameters and a body payload to be used. This can lead to confusion when certain query parameters are duplicated within the request body. We can solve this in two ways:

  1. All filtering should be done in the body payload
  2. All filtering should be done with query parameters

From these two options, the second one seems the most straightforward one. This in part since it will adhere to API conventions of other services within OpenKAT. We do however need to decide upon how we would structure our queryparams especially when regarding complex queries.

Related: https://github.com/minvws/nl-kat-coordination/issues/3162