-
Notifications
You must be signed in to change notification settings - Fork 25.2k
Warn about routing with special chars /6 #28213
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@SchroterQuentin @ArthurHNL @DanielBryars @skovalyova @talshloman @celluj34 @thegrahamking please review and suggest better wording. I'm hoping to make this very visible so folks can find it before spending a lot of time debugging. Hopefully those who hit this problem will click on the GitHub issue and provide feedback so we can get a good idea how common this problem is. |
I've only observed the behaviour with |
| ----| ---------- | | ||
| `/` | `%2F` | | ||
| `+` | `%2B` | | ||
| ` ` | `+` | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+
->
is form encoding, not url encoding, it shouldn't affect the path.
@Rick-Anderson Sorry for my late reply, I got quite sick over the weekend. But this change looks good. Thanks for clearing this up in the documentation. |
1 similar comment
@Rick-Anderson Sorry for my late reply, I got quite sick over the weekend. But this change looks good. Thanks for clearing this up in the documentation. |
i feel like |
* Warn about routing with special chars /6 * Update aspnetcore/includes/routeSlash.md
Fixes #27900
Internal review URL
Public review of the doc build:
review.
from URL.