Skip to content

Don't decode pathname before matching#314

Open
mlohbihler wants to merge 1 commit intovisionmedia:masterfrom
mlohbihler:master
Open

Don't decode pathname before matching#314
mlohbihler wants to merge 1 commit intovisionmedia:masterfrom
mlohbihler:master

Conversation

@mlohbihler
Copy link
Copy Markdown

Path should already have been decoded by the time this method is called.

Code:

    this.route('/organization/:name/:action?/:subAction?', function(context) {

If i go to "/organization/test%2Ftest", it does not work as expected unless the given code change is made. Before the change the result is "name" = "test", "action" = "test". After the change "name" = "test/test", "action" = null, as expected.

Path should already have been decoded by the time this method is called.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant