Parse duration config strings like 1s into kotlin.time.Duration

[dpozinen]

With an application.yml like so:

my-app:
  duration: 4h

and a class property like so:

import kotlin.time.Duration

@Service
class MyClass(
   @Value("\${my-app.duration}") private val followDuration: Duration
)

getting

Unsatisfied dependency expressed through constructor parameter 3: Failed to convert value of type 'java.lang.String' to required type 'long'; For input string: "PT4h"
...
Caused by: org.springframework.beans.TypeMismatchException: Failed to convert value of type 'java.lang.String' to required type 'long'; For input string: "PT4h"
...
Caused by: java.lang.NumberFormatException: For input string: "PT4h"

Spring boot version: 3.1.4
Kotlin: 1.9.20-Beta

[Sap004]

@dpozinen I think you are facing this issue because

converter provided by Spring Boot works with java.time.Duration and not kotlin.time.Duration

[dpozinen]

@Sap004 yes I assume that is the case as well, hence the feature request

[Sap004]

@dpozinen I am currently working on addressing this issue. I'll be submitting a pull request shortly for review

[wilkinsona]

I'd like to keep this as waiting for triage until we know what's necessary. If it requires further Kotlin code we'll need to consider if the potential benefits justify that complexity. There's also the Framework side of things to consider such as spring-projects/spring-framework#30396.

[wilkinsona]

@Sap004 please be aware that, unfortunately, there's a good chance that a PR will be declined at this time. Things are moving towards duration parsing being handling in Spring Framework. spring-projects/spring-framework#30396 is tracking that and we may not want to add any more Boot-specific features in this area until that PR's been resolved one way or another.

[philwebb]

We discussed this today as a team and decided that we don't think this is a good idea for us to implement. The fact that our current DurationStyle enum is tied to java.time. types along with the fact we have a @DurationUnit annotation that clashes with a similarly named Kotlin type makes us think that this will be confusing for users.

Since Kotlin has an extension method to convert a java.time.Duration to a kotlin.time.Duration we recommend that the Java type is used for binding and extension method be used whenever a Kotlin type is needed.