fixed jshttp#5 ignore escape error and return original value better t…

…han throw the error.
fengmk2 · Oct 29, 2012 · 1ae8966 · 1ae8966
1 parent e86c424
commit 1ae8966
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -43,7 +43,11 @@ var parse = function(str) {
 
  // only assign once
  if (undefined == obj[key]) {
- obj[key] = decode(val);
+ try {
+ obj[key] = decode(val);
+ } catch (e) {
+ obj[key] = val;
+ }
  }
  });
 

diff --git a/test/parse.js b/test/parse.js
@@ -23,3 +23,6 @@ test('escaping', function() {
  cookie.parse('email=%20%22%2c%3b%2f'));
 });
 
+test('ignore escaping error and return original value', function() {
+ assert.deepEqual({ foo: '%1', bar: 'bar' }, cookie.parse('foo=%1;bar=bar'));
+});