apache / cordova-plugin-file-transfer

Apache Cordova File Transfer Plugin
https://cordova.apache.org/
Apache License 2.0
596 stars 887 forks source link

CB-10974 Cordova file transfer Content-Length header problem #169

Closed daserge closed 7 years ago

daserge commented 7 years ago

Platforms affected

Android

What does this PR do?

Don't force chunkedMode=true for HTTPS if it was specified as false in the UploadOptions (added a warning about possible OOM for chunkedMode=false+HTTPS case)

What testing has been done on this change?

Manual and auto tests

Checklist

cordova-qa commented 7 years ago

Cordova CI Build has one or more failures. Link: http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-file-transfer-pr/45/

312 tests run, 26 skipped, 51 failed.

daserge commented 7 years ago

@jcesarmobile, @shazron, @stevengill could you please take a look?