cordova-plugin-code-push icon indicating copy to clipboard operation
cordova-plugin-code-push copied to clipboard

Replace cordova-plugin-file-transfer with standard-compliant implementation

Open jacobg opened this issue 6 years ago • 8 comments

cordova-plugin-file-transfer is deprecated and buggy (see #234). The plugin can be completely replaced with a standard-compliant implementation that works on Android >= 4.4 and iOS >= 10.

The Cordova team published a blog article last year explaining this transition: https://cordova.apache.org/blog/2017/10/18/from-filetransfer-to-xhr2.html

Regarding the minimum required versions for the standard-compliant implementation, the latest version of Cordova Android platform already requires a minimum version of 4.4 (SDK 19). Source: https://cordova.apache.org/docs/en/latest/guide/platforms/android/

And Apple has announced that they will soon require all app submissions to target a minimum version of iOS 12:

Starting March 2019, all iOS apps submitted to the App Store will need to be built with the iOS 12.1 SDK or later. Source: https://developer.apple.com/ios/submit/

So it seems that cordova-plugin-code-push should transition off of cordova-plugin-file-transfer to a more modern and stable approach.

jacobg avatar Dec 20 '18 11:12 jacobg

Hi @jacobg, Thanks for the feedback!

Yes, it sounds pretty reasonable, we will add this item to our backlog and look at it in more detail. I can't say for sure when we solve this issue but we'll keep you posted.

yuri-kulikov avatar Jan 08 '19 15:01 yuri-kulikov

@yuri-kulikov Are you open to a pull request to resolve this?

bytenik avatar Mar 21 '19 19:03 bytenik

Hey @jacobg

Of course, we'll review it when we have time. We really appreciate it! 👍

yuri-kulikov avatar Mar 22 '19 16:03 yuri-kulikov

@yuri-kulikov Okay, a member of my team created this PR for you: https://github.com/Microsoft/cordova-plugin-code-push/pull/513

This should resolve this issue.

bytenik avatar Mar 22 '19 18:03 bytenik

@yuri-kulikov It appears your server is not set up correctly for CORS. Since there's no CORS header, XHR requests for the update package fail. Who can I escalate this to in order to get that addressed?

bytenik avatar Mar 29 '19 22:03 bytenik

How is this going? I have installed the latest version of the plugin "1.12.0" and it still uses FileTransfer plugin which is deprecated.

GuilleBarrena avatar Oct 25 '19 13:10 GuilleBarrena

Also having issues with this.

EvanWieland avatar Nov 21 '19 17:11 EvanWieland

#513 needs help from people familiar with the code base.

ermik avatar Nov 23 '19 23:11 ermik