Sub-protocol not included when opening websocket [4.0.0-beta6]

[isaacmanu]

Version

4.0.0-beta6

Summary

I'm trying to migrate our project to 4.0.0-beta6.

I've replaced:

import com.apollographql.apollo3.network.ws.GraphQLWsProtocol
import com.apollographql.apollo3.network.ws.WebSocketNetworkTransport

.subscriptionNetworkTransport(
                WebSocketNetworkTransport.Builder()
                    .serverUrl(BASE_URL)
                    .protocol(GraphQLWsProtocol.Factory())
                    .build()
            )

With:

import com.apollographql.apollo3.network.websocket.GraphQLWsProtocol
import com.apollographql.apollo3.network.websocket.WebSocketNetworkTransport

.subscriptionNetworkTransport(
                WebSocketNetworkTransport.Builder()
                    .serverUrl(BASE_URL)
                    .wsProtocol(GraphQLWsProtocol())
                    .build()
            )

But whenever our app attempts to open a websocket now, it does so without the including any sub protocol:

[martinbonnin]

Thanks for sending this, we'll dig into it.
Just FYI, you can continue using the com.apollographql.apollo3.network.ws APIs if they worked well before, they are still around. But we'll definitely want to send that subprotocol for the new version.

[martinbonnin]

Fix is in #5948. Thanks for catching this!

[github-actions]

Do you have any feedback for the maintainers? Please tell us by taking a one-minute survey. Your responses will help us understand Apollo Kotlin usage and allow us to serve you better.