-
Notifications
You must be signed in to change notification settings - Fork 127
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
URL with square brackets #117
Comments
The valid characters in an URI are:
The other ASCII characters are categorised as following:
This is taken from RFC 3986 (Uniform Resource Identifier (URI): Generic Syntax) Hence square brackets cannot be used as-is. However these can be used with Percent Encoding. |
The The C HTTP parser that we're using has some functionality for url parsing, we might be able to use that. Would probably make url parsing faster too |
We've been supplied files from a third party supplier with files that contain square brackets in the name, i.e
[root.m].js
This is causing the following error:
Invalid data, parser failed with code 2
As the parsing fails the
request.uri
returns/
and subsequently the resource fails to load.The text was updated successfully, but these errors were encountered: