Cannot deserialize LocalDate in @QueryParam due to missing ParamConverterProvider

[snowdrop-bot]

Describe the bug
I'm using Jackson (via quarkus-resteasy-jackson) as ObjectMapper with the KotlinModule and the JavaTimeModule registered using an ObjectMapperCustomizer. When I have an endpoint with a @QueryParam of type LocalDate quarkus does not start.

Expected behavior
I expect resteasy to use the provided ObjectMapper to deserialize a String formated "YYYY-MM-DD" into an instance of LocalDate.

Actual behavior
Omitting the name and method signature for brevity I get

RESTEASY003875: Unable to find a constructor that takes a String param or a valueOf() or fromString() method for javax.ws.rs.QueryParam("adate") for basetype: java.time.LocalDate

To Reproduce
Steps to reproduce the behavior:

1. Use

        <dependency>
            <groupId>io.quarkus</groupId>
            <artifactId>quarkus-resteasy</artifactId>
        </dependency>
        <dependency>
            <groupId>io.quarkus</groupId>
            <artifactId>quarkus-resteasy-jackson</artifactId>
        </dependency>
        <dependency>
            <groupId>com.fasterxml.jackson.module</groupId>
            <artifactId>jackson-module-kotlin</artifactId>
        </dependency>
        <dependency>
            <groupId>com.fasterxml.jackson.datatype</groupId>
            <artifactId>jackson-datatype-jsr310</artifactId>
        </dependency>

2. Register a resource like

@GET
@Path("path")
fun broken( @QueryParam("adate") aDate: LocalDate) {}

3. Run a @QuarkusTest to try a start up the application.

Environment (please complete the following information):

• Output of uname -a or ver:
  Linux ninja-cgd 5.3.0-23-generic SpringJPA: Support spring.jpa.show-sql #25-Ubuntu SMP Tue Nov 12 09:22:33 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
• Output of java -version:
  openjdk version "11.0.5-ea" 2019-10-15
  OpenJDK Runtime Environment (build 11.0.5-ea+10-post-Ubuntu-0ubuntu1)
  OpenJDK 64-Bit Server VM (build 11.0.5-ea+10-post-Ubuntu-0ubuntu1, mixed mode, sharing)
• GraalVM version (if different from Java):
  not used
• Quarkus version or git rev:
  0.25.0
  Additional context

The problem is fixed by adding

@Provider
class LocalDateConverter : ParamConverterProvider {

    override fun <T: Any?> getConverter(rawType: Class<T>?, genericType: Type?, annotations: Array<out Annotation>?): ParamConverter<T>? {
        return if (rawType == LocalDate::class.java) {
            object: ParamConverter<T> {
                val mapper = ObjectMapper().also { JacksonObjectMapperCustomizer().customize(it) }
                override fun toString(value: T?): String {
                    return mapper.writeValueAsString(value)
                }
                override fun fromString(value: String?): T {
                    return mapper.readValue(value, LocalDate::class.java) as T
                }
            }
        } else null
    }
}

to the project. I was expecting that this kind of Provider would be added out of the box when using Jackson's JavaTimeModule.

---

quarkusio#5860

---

$upstream:5860$

[DarioArena87]

If it is needed i found a solution that does not involves using Jackson ObjectMapper (written in Groovy just for brevity). If not strictly for jackson maybe it could be useful for resteasy jsonb. I've not really stress tested it so i don't know if using "new" when returning the converters causes memory leaks.

class LocalDateConverter implements ParamConverter<LocalDate> {
    @Override
    LocalDate fromString(String value) {
        return LocalDate.parse(value)
    }

    @Override
    String toString(LocalDate value) {
        return DateTimeFormatter.ISO_LOCAL_DATE.format(value)
    }
}

class LocalDateTimeConverter implements ParamConverter<LocalDateTime> {

    @Override
    LocalDateTime fromString(String value) {
        return LocalDateTime.parse(value)
    }

    @Override
    String toString(LocalDateTime value) {
        return DateTimeFormatter.ISO_LOCAL_DATE_TIME.format(value)
    }
}

@Provider
class ParamsConverter implements ParamConverterProvider {
    @Override
    <T> ParamConverter<T> getConverter(Class<T> rawType, Type genericType, Annotation[] annotations) {
        switch(rawType) {
            case LocalDate: return new LocalDateConverter() as ParamConverter<T>
            case LocalDateTime: return new LocalDateTimeConverter() as ParamConverter<T>
            default: return null
        }
    }
}