[bug] SubmitCallback and GetCallBack functions must be coroutines.

[jiyoonie9]

this bug was found during an attempt at writing an example http-server in kotlin using TbdexHttpServer.

fun main() {
  val serverConfig = TbdexHttpServerConfig(
    port = 8080,
    pfiDid = DidDht.create(InMemoryKeyManager()).uri,
    offeringsApi = FakeOfferingsApi(),
    exchangesApi = FakeExchangesApi()
  )

  val tbdexServer = TbdexHttpServer(serverConfig)


  tbdexServer.submit(SubmitKind.rfq) { call, messageKind, offering ->
    println("Saving RFQ...")
    println("Validating RFQ.claims against Offering...")
    println("Generating a Quote...")
    println("Saving a Quote...")

    // this line does not compile
    // error message: Suspend function 'respond' should be called only from a coroutine or another suspend function
    call.respond(HttpStatusCode.Accepted)
  }

  tbdexServer.start()

}

in order to return a response, the callback must call ApplicationCall.respond() which is a coroutine function. therefore, the callback must also be a coroutine.

changing typealias SubmitCallback = (ApplicationCall, MessageKind, Offering?) -> Unit to typealias SubmitCallback = suspend (ApplicationCall, MessageKind, Offering?) -> Unit seems to do the trick, but needs testing to confirm.

[chris-tbd]

@jiyoontbd closing