iOS deprecated UIWebView #718

cristianoccazinsp · 2019-09-10T20:05:00Z

Review the documentation: https://docs.viromedia.com/
Search for existing issues: https://github.com/viromedia/viro
Use the latest ViroReact release: https://docs.viromedia.com/docs/releases

Environment

Please provide the following information about your environment:

Development OS: Mac
Device OS & Version: iOS 12.4.1
Version: "react-viro": "2.16.0" "react-native": "0.60.5"
Device(s): iPhone 7

Description

iOS is removing UIWebView and any submitted application will now receive a warning. After a bunch of updates, it seems like this library is the final culprit still using this.

grep -r UIWebView node_modules/*

Binary file node_modules/react-viro/ios/dist/ViroRenderer/static_lib/ViroKit_static_lib.framework/ViroKit_static_lib matches
Binary file node_modules/react-viro/ios/dist/ViroRenderer/static_lib/x86_64/ViroKit_static_lib.framework/ViroKit_static_lib matches
Binary file node_modules/react-viro/ios/dist/ViroRenderer/static_lib/armv7_arm64/ViroKit_static_lib.framework/ViroKit_static_lib matches
Binary file node_modules/react-viro/ios/dist/ViroRenderer/ViroKit.framework/ViroKit matches
Binary file node_modules/react-viro/ios/dist/ViroRenderer/x86_64/ViroKit.framework/ViroKit matches
Binary file node_modules/react-viro/ios/dist/ViroRenderer/armv7_arm64/ViroKit.framework/ViroKit matches

Can we have an update that removes this?

The text was updated successfully, but these errors were encountered:

kala888 · 2019-09-17T09:28:51Z

Same issue report to my email from Apple when upload App to Store

ITMS-90809: Deprecated API Usage - Apple will stop accepting submissions of apps that use UIWebView APIs . See https://developer.apple.com/documentation/uikit/uiwebview for more information.

cristianoccazinsp · 2019-09-17T16:04:15Z

Any updates on this? Is it really react viro using it or is it something else? I can't really seem to find any other culprit.

radvani · 2019-09-30T20:27:58Z

It turns out this was Google Cardboard (their iOS GVR SDK) that had this dependency. If Google fixes this on their end then we'll update with the correction; otherwise we'll have to remove support for Google Cardboard.

cristianoccazinsp · 2019-10-04T14:56:36Z

Is it possible to shrink down / remove iOS dependencies if they are not needed? Similar to what you've done with Android that some dependencies can be omitted.

Truth is we only use the library to support 360 images.

vpallegar · 2019-11-12T04:55:16Z

Hi All. Does anyone know if there is a resolution here? seems like any new app submissions will be rejected from apple until this is resolved.

cristianoccazinsp · 2019-11-12T14:25:24Z

Is apple rejecting uploads already? Are we all sure the deprecation warning comes from here? Should be as simple as updating any google's reference. I believe google has updated their SDKs already.

cristianoccazinsp · 2019-11-12T14:30:16Z

Are these libraries the culprits maybe? https://github.com/viromedia/virocore/blob/master/ios/Podfile

Note that they are not even from viro react, but rather the core. When using the static lib, I'm not even sure if updating this is possible without having a new build of the static libs.

vpallegar · 2019-11-12T17:28:31Z

The last email i got from apple regarding my dev app was that Apple will stop accepting submissions of apps that use UIWebView APIs - so i'm assuming any new app submissions will be rejected.

Digging deeper as radvani mentioned it seems to be GVRSDK which has the issue. Going thru the viro source code and running grep -r UIWebView * on the Pods folder (Virocore/ios/Pods) it returns Binary file GVRSDK/Libraries/libGVRSDK.a matches...and looks like google does not support that anymore, telling users to upgrade cardboard instead. I tried pulling out the GVRSDK dependency but that causes part of the code to no longer work when recompiling viro.

radvani · 2019-11-12T17:59:18Z

Yes, Google just open sourced Cardboard but it looks like they’re effectively deprecating it. We may need to release a version without Cardboard to mitigate this. Raj

…

On Nov 12, 2019, at 12:28 PM, vpallegar ***@***.***> wrote: The last email i got from apple regarding my dev app was that Apple will stop accepting submissions of apps that use UIWebView APIs - so i'm assuming any new app submissions will be rejected. Digging deeper as radvani mentioned it seems to be GVRSDK which has the issue. Going thru the viro source code and running grep -r UIWebView * on the Pods folder (Virocore/ios/Pods) it returns Binary file GVRSDK/Libraries/libGVRSDK.a matches...and looks like google does not support that anymore, telling users to upgrade cardboard instead. I tried pulling out the GVRSDK dependency but that causes part of the code to no longer work when recompiling viro. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

Hless · 2019-12-20T10:07:39Z

@radvani I checked the open source Cardboard library, and I don't get the impression they are trying to abandon the project. We are on a tight release schedule with our current project, but we will keep relying on viro, so eventually we will have to get rid of the UIWebView depedency before we are unable to release updates for our app.

Apparently the UIWebView dependency also no longer exist in the new Cardboard library.

Can you please point me in the general direction of where to start to replace the GVRSDK libary with the new Cardboard library in the viro core? I will take a swing at it as soon as I can.

Edit:
Perhaps we should continue the discussion here:
viromedia/virocore#320

TwR-Slice · 2020-01-31T09:35:15Z

Are there any news about this issue?

Jackyaung · 2020-03-20T10:21:00Z

I am facing the same problem, any solution and how to replace the GVRSDK

Jackyaung · 2020-03-20T12:31:45Z

@TwR-Slice did you solve the problem.

cristianoccazinsp · 2020-04-09T15:59:50Z

Any updates on this? Looks like Apple will stop accepting uploads with UIWebView starting in April 2020 (i.e., now). This has just become a breaking change and needs immediate attention. Any guide on how to update GVRSDK is welcome.

cristianoccazinsp · 2020-04-09T19:27:21Z

I've been trying to upgrade react's version to 0.61.5 in this branch (https://github.com/cristianoccazinsp/viro/tree/rn-upgrade-attempt) alongside updating google SDKs for iOS to remove some deprecation warnings. Compilation of the new android aar file seems fine, but the iOS migration step is harder than I thought and I can't seem to make it compile with the new podspec file. If someone is interested in taking it from there... I'm most likely not putting more effort into this and will be searching for another 360 viewer shortly.

cristianoccazinsp · 2020-04-10T17:10:34Z

Well, as much as I tried, fixing the iOS project to work with the new RN pods setup seems impossible. Will pass on this one.

Also, seems like updating Google's SDK won't fix the issue since it hasn't been updated in some time.

ManigandanRaamanathan · 2020-05-03T18:46:26Z

Guys any update on this issue? Apple already stopped receiving the UIWebView API. Now we are unable to upload the app

ManigandanRaamanathan · 2020-05-03T18:54:11Z

@dthian please fix the OS deprecated UIWebView issue

dthian · 2020-05-03T22:42:32Z

Hey @ManigandanRaamanathan, unfortunately I'm not actively working on Viro at the moment. Having said that, Viro + ViroCore is open sourced now, so you should be able to go in and patch the source to remove the UIWebview. Good luck!

cristianoccazinsp · 2020-05-03T23:36:31Z

I don't think there's any way to patch this. Viro uses google VR SDK, which is deprecated and hasn't been updated in ages. To fix it you need to upgrade to the new google cardboard API; most likely a lot of work.

dthian · 2020-05-03T23:38:41Z

Another way as well (if you are using AR) is to just remove GVR from the build, and comment out (or abstract out) the components that are using it. The ARCode path vs VR Code path is different, so that should work and is easier to do.

silvainSayduck · 2020-05-04T06:43:00Z

Hi @dthian,

Thanks for the pointer! For an AR-only app, would you have any practical advice as to how does one remove GVR from the build, and find out which components are using it?

I've been trying a few different search and replace approaches, and tried removing libraries from the Podfile, directly from Xcode, ... but the warning always remained.

ManigandanRaamanathan · 2020-05-04T06:56:26Z

@silvainSayduck @dthian may I know how to remove GVR out? Im new to native iOS

mbryk · 2020-05-04T15:13:44Z

@ManigandanRaamanathan @silvainSayduck I haven't gotten into the core of this library at all, but my best guess would be to follow the "Manual Building of the Renderer" instructions here, remove GVR there, and make sure to do 3b in the iOS section to build the ViroKit_static_lib. Then, use this result in your node_modules instead of the old ViroKit.

If after doing this, you can run grep -r UIWebView node_modules/* and get no results, you should be good. (And if you do try this and it works, please post a fork! Thanks!)

ManigandanRaamanathan · 2020-05-04T15:18:25Z

@mbryk for the time being I'm using this commits v2.18.0 from @HedwigAR https://github.com/mendix/viro/pull/1

dthian · 2020-05-04T15:44:41Z

Hedwig managed to remove it and it seems to be working on his end. The commit for the changes here might help. @mbryk

silvainSayduck · 2020-05-05T16:51:42Z

Hi,

Due to #835, I had to use a Viro custom build, and thus tried to also not reference to UIWebView that will not be accepted on the App Store by Apple anymore after December 2020, but it didn't work... Would anyone be able to explain why my approach did not work? Thanks in advance!

Here are the steps I followed, losely inspired from several issues and forks and the various Viro README instructions.
Disclaimer: I am very unexperienced with this kind of manoeuvre, and as such, some or all of this might actually be clumsy or unnecessary

Clone the repo into your workspace with git: git clone https://github.com/mendix/virocore/
Revert to the commit with the fix for NSURLResponse crash: git reset 621b2524f44f8e93a08085b3770d457f5e002e43 --hard
Clone the react-viro repo (named viro) in the same workspace (same parent directory as virocore) with git: git clone https://github.com/mendix/viro.git
Revert to commit with the fix for UIWebView deprecation: git checkout build and git reset ecc4a15e456aeea57d782e189ca2318b829958f5 --hard
$ cd virocore/ios/ and $ pod install
Open ViroRenderer.xcworkspace in Xcode (make sure you open the .xcworkpace file, and not the .xcodeproj file!). Build ViroKit scheme with Build Configuration set to Release and target set to Generic iOS Device
(Optional) A build error might happen as Xcode tries to find ios/Pods/Target Support Files in ios/Pods/Pods/Target Support Files. If so, copy/paste all files and folders from ios/Pods to ios/Pods/Pods
When the build succeeds, you should see an updated armv7_arm64 folder and updated ViroKit.podspec file in the viro/ios/dist/ViroRenderer folder
$ cd viro and $ npm install
$ cd viro/ios and $ pod install
Open ViroReact.xcworkspace in Xcode.
Build ViroReact scheme with Build Configuration set to Release and target set to Generic iOS Device
You should see a new file libViroReact.a at ios/dist/lib/libViroReact.a
Replace your_project/node_modules/react-viro/ios/dist/lib/libViroReact.a with the newly generated ios/dist/lib/libViroReact.a
Replace your_project/node_modules/react-viro/ios/dist/ViroRenderer with the one from viro/ios/dist/ViroRenderer
Build your project

If this also helps, here are my fixed libViroReact.a and ViroRenderer: https://drive.google.com/file/d/1X09c3UUpl7g9MhyJOOwFrMzYMnA0eyx7/view?usp=sharing

Huge thanks to all the contributors that enabled me to patch this together, and especially @dthian, @HedwigJDoets and @HedwigAR!

namquyuit · 2020-05-06T03:38:54Z

Awesome @silvainSayduck I just ran into this issue yesterday. You saved me! Thank you very much!

cristianocca · 2020-05-06T04:01:39Z

@silvainSayduck any thoughts on making the build process easier? With RN >= 0.60, everything can be migrated to pods (including manually linked RN libraries). Ideally, one project should include the other with just a podfile configuration rather than including pre-built files.

A similar issue happens with Android. There's one build step that is linked directly against a specific RN version, which is then copied over to the other project. Quite messy.

Take this lib for example, multiple dependencies, all resolved nicely, even swift dependencies: https://github.com/lightbasenl/react-native-panorama-view

silvainSayduck · 2020-05-06T04:46:52Z

@cristianocca Unfortunately, as I stated, I know very little about build processes, both for Xcode and Android... The above steps were the results of a painful trial-and-error approach, and I'm not sure they all make sense at all...

@namquyuit do you mean that for you, my steps above worked to solve this issue (references to UIWebView which will be refused by Apple from December 2020), or the crashes on iOS 13.4+ (#835)?

ranjanpoudel1234 · 2020-05-10T16:01:05Z

@silvainSayduck , thank you for sharing. I tried your steps and copy pasted the libViroReact.a file and ViroRenderer folder respectively. However, after that, when I did grep -r UIWebView node_modules/, I am still seeing the following. Did you or @namquyuit see the same ? @ManigandanRaamanathan , you said it worked for you, could you please share your steps ?

Binary file node_modules//react-viro/ios/dist/ViroRenderer/static_lib/ViroKit_static_lib.framework/ViroKit_static_lib matches Binary file node_modules//react-viro/ios/dist/ViroRenderer/static_lib/x86_64/ViroKit_static_lib.framework/ViroKit_static_lib matches Binary file node_modules//react-viro/ios/dist/ViroRenderer/ViroKit.framework/ViroKit matches Binary file node_modules//react-viro/ios/dist/ViroRenderer/x86_64/ViroKit.framework/ViroKit matches

ManigandanRaamanathan · 2020-05-11T04:30:43Z

@ranjanpoudel1234 My way was much easier, I did not modify the native static_lib's. I simply installed directly from this branch in package.json https://github.com/mendix/viro/pull/1 and it starts working fine. @HedwigAR have already removed the GVR contents and rebuilt the static_lib in that branch.

@silvainSayduck @dthian

namquyuit · 2020-05-11T04:34:44Z

@silvainSayduck I used your shared library files in Google Drive and patched to my project, it passed Apple rejection.

MinhazMM · 2020-05-11T04:42:37Z

@ManigandanRaamanathan I am developing a VR related application, so if I remove GVR will it not remove the entire VR support in the application?

ManigandanRaamanathan · 2020-05-11T05:05:42Z

@MinhazMM I'm afraid VR wont work in iOS build after stripping GVR feature. You may refer changes made by @HedwigAR

quangduz · 2020-05-17T02:46:06Z

Hi,

Due to #835, I had to use a Viro custom build, and thus tried to also not reference to UIWebView that will not be accepted on the App Store by Apple anymore after December 2020, but it didn't work... Would anyone be able to explain why my approach did not work? Thanks in advance!

Here are the steps I followed, losely inspired from several issues and forks and the various Viro README instructions.
Disclaimer: I am very unexperienced with this kind of manoeuvre, and as such, some or all of this might actually be clumsy or unnecessary

Clone the repo into your workspace with git: git clone https://github.com/mendix/virocore/

Revert to the commit with the fix for NSURLResponse crash: git reset 621b2524f44f8e93a08085b3770d457f5e002e43 --hard

Clone the react-viro repo (named viro) in the same workspace (same parent directory as virocore) with git: git clone https://github.com/mendix/viro.git

Revert to commit with the fix for UIWebView deprecation: git checkout build and git reset ecc4a15e456aeea57d782e189ca2318b829958f5 --hard

$ cd virocore/ios/ and $ pod install

Open ViroRenderer.xcworkspace in Xcode (make sure you open the .xcworkpace file, and not the .xcodeproj file!). Build ViroKit scheme with Build Configuration set to Release and target set to Generic iOS Device

(Optional) A build error might happen as Xcode tries to find ios/Pods/Target Support Files in ios/Pods/Pods/Target Support Files. If so, copy/paste all files and folders from ios/Pods to ios/Pods/Pods

When the build succeeds, you should see an updated armv7_arm64 folder and updated ViroKit.podspec file in the viro/ios/dist/ViroRenderer folder

$ cd viro and $ npm install

$ cd viro/ios and $ pod install

Open ViroReact.xcworkspace in Xcode.

Build ViroReact scheme with Build Configuration set to Release and target set to Generic iOS Device

You should see a new file libViroReact.a at ios/dist/lib/libViroReact.a

Replace your_project/node_modules/react-viro/ios/dist/lib/libViroReact.a with the newly generated ios/dist/lib/libViroReact.a

Replace your_project/node_modules/react-viro/ios/dist/ViroRenderer with the one from viro/ios/dist/ViroRenderer

Build your project

If this also helps, here are my fixed libViroReact.a and ViroRenderer: https://drive.google.com/file/d/1X09c3UUpl7g9MhyJOOwFrMzYMnA0eyx7/view?usp=sharing

Huge thanks to all the contributors that enabled me to patch this together, and especially @dthian, @HedwigJDoets and @HedwigAR!

Thank you so much !
You save my life, now i can go to sleep

silvainSayduck · 2020-05-18T07:12:54Z

@silvainSayduck I used your shared library files in Google Drive and patched to my project, it passed Apple rejection.

I didn't work for me, but I guess it means that some other library I use has some reference to UIWebView too... Thanks for the confirmation :)

dthian · 2020-05-18T16:02:37Z

We've also attempted removing GVR from the renderer in ViroCore, you guys could also try the built one from the pipeline here.

Jackyaung · 2020-06-05T13:11:55Z

@ManigandanRaamanathan can you please tell how I install the version 2.18.0 ? I tried to run yarn add react-viro@2.18.0, but there is no option for this one. I am stucking at this issue now.

ktemby · 2020-06-10T07:58:14Z

@dthian thank you for this amazing project! For the GVR removal I was able to get through the UIWebView issue in my RN 0.62.2 project last night - details here #852 (comment)

amirhosein5858 · 2020-07-26T08:12:19Z

Hi,

Due to #835, I had to use a Viro custom build, and thus tried to also not reference to UIWebView that will not be accepted on the App Store by Apple anymore after December 2020, but it didn't work... Would anyone be able to explain why my approach did not work? Thanks in advance!

Here are the steps I followed, losely inspired from several issues and forks and the various Viro README instructions.
Disclaimer: I am very unexperienced with this kind of manoeuvre, and as such, some or all of this might actually be clumsy or unnecessary

Clone the repo into your workspace with git: git clone https://github.com/mendix/virocore/

Revert to the commit with the fix for NSURLResponse crash: git reset 621b2524f44f8e93a08085b3770d457f5e002e43 --hard

Clone the react-viro repo (named viro) in the same workspace (same parent directory as virocore) with git: git clone https://github.com/mendix/viro.git

Revert to commit with the fix for UIWebView deprecation: git checkout build and git reset ecc4a15e456aeea57d782e189ca2318b829958f5 --hard

$ cd virocore/ios/ and $ pod install

Open ViroRenderer.xcworkspace in Xcode (make sure you open the .xcworkpace file, and not the .xcodeproj file!). Build ViroKit scheme with Build Configuration set to Release and target set to Generic iOS Device

(Optional) A build error might happen as Xcode tries to find ios/Pods/Target Support Files in ios/Pods/Pods/Target Support Files. If so, copy/paste all files and folders from ios/Pods to ios/Pods/Pods

When the build succeeds, you should see an updated armv7_arm64 folder and updated ViroKit.podspec file in the viro/ios/dist/ViroRenderer folder

$ cd viro and $ npm install

$ cd viro/ios and $ pod install

Open ViroReact.xcworkspace in Xcode.

Build ViroReact scheme with Build Configuration set to Release and target set to Generic iOS Device

You should see a new file libViroReact.a at ios/dist/lib/libViroReact.a

Replace your_project/node_modules/react-viro/ios/dist/lib/libViroReact.a with the newly generated ios/dist/lib/libViroReact.a

Replace your_project/node_modules/react-viro/ios/dist/ViroRenderer with the one from viro/ios/dist/ViroRenderer

Build your project

If this also helps, here are my fixed libViroReact.a and ViroRenderer: https://drive.google.com/file/d/1X09c3UUpl7g9MhyJOOwFrMzYMnA0eyx7/view?usp=sharing

Huge thanks to all the contributors that enabled me to patch this together, and especially @dthian, @HedwigJDoets and @HedwigAR!

hi , i try your fixed files , and apple not accept :(
is any steps that i forgot?

haider792 · 2020-11-08T12:10:46Z

@ktemby have you worked with the Vr feature in the react-viro?

nguyenvanphuc2203 · 2021-02-23T02:28:32Z

same issue for me, please help!

cristianoccazinsp mentioned this issue Nov 12, 2019

iOS - Update Google dependencies? viromedia/virocore#320

Open

4 tasks

ChauVV mentioned this issue Jan 11, 2021

v2.13.0 + RN0.57.7 not working when not using !use_frameworks #542

Closed

iOS deprecated UIWebView #718

iOS deprecated UIWebView #718

Comments

cristianoccazinsp commented Sep 10, 2019

Environment

Description

kala888 commented Sep 17, 2019

cristianoccazinsp commented Sep 17, 2019

radvani commented Sep 30, 2019 • edited

cristianoccazinsp commented Oct 4, 2019

vpallegar commented Nov 12, 2019

cristianoccazinsp commented Nov 12, 2019

cristianoccazinsp commented Nov 12, 2019

vpallegar commented Nov 12, 2019

radvani commented Nov 12, 2019 via email

Hless commented Dec 20, 2019 • edited

TwR-Slice commented Jan 31, 2020

Jackyaung commented Mar 20, 2020

Jackyaung commented Mar 20, 2020

cristianoccazinsp commented Apr 9, 2020

cristianoccazinsp commented Apr 9, 2020

cristianoccazinsp commented Apr 10, 2020

ManigandanRaamanathan commented May 3, 2020

ManigandanRaamanathan commented May 3, 2020

dthian commented May 3, 2020

cristianoccazinsp commented May 3, 2020

dthian commented May 3, 2020

silvainSayduck commented May 4, 2020

ManigandanRaamanathan commented May 4, 2020

mbryk commented May 4, 2020

ManigandanRaamanathan commented May 4, 2020 • edited

dthian commented May 4, 2020 • edited

silvainSayduck commented May 5, 2020 • edited

namquyuit commented May 6, 2020 • edited

cristianocca commented May 6, 2020

silvainSayduck commented May 6, 2020

ranjanpoudel1234 commented May 10, 2020

ManigandanRaamanathan commented May 11, 2020 • edited

namquyuit commented May 11, 2020

MinhazMM commented May 11, 2020

ManigandanRaamanathan commented May 11, 2020

quangduz commented May 17, 2020

silvainSayduck commented May 18, 2020

dthian commented May 18, 2020 • edited

Jackyaung commented Jun 5, 2020

ktemby commented Jun 10, 2020 • edited

amirhosein5858 commented Jul 26, 2020

haider792 commented Nov 8, 2020

nguyenvanphuc2203 commented Feb 23, 2021

radvani commented Sep 30, 2019 •

edited

Hless commented Dec 20, 2019 •

edited

ManigandanRaamanathan commented May 4, 2020 •

edited

dthian commented May 4, 2020 •

edited

silvainSayduck commented May 5, 2020 •

edited

namquyuit commented May 6, 2020 •

edited

ManigandanRaamanathan commented May 11, 2020 •

edited

dthian commented May 18, 2020 •

edited

ktemby commented Jun 10, 2020 •

edited