joyent/node — pull request #8261 by twberger (2014-08-26 09:12:57+00:00)
Title: Fix issue 4055, querystring.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.