Lost content-length header fix

[francisco-sanchez-molina]

No description provided.

[wkh237]

@francisco-sanchez-molina , thanks for the PR.

I've noticed that request on Android does not contains Content-Length field but IOS does, after some simple tests.

However, when I try to add these lines to Android, it throws an error. After some investigation, the Content-Type field is created by OkHttp conditionally per this.

Therefore, I'm not sure if this change is necessary 😧

[francisco-sanchez-molina]

Hi @wkh237 !

I sent a photo of camera-roll and everything was fine, including content-length header.

In Request-body api ( https://square.github.io/okhttp/3.x/okhttp/okhttp3/RequestBody.html ) I found:
long contentLength() Returns the number of bytes that will be written to out in a call to writeTo(okio.BufferedSink), or -1 if that count is unknown.

So I think it is necessary.

If it throws an exception, it may be because the variable contentLength has an incorrect value in some cases

thanks you so much!

[MQZhangThu]

hi, @francisco-sanchez-molina

I have the same problem and I am wondering how did you calculate the content-length value? thx.