url: parse, replace % not part of urlencoded character #7236

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@Swaagie
Swaagie commented Mar 4, 2014

Check if % characters are part of an urlencoded char if not replace to prevent malformed uri errors from being thrown, fixes #7234

Will add tests in subsequent commit.

@Swaagie Swaagie [minor] check if % characters are part of an urlencoded char if not r…
…eplace to prevent malformed uri errors from being thrown
2edcc7b

Thank you for contributing this pull request! Here are a few pointers to make sure your submission will be considered for inclusion.

Commit Swaagie/node@2edcc7b has the following error(s):

  • First line of commit message must be no longer than 50 characters
  • Commit message must indicate the subsystem this commit changes

The following commiters were not found in the CLA:

  • Swaagie

You can fix all these things without opening another issue.

Please see CONTRIBUTING.md for more information

Owner
indutny commented Mar 4, 2014

See my reply on the issue.

@indutny indutny closed this Mar 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment