in kmm why can’t we use apollo library under IOS 13 ?

# apollo-kotlin

[Mobile Dev.]

in kmm why can’t we use apollo library under IOS 13 ?

[mbonnin]

There was something about WebSockets IIRC

[mbonnin]

Yep, it's iOS13+: https://developer.apple.com/documentation/foundation/nsurlsessionwebsockettask

[mbonnin]

Out of curiosity, what is the error you're getting? I remember (but I'm not 100% sure anymore) someone was able to link correctly on below versions

[Mobile Dev.]

What exactly is the purpose of the NSURLSessionWebSocketTask? is that for graphql subscriptions ?

[mbonnin]

yes, exactly subscriptions

[Mobile Dev.]

Maybe a version without subscriptions could be released in order to serve users under iOS 13. not sure if that would be hard

[Mobile Dev.]

for example we are not using any subscriptions whole project and we have a lot of users under ios 13

[mbonnin]

Do you get an error when trying to build your iOS app?

[mbonnin]

I'm asking because chances are that it's working already or that we can maybe workaround the error if any

[Mobile Dev.]

Sorry for the late response,

when i run the app with iphone 5 ( ios 12.4 ) i'm getting this crash

dyld: Symbol not found: _OBJC_CLASS_$_NSURLSessionWebSocketMessage
 Referenced from: /Users/alkincakiralar/Library/Developer/Xcode/DerivedData/fizbot-alhlyiyitptqumeebpzkemzcvngh/Build/Products/Debug-iphonesimulator/Bifrost.framework/Bifrost
 Expected in: /Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 12.4.simruntime/Contents/Resources/RuntimeRoot/System/Library/Frameworks/Foundation.framework/Foundation
 in /Users/alkincakiralar/Library/Developer/Xcode/DerivedData/fizbot-alhlyiyitptqumeebpzkemzcvngh/Build/Products/Debug-iphonesimulator/Bifrost.framework/Bifrost
dyld: launch, loading dependent libraries
DYLD_FRAMEWORK_PATH=/Users/alkincakiralar/Library/Developer/Xcode/DerivedData/fizbot-alhlyiyitptqumeebpzkemzcvngh/Build/Products/Debug-iphonesimulator:/Users/alkincakiralar/Library/Developer/Xcode/DerivedData/fizbot-alhlyiyitptqumeebpzkemzcvngh/Build/Products/Debug-iphonesimulator/PackageFrameworks
DYLD_FALLBACK_LIBRARY_PATH=/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 12.4.simruntime/Contents/Resources/RuntimeRoot/usr/lib
DYLD_ROOT_PATH=/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 12.4.simruntime/Contents/Resources/RuntimeRoot
DYLD_FALLBACK_FRAMEWORK_PATH=/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 12.4.simruntime/Contents/Resources/RuntimeRoot/System/Library/Frameworks
DYLD_INSERT_LIBRARIES=/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 12.4.simruntime/Contents/Resources/RuntimeRoot/usr/lib/libBacktraceRecording.dylib:/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS 12.4.simruntime/Contents/Resources/RuntimeRoot/usr/lib/
(lldb)

[Mobile Dev.]

although i'm not using any graphql subscription method, i only use query and mutation

[mbonnin]

Alright 👍

[mbonnin]

SOunds like it crashes at start time, right?

[Mobile Dev.]

yes at start time, also i just implement the library with spm and didn't import any viewcontroller or classes

[mbonnin]

I see. Mind opening an issue about this?

[Mobile Dev.]

You want me to open an issue about it ?

[mbonnin]

Yes please. Doesn't look easy to workaround unfortunately 😞

[mbonnin]

So we'll have to go the route of separate dependencies

[Mobile Dev.]

i see, im gonna open an issue thanks for your help 👍

[mbonnin]

For the record, I just bumped into the previous issue that was mentioning this

[mbonnin]

Looks like a lot of users are on iOS 13+ these days so it'll be hard to prioritize work on this but please open an issue nevertheless as some other folks from the community might be interested in contributing

[Mobile Dev.]

Should i open the ticket as bug report or feature request ?

[mbonnin]

"it depends" 🙂

[mbonnin]

