@gabrielerzinger Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@gabrielerzinger Thank you for signing the Contributor License Agreement!

@sdeleuze i saw that you were the one who recently changed those files, do you think this makes sense?

Thanks for your contribution, I need to have a deeper look but it seems to make sense indeed.

Hey @sdeleuze, are there any updates or blockers on this?

To expand on @gabrielerzinger's use-case, prior to 5.2 (when awaitExchange() was deprecated), it was possible to write the following code:

// Check whether the role already exists.
val existing = client.get().uri("/api/roles/${role.id}")
    .awaitExchange()
    .let {
        when (val status = it.statusCode()) {
            HttpStatus.OK -> it.awaitBodyOrNull<Role>()
            HttpStatus.NOT_FOUND -> null
            else -> {
                val resp = it.awaitBodyOrNull<String>()
                throw IllegalStateException("Unexpected response from '/api/roles/${role.id}': status=$status, body=$resp")
            }
        }
    }

Because awaitExchange { } doesn't have a nullable counterpart, it's not possible to represent the same logic. I was actually in the midst of upgrading a project when I came across this issue.

Let me know if there's any way I can help get this merged.