Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate to kotlinx-io and ktor 3.0 #280

Open
wants to merge 1 commit into
base: deps
Choose a base branch
from
Open

Migrate to kotlinx-io and ktor 3.0 #280

wants to merge 1 commit into from

Conversation

whyoleg
Copy link
Member

@whyoleg whyoleg commented Nov 11, 2024

Fixes #277 and probably prepares ground for #110 (in some way, if it's possible).
Note: it's breaking change!

  • rsocket-core now depends only on kotlinx.coroutines and kotlinx.io!
  • ByteReadPacket was replaced with Buffer and could be revisited soon for Payload/Metadata API in favor of ByteString or Source
    • Buffer was done to ease migration: it has size and copy support which is needed for Payload API
  • ktor transports updated to 3.0.1
    • wasm and all native targets support will be implemented in next PR
  • leak tracking in tests was removed, as there is no way to do it until Leak tracing mechanism Kotlin/kotlinx-io#144
    • at this moment it's less of an issue, as kotlinx.io.Buffer is backed by ByteArray and so is fully managed by GC

@whyoleg whyoleg self-assigned this Nov 11, 2024
Copy link

Test Results

  878 files   -   8      1 errors    877 suites   - 9   44m 57s ⏱️ - 28m 11s
3 873 tests  -  48  3 128 ✅  -  33    739 💤  - 12   6 ❌  - 3 
8 304 runs   - 216  6 674 ✅  - 160  1 616 💤  - 54  14 ❌  - 2 

For more details on these parsing errors and failures, see this check.

Results for commit 9feca27. ± Comparison against base commit 353fe93.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant