Yes, they are necessary. You can ask me why it was done or explain it in plain english.
We do have tests. Did you check out our whitelist tests? They all pass: https://github.com/apache/cordova-ios/blob/master/CordovaLibTests/CDVWhitelistTests.m On Fri, Jan 18, 2013 at 5:41 PM, Jesse <[email protected]> wrote: > I have noticed this as well, while implementing for Windows Phone. > There are also issues with the iOS implementation [1] > > There are multiple string replacements that replace eachother ... > > First we replace the "*." with "(\\s{0}|*.)" > Then, we replace "." with "\\." > and "*" with ".*" > The problem is that the . and * in the first piece of regex is replaced by > the others ... > So well end up with "(\\s{0}|.*\\.)" > > Good times in the land of regex .... > > I think it would be beneficial to write some native tests to simply test > the function and verify all url wildcard types and positions, as it appears > it is broken everywhere. > > [1] > > https://github.com/apache/cordova-ios/blob/master/CordovaLib/Classes/CDVWhitelist.m#L126 > > On Fri, Jan 18, 2013 at 4:15 PM, Joe Bowser <[email protected]> wrote: > > > Hey > > > > It seems that the recent move to the Config class has broken > > whitelisting. I'm currently trying to figure out why, but at the > > moment, it's broken. I created a bug and set it to be a blocker, since > > we should be able to load whitelisted URIs directly in the > > CordovaWebView. > > > > Am I the only one who notices this behaviour? > > > > Joe > > > > > > -- > @purplecabbage > risingj.com >
