Http post not sending body on iOS

[tejainece]

Map lMap = {"Foo1": "Bar1", "Foo2": "Bar2"};
String lData = JSON.encode(lMap);  

Map lHeaders = {"Content-type": "application/json", "Accept": "application/json"};

http.Response lResp = await http.post(lUrl, body: lData, headers: lHeaders);

The body is empty.

PS: Also missing the cozy network tab in DevTools in chromium.

[abarth]

/cc @collinjackson

[tejainece]

Can I get an ETA on this?

[eseidelGoogle]

Attempting to repro now.

[eseidelGoogle]

Current attempt: https://github.com/eseidelGoogle/repro_flutter_1343

[eseidelGoogle]

Android is complaining with
android: I/flutter : NetworkService unavailable MojoInternalError: Failed to register the MojoHandle.
I'm not sure what that means yet. @collinjackson @mpcomplete

[eseidelGoogle]

So I'm not yet able to get as far as @tejainece did. I wonder if there is a regression on master.

[eseidelGoogle]

OK. I have it working on Android. Checking iOS. Ran into #1345 (our iOS workflow still needs a ton of work).

[eseidelGoogle]

OK. I've confirmed this bug on iOS. See https://github.com/eseidelGoogle/repro_flutter_1343/blob/master/README.md

[eseidelGoogle]

Yup. It appears the iOS code doesn't use the body parameter at all. :/

https://github.com/flutter/engine/blob/master/sky/services/ns_net/url_loader_impl.mm

So the real bug here is that we don't have any testing of our http library. The dart http library appears to have tests, maybe we can just run them:
https://github.com/dart-lang/http

[eseidelGoogle]

@chinmaygarde has offered to write the missing body code. I expect we'll have this fixed today.

[eseidelGoogle]

The android version only supports a single part body. We can do the same on iOS:
https://github.com/flutter/engine/blob/master/sky/services/oknet/src/org/domokit/oknet/UrlLoaderImpl.java#L161

[eseidelGoogle]

I think Android is also hard-coding the content type which is wrong:
https://github.com/flutter/engine/blob/master/sky/services/oknet/src/org/domokit/oknet/UrlLoaderImpl.java#L189

[eseidelGoogle]

I'm not sure this can be closed yet. There is still an issue of android hard-coding the content type which is causing android to post with 'form=' instead of 'data=' in this example. At least that's my understanding. Can you confirm @tejainece?

[tejainece]

I am sorry. I am in a training, so don't have access to my Linux and mac machines. Will test it as soon as i get back home on Thursday.

Thanks for quick response.

[tejainece]

It doesn't work yet. It still sends empty body.

Maybe I don't have the recent version. I did flutter upgrade. How do I verify it?

[chinmaygarde]

I am not able to reproduce this. Can you try removing the ios/ directory and try flutter ios --init again (after the flutter upgrade that you have already done.)

[tejainece]

I am testing it on Android, not ios. But sure, I will try that.

[tejainece]

Working on Android now. Thanks :)

[sethladd]

Cool, thanks for testing!

[bennybat50]

final Map<String,dynamic>userData={
      'firstname': _firstNameField,
      'lastname': _lastNameField,
      'display_name': _userNameField,
      'email': _emailField,
      'password': _passwordField
    };

    final response = await http.post(url, body:json.encode(userData) );

I tried sending a post request but its sending an empty body to the server how do i resolve this

[eseidelGoogle]

@bennybat50 would you be willing to open a new bug with more info (e.g. your flutter version information, etc?) . Thanks!

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.