Multiplatform #39

luca992 · 2022-06-06T06:35:15Z

A multiplatform port of the project. Kotlin and dependencies have been updated to the latest

Targets:

"macosX64", "macosArm64",
"iosArm64", "iosX64", "iosSimulatorArm64",
"tvosArm64", "tvosX64", "tvosSimulatorArm64",
"watchosArm32", "watchosArm64", "watchosX86", "watchosX64", "watchosSimulatorArm64",
"linuxX64"
"mingwX64"
"jvm"
"js"

Libraries ported:
okhttp -> ktor
moshi -> kotlinx.serialization

Multiplatform file support with okio

It's working well so far for me. I published a release here:

implementation("io.github.luca992:ipfs-api:0.17.0")

Todo:

Port jvm tests to multiplatform
Figure out if anything in the root build.gradle.kts was needed

Bumps [mockk](https://github.com/mockk/mockk) from 1.12.0 to 1.12.4. - [Release notes](https://github.com/mockk/mockk/releases) - [Commits](mockk/mockk@v1.12.0...1.12.4) --- updated-dependencies: - dependency-name: io.mockk:mockk dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>

for tests set responses serving json to have a header of setHeader("Content-Type", ContentType.Application.Json)

note ktor automatically mulitpart request headers look like: Content-Disposition: form-data; name=""; file; filename="temptestdir7160507879139843984%2Fsubdir" Content-Transfer-Encoding: binary Content-Type: application/x-directory Content-Length: 0 vs Perviously with OkHttp: Content-Disposition: file; filename="temptestdir14927010351886631568%2Fsubdir" Content-Transfer-Encoding: binary Content-Type: application/x-directory; charset=utf-8 Content-Length: 0

that way the library artifact has an appropriate name when publishing outside of jitpack

tests broken :/

BufferedSource.readByteArray clears the source buffer. Which was not my intention

https://youtrack.jetbrains.com/issue/KTOR-3840/OkHttp-Socket-timeouts-without-timeout-configured

dependabot bot and others added 30 commits May 11, 2022 09:57

update to kotlin 1.6.21 with multiplatform plugin for lib

1330fcd

remove moshi. use kotlinx serialization

bb70d3b

update dependencies

271d312

port callCmd to ktor

802d57b

for tests set responses serving json to have a header of setHeader("Content-Type", ContentType.Application.Json)

remove unused import

5074f4a

remove reflect

9ae9c4d

include quotes around filename for string

7f680c1

move code to common

fc0af5e

fix file add

38adc4e

move more dependencies to buildsrc versions

49dee73

add nearly all the targets ktor supports

a6fdb82

port from using jvm file to okio filesystem

7744979

add function to add file to ipfs using a BufferedSource

19893ed

use experimental memory model

bc237d9

change project library folder folder from being named lib to ipfs-api

f740b60

that way the library artifact has an appropriate name when publishing outside of jitpack

update ktor

0cb3154

add an option progress callback for adding files

4596b38

tests broken :/

update kotlin and dependencies

a6963d9

fix add

281e9a4

fix tests

93f50a0

update example program to add a file

bc0b4d5

resolve warning

63458b2

calculate progress percentage of upload and add

4db755c

Add.file: change source from BufferedSource to ByteArray.

65a18c5

BufferedSource.readByteArray clears the source buffer. Which was not my intention

fix add progress callback

b38f3f3

Merge branch 'add-progress' into multiplatform

ae2a5fc

Fix getting bytes

361012a

Infuras public gateway does not exist anymore add support for basic auth

689707c

luca992 added 3 commits August 13, 2022 01:37

gradle and coroutines updates

c4c2210

workaround for SocketTimeoutException

71ad41e

https://youtrack.jetbrains.com/issue/KTOR-3840/OkHttp-Socket-timeouts-without-timeout-configured

update ktor

549cdd9

luca992 force-pushed the multiplatform branch from 1abd595 to 549cdd9 Compare August 13, 2022 23:07

luca992 added 6 commits August 13, 2022 20:16

add a prepareCallCmd to handle streaming data for add and get

65999be

reformat

cff2f9a

return as channel instead of callback

782553b

Return content length with cat's ByteReadChannel when streaming

fb69325

fix getting content length

ec4d9d4

revert back to returning in callback handler to prevent channel closing

cdc20a0

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Multiplatform #39

Multiplatform #39

luca992 commented Jun 6, 2022

Multiplatform #39

Are you sure you want to change the base?

Multiplatform #39

Conversation

luca992 commented Jun 6, 2022

Todo: