upgrade to ktor 3

[TheDome0]

🔗 Related Issues

#2037

[TheDome0]

there are still three tests failing, two of them due to an NPE in KtorGraphQLRequestParser where I'm not really sure what's happening

[ronjunevaldoz]

any update on this one :)

[TheDome0]

No, because in v9 I ran into a different problematic issue where you cannot have a class which implements an interface as a response for graphql queries.

ConflictingTypesException: Conflicting class names in schema generation [class com.example.TestError, class com.example.TestError]

sealed interface Err {
    val message: String
}

data object TestError: Err {
    override val message: String = "Test error"
}

class TestQueries : Query {
    suspend fun test(): TestError = TestError
}

Removing the message property from the interface fixes it, but there are also other situations like implementing multiple interfaces (typical error union setup) where this happens.

I tried adding a custom willgenerateGraphQLType hook but then I'm getting You have redefined the type 'TestError' from being a 'GraphQLObjectType' to a 'GraphQLObjectType'.

And this does not happen in v8 / prior to Kotlin 2.0

https://github.com/TheDome0/graphql-repros