KenCorbettJr commented on code in PR #1609:
URL: https://github.com/apache/cordova-android/pull/1609#discussion_r1181882515


##########
templates/project/res/xml/opener_paths.xml:
##########
@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!-- 
https://developer.android.com/reference/android/support/v4/content/FileProvider.html#SpecifyFiles
 -->
+<paths xmlns:android="http://schemas.android.com/apk/res/android";>

Review Comment:
   Great idea.  I added a comment to the `cdv_core_file_provider_paths.xml` 
file that reads:
   
   >  Note: This File provider may only be used by the Cordova core itself and 
should not be used for responding to Intents because it will expose the app's 
private data folders.
   
   Does that sound OK?



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

To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org

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


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

Reply via email to