Mailing-List: contact callback-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: callback-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of pmuellr@gmail.com designates
 209.85.220.175 as permitted sender)
MIME-Version: 1.0
Date: Tue, 9 Oct 2012 13:17:14 -0400
Message-ID: 
 <CACAoQH0jbGdOA3BuiY4UEUKwfWp=fRg0ZeVckhy9ir_+ZyFG+w@mail.gmail.com>
Subject: white-listing File Transfer on iOS and bug CB-1600
From: Patrick Mueller <pmuellr@gmail.com>
To: callback-dev <callback-dev@incubator.apache.org>
Content-Type: multipart/alternative; boundary=20cf307cfc785e9b3104cba3828f

--20cf307cfc785e9b3104cba3828f
Content-Type: text/plain; charset=ISO-8859-1

I'm looking at https://issues.apache.org/jira/browse/CB-1600

I'm getting a crash running Mobile Spec on iOS 4.3 on iOS Simulator (Xcode
4.5.1 on Lion).

Coupla different things going on here.

One is that the crash occurs when you try to download a file from a host
not in the white-list.

There's some interesting code and comments here:


https://github.com/apache/incubator-cordova-ios/blob/master/CordovaLib/Classes/CDVURLProtocol.m#L105

Note the comment a few lines down:

      // if it FAILS the whitelist, we return TRUE, so we can fail the
connection later

Um, WHO fails the connection later?  Can't quite figure out that puzzle.
 As the comments before it indicate, I'm getting [canInitWithRequest]
invoked 3 times per failed whitelist request, and then the code blindly
continues processing the file download.  At some point, it fails, as
[CDVFileTransfer connection:didReceiveResponse:] is getting sent an
NSURLResponse and not a NSHTTPURLResponse, and the trap involves an invalid
casting of the NSURLResponse to a NSHTTPURLResponse.

(as a general cleanliness measure, perhaps we should always do isKindOf: or
whatever tests before casting?)

=== questions ===

- can we just do the white-list check FIRST, before even starting the file
transfer?  That way we never get into this mess.

- maybe that doesn't work - do these NSURLConnections silently handle
redirects, and then end up potentially failing in the white-list there?

- I think we can check for white-list failure in
the [CDVFileTransfer connection:didReceiveResponse:] method with an
isKindOf: or whatever check, but I'm not sure if there's any other cleanup
that needs to take place here.

-- 
Patrick Mueller
http://muellerware.org

--20cf307cfc785e9b3104cba3828f--