whats the best way to convert this code to a suspend method instead of returning a future?

fun createCaptureServer(port: Int): CompletableFuture<String> {
        val future = CompletableFuture<String>()
        val undertow = Undertow.builder().addHttpListener(port, "127.0.0.1").setHandler {
            future.complete(it.queryString)
        }.build()
        undertow.start()
        return future.thenApplyAsync {
            undertow.stop()
            it
        }
    }

possibly using ktor instead of undertow.

if you're using ktor, it basically just becomes a normal function -- ktor's http methods are all suspend

because it is a callbac I think this applies https://stackoverflow.com/questions/48552925/existing-3-function-callback-to-kotlin-coroutines/48562175#48562175

yes, you could do it that way as well

thanks guys!

use

try-finally

(assuming you go the

suspendCancellableCoroutine

route)

why is

suspendCancellableCoroutine

better? when would it cancel?

if you cancel the Job that the suspend function is a part of, then it properly cancels and throws an exception

suspend fun createCaptureServer(port: Int): String {
        val result =  suspendCoroutine<String>{cont->
            val undertow = Undertow.builder().addHttpListener(port, "127.0.0.1").setHandler {
                cont.resume(it.queryString)
            }.build()
            undertow.start()
        }
        undertow.close()
        return result
    }

this is what i came up with, but it does not work that way because undertow is inside the callback, so i would have to make it nullable and a var. which is not so great

i'd be concerned about leaking the server in the case of an exception