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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
