Search endpoints should validate dates individually about ocpi-toolkit HOT 4 OPEN

yeah ... that's my bad for not debuging this more deeply first ... I didn't realize that we are already using Instant at that point, so the actual date validation is done by Server when parsing. The issue is, that those throw DateTimeParseException which leads to a HTTP 500, instead of a client side error.

                        dateFrom = dateFrom?.let { Instant.parse(it) },
                        dateTo = dateTo?.let { Instant.parse(it) },

Simply adding another catch on the server side wouldn't help, as we wouldn't be able to distinguish between client and server error, in case this happens somewhere else. Similar issue with the .toInt() calls.

I'd suggest we add a few helpers for extracting specific types from params, so that we can produce proper response ... eg

fun HttpRequest.queryParamAsInstant(key: String): Instant?
fun HttpRequest.queryParamAsInt(key: String): Int?

they could convert numberformat and date format exceptions, and turn them into validation exceptions

from ocpi-toolkit.

You are absolutely right, this is an issue

from ocpi-toolkit.

Actually, what should we validate if only dateFrom is set or dateTo is set? validateDates currently only checks if to is after from

from ocpi-toolkit.

You are right, your suggestion is great, we should implement it

from ocpi-toolkit.