[GitHub] [cordova-docs] jcesarmobile commented on pull request #1172: doc(android): document `AndroidInsecureFileModeEnabled`

[GitBox <gi...@apache.org>]

jcesarmobile commented on pull request #1172:
URL: https://github.com/apache/cordova-docs/pull/1172#issuecomment-833381794


   I've fixed the merge conflicts after merging my PR that removed the AndroidX preference.
   I edited right before you commented, so you probably didn't see, can you add the preference to the examples section too?
   
   If you don't want to go into detail about the data loss in this PR it's ok, but it should still be about **loading** from file instead of **accessing** file urls, because it's not the same thing.
   If you load from file url, you obviously need to access file urls, but you might still want to load from https and access file urls (i.e. take a picture from the camera and assign it to a img tag) and can wrongly enable this preference to do so.
   
   BTW, that example made me think, do we have some helper like on iOS to convert file urls like the one you get from camera to https urls? would the assets loader be able to load those urls somehow?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org