Ktor 3.0 support

[lexa-diky]

Is your feature request related to a problem? Please describe.
JB recently released Ktor 3.0.0-betta1. Currently Ktor does not support it, JB changed API. In particular access to monitor now done via Application not Environment, maybe there is more incompatibilities.

Describe the solution you'd like
Separate ktor-3 module to have both ktor 2.* and 3.*

Target Koin project
ktor

[Nek-12]

Can second this, but I would prefer a simple dependency upgrade instead of separate artifact

[Nek-12]

The workaround for now is to copy everything from the koin Ktor sources and fix the compile time error

[lexa-diky]

If koin goes with single module approach, it will break comp with ktor-2, and I guess bunch of people will not update for a while

[Nek-12]

The only updates for the past months have been dependency updates... The whole library is 2 files

[arnaudgiuliani]

Yes, great topic coming 👍

[Nek-12]

Can you please add ktor 3.0 support first before you refactor, add new stuff, and break something with the update you described in your article? @arnaudgiuliani

Changing one line of code, that is, so that we don't wait till Q4 2024 for this to get fixed

[arnaudgiuliani]

something I can setup for 3.6.0 - Q1
Seems that we need to separate both ktor & ktor3

[FlyingYu-Z]

Is there any solution now?

[JoonasC]

@arnaudgiuliani I would appreciate a solution for this, as I am currently unable to use SSE (server sent events) in my application due to it only being supported in Ktor 3.0.0. Let me know if I can speed up the work, maybe by making a PR?

[fluxxion82]

I'm am interested in this as well and will be watching for updates.

[Nek-12]

@fluxxion82 Just copy all sources of the Ktor module to your project. This hasn't been addressed in almost a year and the author has no intention of doing so.

[KevinMeneses]

it would be nice to get support for this

[hashir1296]

Still waiting for this. No solutions of as of yet?

[Nek-12]

Like I said, just copy and paste all sources of Koin-ktor module into your project and fix the breaking changes. Already using this approach in prod

[hashir1296]

Like I said, just copy and paste all sources of Koin-ktor module into your project and fix the breaking changes. Already using this approach in prod

Pardon but can you please briefly describe what do you mean by this? Earlier we were using koin as:

install(Koin) {
        slf4jLogger()
        modules(appModule)
    }

How do i migrate from this?

[hashir1296]

Like I said, just copy and paste all sources of Koin-ktor module into your project and fix the breaking changes. Already using this approach in prod

Pardon but can you please briefly describe what do you mean by this? Earlier we were using koin as:

install(Koin) {
        slf4jLogger()
        modules(appModule)
    }

How do i migrate from this?

Figured it out, basically

1. Goto this link: https://github.com/InsertKoinIO/koin/tree/main/projects/ktor/koin-ktor
2. Manually copy/paste all files under koin-ktor package (I copied only KoinPlugin)
3. In your install block, use your koin plugin instead of ktor's

Warning: This is a workaround, with this:

1. You will lose out on future Ktor or Koin updates
2. You’ll need to keep an eye on both the Koin and Ktor projects for official updates that may render this workaround obsolete.

[chrisjenx]

Yeah this is now broken on ktor 3.x:

java.lang.IncompatibleClassChangeError: Found interface io.ktor.server.routing.Routing, but class was expected
	at org.koin.ktor.ext.RoutingExtKt.getKoin(RoutingExt.kt:74)

Will need a ktor3 package, really the impl is basically the same, but they are incompatible due to changes to classes in ktor3

[chrisjenx]

PR here: #1978

There is jar with the ktor3 target if you need it for now

[chrisjenx]

@arnaudgiuliani if no objections, can we fast track to 4.0? Happy to maintain the module if you ping me. (And get in the next RC?)