Skip to content

Escape URL string before creating an NSURL - #786

Closed
ghost wants to merge 1 commit into
masterfrom
unknown repository
Closed

Escape URL string before creating an NSURL#786
ghost wants to merge 1 commit into
masterfrom
unknown repository

Conversation

@ghost

Copy link
Copy Markdown

If we make a request that contains a pipe character, URLWithString will return null. This fix escapes these and other characters before creating the NSURL.

If we make a request that contains a pipe character, URLWithString will return null. This fix escapes these and other characters before creating the NSURL.
@facebook-github-botfacebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 10, 2015
@vjeuxvjeux closed this in f7e3d59Apr 13, 2015
vjeux pushed a commit to vjeux/react-native that referenced this pull request Apr 14, 2015
Summary:
If we make a request that contains a pipe character, URLWithString will return null. This fix escapes these and other characters before creating the NSURL.
Closesreact#786
Github Author: Blair Vanderhoof <blairv@gmail.com>
Test Plan: Imported from GitHub, without a `Test Plan:` line.
vjeux pushed a commit to vjeux/react-native that referenced this pull request Apr 15, 2015
Summary:
If we make a request that contains a pipe character, URLWithString will return null. This fix escapes these and other characters before creating the NSURL.
Closesreact#786
Github Author: Blair Vanderhoof <blairv@gmail.com>
Test Plan: Imported from GitHub, without a `Test Plan:` line.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@facebook-github-bot