fix: match pipe characters in parameters (Firefox)

troch · Apr 9, 2018 · d6e6d27 · d6e6d27
1 parent c4b4c4d
commit d6e6d27
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/modules/rules.ts b/modules/rules.ts
@@ -1,5 +1,5 @@
 export const defaultOrConstrained = (match: string): string =>
-    '(' + (match ? match.replace(/(^<|>$)/g, '') : "[a-zA-Z0-9-_.~%':]+") + ')'
+    '(' + (match ? match.replace(/(^<|>$)/g, '') : "[a-zA-Z0-9-_.~%':|]+") + ')'
 
 export type RegExpFactory = (match: any) => RegExp
 

diff --git a/test/main.js b/test/main.js
@@ -272,4 +272,12 @@ describe('Path', function() {
         path.test('/Test').should.eql({})
         should.not.exist(path.test('/TEST', { caseSensitive: true }))
     })
+
+    it('should match unencoded pipes (Firefox)', () => {
+        const path = new Path('/test/:param')
+
+        path.test('/test/1|2').should.eql({
+            param: '1|2'
+        })
+    })
 })