cordova-plugin-remote-injection icon indicating copy to clipboard operation
cordova-plugin-remote-injection copied to clipboard

Published 20 hours ago • verified publisher icon TruckMovers

UIWebView issue

Open mkdevel opened this issue 4 years ago • 8 comments

Plugin is not working any more. Now if I add this plugin to iOS project, I get following error from app store: ITMS-90809: Deprecated API Usage - New apps that use UIWebView are no longer accepted. Instead, use WKWebView for improved security and reliability.

Can you remove UIWebView components from plugin? Problems are all because of new App store requirements for any future update.

mkdevel avatar May 11 '20 14:05 mkdevel

This plugin already supports WKWebView. Are you sure you've configured your Cordova project to use WKWebView?

Maybe it's changed since I last looked, but a new Cordova project will run with UIWebView unless you install a plugin that changes the web view to WKWebView. (Unrelated to this plugin)

markdon avatar May 11 '20 23:05 markdon

Yes, before it gave just warnings about that, but now it shows it as error from uploading to app store. Im using WKWebView (as shown in config.xml below) and if I remove only your plugin, then it uploads to app store. It seems that app store gives error even if there is any UIWebView code in plugin.

<platform name="ios"> <preference name="WKWebViewOnly" value="true" /> <preference name="CordovaWebViewEngine" value="CDVWKWebViewEngine" /> <feature name="CDVWKWebViewEngine"> <param name="ios-package" value="CDVWKWebViewEngine" /> </feature> </platform> <plugin name="cordova-plugin-wkwebview-engine" />

mkdevel avatar May 12 '20 10:05 mkdevel

If you're sure it's this plugin, I would have a look at the more recent forks here: https://github.com/TruckMovers/cordova-plugin-remote-injection/network to look for anyone what has already fixed the issue. If it's not fixed in a fork, you probably need to fix it yourself. There's a slight chance if you open an issue on one of the forks, they might fix the issue if they are still actively using the plugin.

markdon avatar May 13 '20 00:05 markdon

Yes, before it gave just warnings about that, but now it shows it as error from uploading to app store. Im using WKWebView (as shown in config.xml below) and if I remove only your plugin, then it uploads to app store. It seems that app store gives error even if there is any UIWebView code in plugin.

<platform name="ios"> <preference name="WKWebViewOnly" value="true" /> <preference name="CordovaWebViewEngine" value="CDVWKWebViewEngine" /> <feature name="CDVWKWebViewEngine"> <param name="ios-package" value="CDVWKWebViewEngine" /> </feature> </platform> <plugin name="cordova-plugin-wkwebview-engine" />

Did you use this: https://cordova.apache.org/announcements/2019/11/25/cordova-ios-release-5.1.0.html Same problem, but fixed it with 5.1.x.

samall76 avatar May 19 '20 12:05 samall76

Also after I cloned your projects and removed UI components, then app store approved my project and everything works fine.

mkdevel avatar May 19 '20 14:05 mkdevel

@mkdevel it would be very good of you to fork the repository and commit the changes you've made to help others that find this issue.

Or just provide more details about the changes you made if they're simple.

markdon avatar May 19 '20 23:05 markdon

I added some links to various forks in PR #47.

I find the network view very handy to see the most recent forks & activity.

davidgovea avatar May 20 '20 16:05 davidgovea

As a side note on @davidgovea PR, the forks can be used in Cordova, by replacing the plugin name in package.json with the URL of a fork:

github:RobertY9/cordova-plugin-remote-injection

I was able to pass certification with this setting

Thanks @davidgovea ! Fabien

fnicollet avatar Apr 02 '21 09:04 fnicollet