add decodeURIComponent, decodeURI does not always catch the path when…

… decoding with test
heapwolf · Apr 22, 2015 · a503730 · a503730
1 parent 8ceaa1c
commit a503730
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -23,6 +23,7 @@ function regify(vurl) {
 
 function paramify(url) {
   try { url = decodeURI(url) } catch($) {}
+  try { url = decodeURIComponent(url) } catch($) {}
 
   function match (vurl) {
 

diff --git a/test/paramify.js b/test/paramify.js
@@ -49,4 +49,13 @@ test('regex fail to match', function (t) {
 
   match = paramify('/@username')
   t.assert(!match(/^\/(xprofile|@#(.*?))/))
-})
+})
+
+test('encoded paths', function(t) {
+  t.plan(3)
+
+  var match = paramify('/@example%2ftest')
+  t.assert(match('/@:first/:second'))
+  t.equal(match.params.first, 'example')
+  t.equal(match.params.second, 'test')
+})