Added fix to ensure that the request_domain and request_path get passed to the Cookie constructor.

danielmcq · bmeck · commit ab4106c3a7e1 · 2015-09-08T15:30:15.000-05:00
diff --git a/cookiejar.js b/cookiejar.js
@@ -33,7 +33,7 @@
             }
             return this;
         }
-        return new Cookie(cookiestr);
+        return new Cookie(cookiestr, request_domain, request_path);
     }
     exports.Cookie = Cookie;
 
@@ -247,7 +247,9 @@
         var successful = [],
             i,
             cookie;
-        cookies = cookies.map(Cookie);
+        cookies = cookies.map(function(item){
+            return Cookie(item, request_domain, request_path);
+        });
         for (i = 0; i < cookies.length; i += 1) {
             cookie = cookies[i];
             if (this.setCookie(cookie, request_domain, request_path)) {

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@`
`33`	`33`	`}`
`34`	`34`	`return this;`
`35`	`35`	`}`
`36`		`- return new Cookie(cookiestr);`
	`36`	`+ return new Cookie(cookiestr, request_domain, request_path);`
`37`	`37`	`}`
`38`	`38`	`exports.Cookie = Cookie;`
`39`	`39`
`@@ -247,7 +247,9 @@`
`247`	`247`	`var successful = [],`
`248`	`248`	`i,`
`249`	`249`	`cookie;`
`250`		`- cookies = cookies.map(Cookie);`
	`250`	`+ cookies = cookies.map(function(item){`
	`251`	`+ return Cookie(item, request_domain, request_path);`
	`252`	`+ });`
`251`	`253`	`for (i = 0; i < cookies.length; i += 1) {`
`252`	`254`	`cookie = cookies[i];`
`253`	`255`	`if (this.setCookie(cookie, request_domain, request_path)) {`