Merge pull request #19 from andyburke/fix_explicit_path

Fix an issue with explicit_path

Author: bmeck

cookiejar.js

@@ -133,10 +133,8 @@
         if (this.path && access_info.path.indexOf(this.path) !== 0) {
             return false;
         }
-        if (!this.explicit_path) {
-           if (this.path !== access_info.path) {
-               return false;
-           }
+        if (this.explicit_path && this.path !== access_info.path) {
+           return false;
         }
         var access_domain = access_info.domain && access_info.domain.replace(/^[\.]/,'');
         var cookie_domain = this.domain && this.domain.replace(/^[\.]/,'');
@@ -209,6 +207,7 @@
                         }
                         continue;
                     }
+
                     if (cookie.matches(access_info)) {
                         return cookie;
                     }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "cookiejar",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "author": {
     "name": "bradleymeck"
   },

tests/test.js

@@ -70,8 +70,8 @@ var cookie = test_jar.getCookie("sub", CookieAccessInfo("test.com", "/"));
 assert.equal(cookie.name, "sub");
 assert.equal(cookie.domain, "test.com");
 
-test_jar.setCookie(new Cookie("sub=4;", "test.com", "/accounts"));
-var cookie = test_jar.getCookie("sub", CookieAccessInfo("test.com", "/"));
+test_jar.setCookie(new Cookie("sub=4;path=/accounts", "test.com", "/accounts"));
+var cookie = test_jar.getCookie("sub", CookieAccessInfo("test.com", "/foo"));
 assert.equal(cookie, undefined);
 
 var cookie = test_jar.getCookie("sub", CookieAccessInfo("test.com", "/accounts"));