Fix panic when using URLize #2274

Merged
merged 1 commit into from Jul 11, 2016

Projects

None yet

3 participants

@biilmann
Contributor

Using URLize on a string like '100%-true' would cause a panic.

This fixes the issue by verifying that a % is followed by two hex characters in the UnicodeSanitize function in the path helper.

@biilmann biilmann Fix panic when using URLize
Using URLize on a string like '100%-true' would cause a panic
ad78ade
@bep bep merged commit 330639d into spf13:master Jul 11, 2016

4 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/wercker Wercker build passed
Details
licence/cla Contributor License Agreement is signed.
Details
@anthonyfok anthonyfok added this to the v0.17 milestone Jul 11, 2016
@anthonyfok anthonyfok added the kind/bug label Jul 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment