Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update running-on-device.md #1312

Merged
merged 1 commit into from
Oct 7, 2019
Merged

Update running-on-device.md #1312

merged 1 commit into from
Oct 7, 2019

Conversation

RobinCsl
Copy link
Contributor

@RobinCsl RobinCsl commented Oct 1, 2019

NSAllowsArbitraryLoads was introduced in facebook/react-native#19643 and defaults to true. That means that removing localhost from NSExceptionDomains is not enough anymore, as describe in #1058.

Kudos to @Ashoat who tracked down the origin of the issue: #1058 (comment)

Closes #1058

`NSAllowsArbitraryLoads` was introduced in facebook/react-native#19643 and defaults to `true`. That means that removing `localhost` from `NSExceptionDomains` is not enough anymore, as describe in #1058.

Kudos to @Ashoat who tracked down the origin of the issue: #1058 (comment)

Closes #1058
@react-native-bot
Copy link

Deploy preview for react-native ready!

Built with commit 86f6b98

https://deploy-preview-1312--react-native.netlify.com

Changes to docs/ are reflected in the next "master" version.

Thank you for your contributions.

How to ContributeDocumentation Sources

@elicwhite elicwhite merged commit 615122a into facebook:master Oct 7, 2019
@RobinCsl RobinCsl deleted the patch-1 branch October 28, 2019 12:47
espipj pushed a commit to espipj/react-native-website that referenced this pull request Feb 8, 2020
`NSAllowsArbitraryLoads` was introduced in facebook/react-native#19643 and defaults to `true`. That means that removing `localhost` from `NSExceptionDomains` is not enough anymore, as describe in facebook#1058.

Kudos to @Ashoat who tracked down the origin of the issue: facebook#1058 (comment)

Closes facebook#1058
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Question: Are the instructions for production AppTransportSecurity complete?
3 participants