jshttp#5 ignore escape error and return original value better than th…

…row the error.

index.js

@@ -50,8 +50,20 @@ var parse = function(str) {
  return obj;
 };
 
-var encode = encodeURIComponent;
-var decode = decodeURIComponent;
+var encode = function(val) {
+ try {
+ return encodeURIComponent(val);
+ } catch (e) {
+ return val;
+ }
+};
+var decode = function(val) {
+ try {
+ return decodeURIComponent(val);
+ } catch (e) {
+ return val;
+ }
+};
 
 module.exports.serialize = serialize;
 module.exports.parse = parse;

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'));
+});

test/serialize.js

@@ -48,6 +48,11 @@ test('escaping', function() {
  assert.deepEqual('cat=%2B%20', cookie.serialize('cat', '+ '));
 });
 
+test('ignore escaping error and return original value', function() {
+ var s = String.fromCharCode(0xDFFF);
+ assert.deepEqual('cat=' + s, cookie.serialize('cat', s));
+});
+
 test('parse->serialize', function() {
 
  assert.deepEqual({ cat: 'foo=123&name=baz five' }, cookie.parse(