Is there anyone experienced with apollo graphql mu...
# multiplatforma
Aleksandr Ivanov
10/15/2020, 12:24 PMIs there anyone experienced with apollo graphql multiplatform library? It is working fine on android but on iOS we got empty body on our backend.
l
louiscad
10/15/2020, 12:50 PMIf you have more details, I think @mbonnin might be able to help you as he's been working on it.
👀 1
m
mbonnin
10/15/2020, 12:54 PMYup, we even have #apollo-android since a few days !
mbonnin
10/15/2020, 12:55 PM@Aleksandr Ivanov do you have more details? Does your backend receive an empty body or does it respond with an empty body?
mbonnin
10/15/2020, 12:55 PMIn all cases, I would recommend setting up a Charles proxy to capture the traffic
a
Aleksandr Ivanov
10/15/2020, 12:58 PMOh, Charles is a good idea. Will try it. Post body of the request is empty when came from iOS build. On Android it is working fine. Request is based on query defined in schema.
m
mbonnin
10/15/2020, 1:05 PMAny chance your code is open source or can be shared? That'd be super helpful to debug
a
Aleksandr Ivanov
10/15/2020, 1:12 PMIt is private unfortunatelly. But I will try to extract a part of it to demo project.
👍 1
Aleksandr Ivanov
10/15/2020, 2:54 PMPreviously I sent wrong information. The request body from iOS build is not empty.
Experimented with charles. On android (successful response):
Copy code
:method: POST
:path: /graphql
:scheme: https
content-type: application/json; charset=utf-8
content-length: 316
accept-encoding: gzip
user-agent: okhttp/4.5.0
pragma: no-cache
cache-control: no-cache
{"operationName":"DriverSchedulePageQuery","variables":{"filter":{"from":"2020-10-15","to":"2020-10-29"}},"query":"query DriverSchedulePageQuery($filter: DateFilter!) { driverSchedule(filter: $filter) { __typename content { __typename date duties { __typename id driverShiftId type startAt endAt } } totalPages } }"} Copy code
POST /graphql HTTP/1.1
Content-Type: application/x-www-form-urlencoded
User-Agent: Driver%20App/140 CFNetwork/1197 Darwin/19.6.0
Connection: keep-alive
Accept: */*
Accept-Language: en-us
Content-Length: 316
Accept-Encoding: gzip, deflate, br
{"operationName":"DriverSchedulePageQuery","variables":{"filter":{"from":"2020-10-15","to":"2020-10-29"}},"query":"query DriverSchedulePageQuery($filter: DateFilter!) { driverSchedule(filter: $filter) { __typename content { __typename date duties { __typename id driverShiftId type startAt endAt } } totalPages } }"}Content-TypeAleksandr Ivanov
10/15/2020, 3:20 PMYes the problem was in the Content-Type. I manually set
content-type: application/json; charset=utf-8l
louiscad
10/15/2020, 3:22 PMWhy the Android app didn't have that issue? OkHttp adding the content-type by default?
m
mbonnin
10/15/2020, 3:45 PMInteresting, thanks for the follow up
mbonnin
10/15/2020, 3:46 PMI don't think OkHttp does anything by default, we are doing it: https://github.com/apollographql/apollo-android/blob/62673dc817967725bfcadc2ceb1bed3b503e82da/apollo-runtime-kotlin/src/jvmMain/kotlin/com/apollographql/apollo/network/http/ApolloHttpNetworkTransport.kt#L169
mbonnin
10/15/2020, 3:47 PM@Aleksandr Ivanov do you mind opening an issue, we should do the same for iOS
l
louiscad
10/15/2020, 3:49 PM@mbonnin Can't it be put in common code to avoid forgetting it when adding support for more platforms?
m
mbonnin
10/15/2020, 3:52 PMWe could put the string constant in common code but the way to set the headers is different on iOS and Android so some of it would have to be duplicated anyways
☝️ 1
mbonnin
10/15/2020, 3:53 PM(Also I don't really want to depend on ktor for a simple use case like this)
a
Aleksandr Ivanov
10/15/2020, 7:49 PMYep, will open it tomorrow. It is very confusing that it works differently.
🙏 1
Aleksandr Ivanov
10/16/2020, 1:55 PMm
mbonnin
10/16/2020, 1:56 PMThanks you so much ! If you have the iOS code at hand, a PR is very welcome, if not, I'll fix this somewhere next week
a
Aleksandr Ivanov
10/16/2020, 3:39 PMI’m not very familiar with iOS development 🤷♂️
m
mbonnin
10/16/2020, 3:41 PMAH ok, I though you had it fixed on your side already, no problem, we'll make a fix
a
Aleksandr Ivanov
10/16/2020, 6:53 PMI fixed it in multiplatform code writen in kotlin by manually adding header to ApolloClient. If it is how it should be fixed in library I could do that. Just point me to right module to look into.
m
mbonnin
10/16/2020, 6:56 PMAh, got it! We want that to be in iOS code ultimately, I'll fix this
a
Aleksandr Ivanov
10/16/2020, 7:41 PM👌
9 Views