[GitHub] [cordova-ios] dpogue commented on pull request #1249: (ios) Add LimitsNavigationsToAppBoundDomains configuration key

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

dpogue commented on PR #1249:
URL: https://github.com/apache/cordova-ios/pull/1249#issuecomment-1228640719

   Thanks for the pull request! You probably want to target this to the master branch though.
   
   I think app-bound domains are definitely something we want to support in cordova-ios, particularly because it will enable easier access to a bunch of APIs, as you mentioned.
   
   However, I think for this to work, it will also need to populate the `WKAppBoundDomains` array in the application's Info.plist file with a list of domains. The blog post you linked is using `config-file` directives in config.xml to inject the domains they want into the `WKAppBoundDomains` list, but it probably makes sense to try to automatically populate that from a combination of the `hostname` value, `access` tags, and `allow-navigation` tags. I believe any communication with domains **not** listed will be blocked.


-- 
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-unsubscribe@cordova.apache.org

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