Force encoding of US-ASCII to UTF-8 in unescape_uri. #16123

Merged
merged 1 commit into from Jul 10, 2014

Conversation

Projects
None yet
2 participants

Because URI paths may contain non US-ASCII characters we need to force
the encoding of any unescaped URIs to UTF-8 if they are US-ASCII.
This essentially replicates the functionality of the monkey patch to
URI.parser.unescape in active_support/core_ext/uri.rb.

Fixes #16104.

Force encoding of US-ASCII to UTF-8 in unescape_uri.
Because URI paths may contain non US-ASCII characters we need to force
the encoding of any unescaped URIs to UTF-8 if they are US-ASCII.
This essentially replicates the functionality of the monkey patch to
URI.parser.unescape in active_support/core_ext/uri.rb.

Fixes #16104.

@pixeltrix pixeltrix self-assigned this Jul 10, 2014

pixeltrix added a commit that referenced this pull request Jul 10, 2014

Merge pull request #16123 from karlentwistle/issues/9299
Force encoding of US-ASCII to UTF-8 in unescape_uri.

@pixeltrix pixeltrix merged commit 6b6832e into rails:master Jul 10, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment