-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Route issue #2579
Comments
The issue template is there for a reason. It's asking you to fill out the details, like which versions you're using, which host your running etc., so we don't have to. But here we are... asking; what host and what versions are you using? 😝 |
There's also a duplicate Stack Overflow question for this by @Radzhab http://stackoverflow.com/questions/39785864/how-pass-url-params-to-route |
This looks like the old "twice decoding" thing? |
Double encoding would resolve the issue:
When this is either not encoded, or is encoded (once), the URL is decoded and end's up being split on the |
I use 1.4.1.0 version |
And the host? |
https://gist.github.com/Radzhab/46abc95cdfe4b018ad3c9a8dae7003ca |
Yeah, that's probably the double-decoding issue that affected self-hosting. This was fixed in #2462 You either have to update to one of the 2.0 pre-release versions or double encode the value you're passing in 😢 |
I intall pre-release but it all the same - not work.. I try this url |
So double encoding did not work for you @Radzhab? Hmmm it should because we only decode the route once. That's strange. I have some free time tonight so I'll try reproduce this and see if there's a quick work around to unblock you. |
update....... |
I have route
Get["/{url}/{printer}"]
If i send this url to Chrome all work fine
localhost:8888/yandex.ru/HP Universal Printing PCL 6
Then i try send this url, but route not work.
localhost:8888/http://www.yandex.ru/HP Universal Printing PCL 6
I also try HttpUtility.UrlEncode("http://www.yandex.ru"); but it not work too.
localhost:8888/http%3a%2f%2fwww.yandex.ru/HP Universal Printing PCL 6
404 - NotFound
The resource you have requested cannot be found.
We're sorry :-(
The text was updated successfully, but these errors were encountered: