Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: nodejs/node-v0.x-archive
base: f39e608c6eb1
Choose a base ref
...
head repository: nodejs/node-v0.x-archive
compare: 0f2956192c51
Choose a head ref
  • 1 commit
  • 2 files changed
  • 1 contributor

Commits on Aug 27, 2014

  1. querystring: fix unescape override 

    Documentation states that `querystring.unescape` may be overridden to
replace unescaper during parsing. However, the function was only
being used as a fallback for when the native decoder throws (on a
malformed URL). This patch moves the call to the native function and
the try/catch around it into querystring.unescape then has the parser
always invoke it, so that an override will always be used.

Fixes #4055

Reviewed-By: Fedor Indutny <fedor@indutny.com>
    @indutny
    Tristan Berger authored and indutny committed Aug 27, 2014
    Copy the full SHA
    0f29561 View commit details
    Browse the repository at this point in the history