PieterVanPoyer edited a comment on pull request #1004: URL: https://github.com/apache/cordova-ios/pull/1004#issuecomment-705767622
Hey Niklas I think this is a great addition. Not having to deal with CORS issue, will make building a Cordova app a lot nicer. I cannot review the code as I am no ios developer, but I think the addition of code looks good structured and is rather small for such a great feature. Will it also work with sending multipartform data to the backend? [ multipart/form-data ] Will it also work for loading local assets like local images or translations or config jsons? In my opinion this must be an addition to the ios platform. This should be the default way for Http requests. Maybe there should be a flag or setting to Opt out of the proxy. (If this is not yet foreseen) Another remark is the ability to inspect the HTTP traffic. We are used to inspect it with the network inspector of the browser, but this will not be possible anymore. Maybe there must be an addition in some documentation about how to inspect the network traffic with this setup. Kind regards Pieter ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
