How odd but I have no choice but to believe in Google :) Still I wish I would have seen these in the docs when I was writing the initial code back in Android 2.x days. It would have made my life a lot easier.
Simon Mac Donald http://hi.im/simonmacdonald On Wed, Dec 5, 2012 at 10:53 PM, Andrew Grieve <agri...@chromium.org> wrote: > I looked at switching to HttpClient when fixing some of the Android FT bugs > a while ago. According to this blog post (by an Android > dev), HttpURLConnection is recommended for gingerbread+. > > http://android-developers.blogspot.ca/2011/09/androids-http-clients.html > > > On Wed, Dec 5, 2012 at 9:55 PM, Simon MacDonald > <simon.macdon...@gmail.com>wrote: > > > Oh my crap. I swear this was not documented when I worked on the original > > File Transfer code for Android. The entire FileTransfer plugin can be > > re-written now to use actual tested code. The only thing we need do is to > > keep the old method signatures. > > > > If you are looking for me I'll be over in the corner kicking the cat. > > > > Simon Mac Donald > > http://hi.im/simonmacdonald > > > > > > > > On Wed, Dec 5, 2012 at 9:49 PM, Brian M Dube <bd...@apache.org> wrote: > > > > > On 10/30/2012 07:53 AM, Simon MacDonald wrote: > > > > I would have loved to have used the Apache HttpClient package but we > > are > > > > trying to stay away from external dependencies. > > > > > > Apache HttpClient is available to applications as part of the Android > > > platform, isn't it? Perhaps I've misunderstood. > > > > > > org.apache.cordova.HttpHandler [1] makes use of HttpClient, but > > > HttpHandler itself doesn't appear to be referenced by any other Cordova > > > code. > > > > > > I'd be happy to work on a pull request if it is in fact available > > > without adding a dependency. > > > > > > -Brian > > > > > > [1] > > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/src/org/apache/cordova/HttpHandler.java > > > > > > > When you set chunkedMode to false I know that the transfer worked as > > per > > > > your comment on JIRA. I will update the docs to mention Nginx > servers. > > > > > > > > Simon Mac Donald > > > > http://hi.im/simonmacdonald > > > > > > > > > > > > On Mon, Oct 29, 2012 at 2:56 PM, Ronald Partridge > > > > <r...@ronald-douglas.com>wrote: > > > > > > > >> See: > > > >> > > > >> > > > > > > https://github.com/apache/incubator-cordova-android/blob/master/framework/src/org/apache/cordova/FileTransfer.java > > > >> private void upload(final String source, final String target, > > JSONArray > > > >> args > > > >> , CallbackContext callbackContext) > > > >> > > > >> Cordova Android may have a bug with the way files are being sent. > The > > > >> Android source code appears to use the built in java > HttpURLConnection > > > and > > > >> the developer who wrote the functionality decided to write the logic > > to > > > >> assemble the post data to be transfered. I examined two different > post > > > >> dumps > > > >> being sent to a web server running Nginx and Varnish. Notice how the > > > >> headers are different between *iOS *and *Android*. > > > >> I would suggest using > > > >> > > > >> > > > > > > http://hc.apache.org/httpcomponents-client-ga/httpclient/apidocs/org/apache/http/impl/client/DefaultHttpClient.html > > > >> > > > >> Here are the two post dumps: > > > >> * > > > >> Android:* > > > >> > > > >> POST /service/claim/photo/75?api_key=123ab > > > >> c HTTP/1.1 > > > >> Connection: Keep-Alive > > > >> Content-Type: multipart/form-data;boundary=***** > > > >> User-Agent: Dalvik/1.6.0 (Linux; U; Android 4.1.1; Nexus S > > Build/JRO03E) > > > >> Host: sit.service.app.mydomain.ca > > > >> Accept-Encoding: gzip > > > >> Transfer-Encoding: chunked > > > >> > > > >> > o<EF><8A>P9)^B^@6^@^@^@6^@^@^@^@^@^L^G<AC><C8>^@PV<8E>^A^@E^@^@(<E1>e@ > > > >> ^@@^F<DE>ESC^_[&c<88><B2>^@P<FF><BD>%&<9C><A5><AF>WĘP^P^Y > > > >> <E6>^^@^@o<EF><8A>P<93>,^B^@<D9>^D^@ > > > >> ^@<D9>^D^@^@^@^@^L^G<AC><C8>^@PV<8E>^A^@E^@^D<CB><E1>f@^@@^ > > > >> F<DA>:<AC>ESC^_[&c<88><B2>^@P<FF><BD>%&<9C><A5><AF>WĘP^X^Y > > > ^?I^@^@HTTP/1.1 > > > >> 503 Service Unavailable > > > >> Server: Varnish > > > >> Content-Type: text/html; charset=utf-8 > > > >> Content-Length: 932 > > > >> Accept-Ranges: bytes > > > >> Date: Fri, 26 Oct 2012 20:15:43 GMT > > > >> X-Varnish: 409357173 > > > >> Age: 0 > > > >> Via: 1.1 varnish > > > >> Connection: close > > > >> X-Cache: MISS > > > >> > > > >> > > > >> *iOS*: > > > >> > > > >> POST /service/claim/photo/73?api_key=123abc HTTP/1.1 > > > >> > > > >> Host: sit.service.app.mydomain.ca > > > >> User-Agent: Mozilla/5.0 (iPhone; CPU iPhone OS 5_1_1 like Mac OS X) > > > >> AppleWebKit/534.46 (KHTML, like Gecko) Mobile/9B206 > > > >> Content-Length: 139038 > > > >> Accept: */* > > > >> Content-Type: multipart/form-data; boundary=*****org.apache. > > > >> cordova.formBoundary > > > >> X-Requested-With: XMLHttpRequest > > > >> > > > >> Accept-Language: en-us > > > >> Accept-Encoding: gzip, deflate > > > >> Connection: keep-alive > > > >> > > > >> 4<EF><8A>P;^F^A^@6^@^@^@6^@^@^@^@^@^L^G<AC><C8>^@PV<8E>^A^@E^@^@(}1@ > > > >> ^@@^FC^S<AC>ESC^_[&c<88><B2>^@P<F7><8F><A1><F9><EB><CF>^^^_<C1>P^P^Y > > > >> ^X`^@^@4<EF><8A>P<F7>^K^A^@ > > > >> <9A>^E^@^@<9A>^E^@^@^@PV<8E>^A:^@^U<FA>~<E0>^Z^H^@E^@^E<8C><D7>C@ > > > >> ^@)^F<FA><9C>&c<88><B2><AC>ESC^_[<F7><8F>^@P^^^_<C1>< > > > >> > > > >> > > > > > > A1><F9><EB><CF>P^P<FF><FF><F9><A6>^@^@--*****org.apache.cordova.formBoundary > > > >> Content-Disposition: form-data; name="description" > > > >> > > > >> Photo > > > >> --*****org.apache.cordova.formBoundary > > > >> Content-Disposition: form-data; name="userfile"; > > > >> filename="cdv_photo_002.jpg" > > > >> Content-Type: image/jpeg > > > >> Content-Length: 138722 > > > >> > > > >> > > > >> > > > >> < > > > >> > > > > > > http://hc.apache.org/httpcomponents-client-ga/httpclient/apidocs/org/apache/http/impl/client/DefaultHttpClient.html > > > >>> > > > >> > > > > > > > > > >
