[GitHub] [cordova-android] ath0mas commented on pull request #1111: fix(android): allow file access for existing behavior

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

ath0mas commented on pull request #1111:
URL: https://github.com/apache/cordova-android/pull/1111#issuecomment-719064974


   As described in the upper Motivation, here is the Android point on [setAllowFileAccess](https://developer.android.com/reference/android/webkit/WebSettings#setAllowFileAccess(boolean)) since SDK 30:
   ```
   The default value is true for apps targeting Build.VERSION_CODES.Q and below, and false when targeting Build.VERSION_CODES.R and above.
   ```
   
   This explicit call should be added next to the `settings.setAllowUniversalAccessFromFileURLs(true);` line?
   as both are since API 30 related to deprecated file:// URLs, and suggested use of [androidx.webkit.WebViewAssetLoader](https://developer.android.com/reference/androidx/webkit/WebViewAssetLoader)
   (and maybe move this "group" a bit from the use of unrelated `setMediaPlaybackRequiresUserGesture`)


----------------------------------------------------------------
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