Loosen loose cookie parser. Closes #1112

hapijs · Oct 25, 2013 · 08945ea · 08945ea
1 parent de17571
commit 08945ea
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/state.js b/lib/state.js
@@ -19,7 +19,7 @@ var internals = {
 
 //                       1: name                                                       2: quoted value               3: value
 internals.strictRx = /\s*([^\x00-\x20\(\)<>@\,;\:\\"\/\[\]\?\=\{\}\x7F]+)\s*=\s*(?:(?:"([^\x00-\x20\"\,\;\\\x7F]*)")|([^\x00-\x20\"\,\;\\\x7F]*))(?:(?:;|(?:\s*\,)\s*)|$)/g;
-internals.looseRx =  /\s*([^\x00-\x20\(\)<>@\,;\:\\"\/\[\]\?\=\{\}\x7F]+)\s*=\s*(?:(?:"([^\"]*)")|([^\;]*))(?:(?:;|(?:\s*\,)\s*)|$)/g;
+internals.looseRx =  /\s*([^=\s]+)\s*=\s*(?:(?:"([^\"]*)")|([^\;]*))(?:(?:;|(?:\s*\,)\s*)|$)/g;
 
 
 // Read and parse body

diff --git a/test/unit/state.js b/test/unit/state.js
@@ -78,7 +78,7 @@ describe('State', function () {
 
             var loose = Hapi.utils.clone(Defaults.server.state);
             loose.cookies.strictHeader = false;
-            pass('a="1; b="2"; c=3', { a: '"1', b: '2', c: '3' }, loose);
+            pass('a="1; b="2"; c=3; d[1]=4', { a: '"1', b: '2', c: '3', 'd[1]': '4' }, loose);
 
             var fail = function (header, settings, definitions, result) {