Missing unescape() from self._qs breaks requests with authentication

[andornaut]

GET requests with http basic authentication credentials such as "http://user:pass@example.com" fail with error "self._qs.unescape is not a function".

This error occurs in request.js:

  if (self.uri.auth && !self.hasHeader('authorization')) {
    var uriAuthPieces = self.uri.auth.split(':').map(function(item) {return self._qs.unescape(item)})
    self.auth(uriAuthPieces[0], uriAuthPieces.slice(1).join(':'), true) 
  }

Changing Querystring.prototype.unescape to use the global unescape() or decodeURIComponent appears to resolve this issue.

Environment: Electron 0.27.2 (chrome 43)

[simov]

That might be related to #1600 but tbh I don't see how's that possible. I need more context, can you provide a short code example that reproduces that bug?

[andornaut]

It seems that the "querystring" library supplied by Browserify does not include querystring.unescape and therefore this assignment doesn't work as expected.

Refs:
browserify/browserify#826
mike-spainhower/querystring#4
Gozala/querystring#6

I'm closing this issue as it does not appear to be caused by a bug in this library.

[sagivo]

@andornaut did you find how to fix it on electron? i still get this error

[dotbloup]

I note that unescape is deprecated: (reference)
consequently self._qs.unescape(item) should be replaced by decodeURI(item)