As far as I'm aware, we're basically waiting for this to be done before starting the vote thread. Does this code exist yet?
On Tue Jan 20 2015 at 12:12:22 PM Andrew Grieve <agri...@chromium.org> wrote: > I was planning on doing exactly what Darryl described. Would love such a > PR! Note that we've just used this approach for the new WebView security > hooks: > > https://github.com/apache/cordova-android/commit/ > 623b394c830b8a83b5c2f16624d8013b6f851cd9 > https://github.com/apache/cordova-android/commit/ > 11002d4a56a4901087f514e2d01f8db392d0abe1 > > > CookieManager has been exposed to plugins for a long time, and it would be > crippling if FileTransfer could not set cookies. > > > > On Tue, Jan 20, 2015 at 1:48 PM, Joe Bowser <bows...@gmail.com> wrote: > > > I think we should make the File Transfer plugin not need a CookieManager. > > It sounds like that's the bigger problem than it having to be tied to a > > particular implementation of Cookies. > > > > On Tue Jan 20 2015 at 10:32:25 AM Darryl Pogue <dvpdin...@gmail.com> > > wrote: > > > > > With the idea of preparing Cordova Android 4.0.x for release starting > > > to come up in discussions, I thought it was worth raising this as a > > > potential blocker. > > > > > > The file transfer plugin uses the Android webview cookie manager. When > > > you're using a Crosswalk webview (or GeckoView presumably), in the > > > best case there are no cookies with file transfer requests and in the > > > worst case it will cause the app to crash on Android 4.2.x. > > > > > > There are a few existing bug reports and PRs related to this, but none > > > of them propose a general solution for different webviews. [1] [2] [3] > > > [4] > > > > > > I was looking at this problem last week and the only general solution > > > I could think of would involve adding a CordovaCookieManager interface > > > and implementing it for each webview engine, which didn't seem to be > > > the most idea situation. > > > > > > I can write that interface and make a PR for it, but I'd rather hear > > > if anyone has better ideas before starting to make changes across > > > multiple repos. > > > > > > > > > [1]: https://github.com/crosswalk-project/crosswalk-cordova- > > > android/pull/38 > > > [2]: https://github.com/apache/cordova-plugin-file-transfer/pull/8 > > > [3]: https://github.com/MobileChromeApps/mobile-chrome-apps/issues/464 > > > [4]: https://github.com/gaochun/cordova-plugin-file-transfer/commit/ > > > 0063249e279b99a0feb4601650fc3a4c9e8a8ed2 > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org > > > For additional commands, e-mail: dev-h...@cordova.apache.org > > > > > > > > >
