Merge pull request troch#25 from faergeek/allow-star-as-a-param-value

Allow star as a param value
yndx-madfriend · Jun 19, 2018 · 857b5db · 857b5db
2 parents a438585 + 14bb91c
commit 857b5db
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,6 +1,6 @@
 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
@@ -220,6 +220,14 @@ describe('Path', function() {
         should.not.exist(path.test('/users;id=Z12345'))
     })
 
+    it('should match and build paths with star (*) as a parameter value', function() {
+        const path = new Path('/test/:param')
+
+        path.build({ param: 'super*' }).should.equal('/test/super*')
+
+        path.test('/test/super*').should.eql({ param: 'super*' })
+    })
+
     it('should match paths with optional trailing slashes', function() {
         let path = new Path('/my-path')
         should.not.exist(path.test('/my-path/', { strictTrailingSlash: true }))