Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

remove variable leak #15

Closed
wants to merge 1 commit into from

2 participants

@Filirom1

Fixed thanks to mocha :
Error: global leak detected: name

@isaacs
Owner

Wow, whoops. Thanks for the patch, landed and published!

@isaacs isaacs closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 27, 2013
  1. @Filirom1

    remove variable leak

    Filirom1 authored
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 0 deletions.
  1. +1 −0  lib/request.js
View
1  lib/request.js
@@ -258,6 +258,7 @@ function requestDone (method, where, cb) {
if (parsed && parsed.error && response.statusCode >= 400) {
var w = url.parse(where).pathname.substr(1)
+ var name
if (!w.match(/^-/) && parsed.error === "not_found") {
w = w.split("/")
name = w[w.indexOf("_rewrite") + 1]
Something went wrong with that request. Please try again.