It doesn't matter much to be honest. More like a feature request I'd say because the limitation is documented currently but feel free to file it as a bug if it's blocking for you

[Mobile Dev.]

https://github.com/apollographql/apollo-kotlin/issues/4276

I opened the issue

[Mobile Dev.]

thanks for your help

[mbonnin]

Thanks for reporting this 💙

[Paddy O'Brien]

Seeing this same issue. Do you happen to have a path for the websockets stuff on 3.x? The link in the original issue appears to be broken.

[mbonnin]

@Paddy O'Brien do you need websockets on iOS < 13 or do you just need the regular queries to work on iOS < 13?

[mbonnin]

• Making regular queries work on iOS < 13 is a matter of removing the dependency and shuffling how the artifacts are built
• Making websockets work on iOS < 13 is a whole different issue as it'll required bundling starscream or another library since the OS doesn't support WebSockets

[Paddy O'Brien]

Nah we just need regular queries to work, Shopify doesn’t use subscriptions. Was just looking for the path where the socket stuff has moved to as it seems to have changed between 2.x and 3.x.

[Paddy O'Brien]

I was wondering about something though. The iOS library does an excellent job of separating out this concern by introducing the NetworkTransport protocol was wondering why not a similar approach on Kotlin.

[Paddy O'Brien]

I guess really KMM just needs to add os version detection 😬
https://youtrack.jetbrains.com/issue/KT-37180

[mbonnin]

Gotcha 👍. It's 100% doable. We have the NetworkTransport API as well as a lower level WebSocketEngine one. The trick here is to do it in a backward compatible way. That certainly means splitting the current apollo-runtime that has a hard dependency on iOS13+ at the moment and introducing separate artifacts, one of those without the iOS13+ dependency

[mbonnin]

For the record, I don't think anything changed regarding this between 2.x and 3.x? Both versions use nsurlsessionwebsockettask

[Paddy O'Brien]

They do, the location of the code just changed. The link in the initial issue you found with Ellen and Ivan seems to be broken.
I think I found it though. building now 😄

[mbonnin]

What link?

[Paddy O'Brien]

The one mentioned here
https://kotlinlang.slack.com/archives/C01A6KM1SBZ/p1658151856484459?thread_ts=1657920672.252619&amp;cid=C01A6KM1SBZ

[mbonnin]

That one? https://github.com/apollographql/apollo-kotlin/blob/main/apollo-runtime-kotlin/src[…]n/kotlin/com/apollographql/apollo/network/ws/ApolloWebSocket.kt

[mbonnin]

(there's gazillions links in that issue 😅 )

[Paddy O'Brien]

yeah
https://github.com/apollographql/apollo-android/blob/main/apollo-runtime-kotlin/sr[…]n/kotlin/com/apollographql/apollo/network/ws/ApolloWebSocket.kt

[mbonnin]

Gotcha

[Paddy O'Brien]

I get a 404

[mbonnin]

Yea, it's using main

[mbonnin]

Matching 3.x code is at https://github.com/apollographql/apollo-kotlin/blob/96db33ae93aa6f3417a52267a43d7d[…]apollographql/apollo3/network/ws/NSURLSessionWebSocketEngine.kt

[Paddy O'Brien]

looks like everything is now in NSURLWebsocketEngine.kt

[Paddy O'Brien]

lol, beat me to it

[Paddy O'Brien]

thanks!

[mbonnin]

If you remove everything in that file with TODO("make this work with iOS < 13") , you should be able to link and run everything

[Paddy O'Brien]

Everything? I tried to leave it as an empty type but im getting errors

[mbonnin]

Something like this should do the job:

actual class DefaultWebSocketEngine : WebSocketEngine {
  override suspend fun open(url: String, headers: List<HttpHeader>): WebSocketConnection {
    TODO("Not yet implemented")
  }

  override suspend fun open(url: String, headers: Map<String, String>): WebSocketConnection {
    TODO("Not yet implemented")
  }

}

[mbonnin]

(not tested but the API is relatively small, there shouldn't be too much needed besides that)

[Paddy O'Brien]

Seems to have done the trick, Thanks

[mbonnin]

Sure thing!