-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Kong does not expect matrix parameters when proxy by request path is used #1510
Comments
+1 |
referring to the last paragraph of 3.3
Then I don't think that this is a standard. Using the ';' character as a delimiter is an application choice, it's custom. It's just that the spec allows you to do it, but it might as well have been any other character 'a', 'z' or anything in between. So by definition it will be impossible to support all possible delimiter-scheme's people can come up with. Or am I missing something here? |
I guess you're right. What about regular expressions? For instance, in this case, instead of configuring the request path to: /some/api, it could be like: /some/api((;)(.)*). I know Kong does not support it yet, but, it could be very useful and resolve other issues like this one: #677. |
Regex support routing is on our roadmap for 0.10 or 0.11. Closing this in the meanwhile, I believe we've answered your concerns. |
Summary
It seems that Kong does not expect matrix parameters when we use proxy by request path. For example, if I do the following call, Kong says there's no api for this request:
GET /some/api;parameter=value
Trying make it work, I discovered that if I add a slash before the matrix parameters, it seems to work as expected:
GET /some/api/;parameter=value
I suppose Kong sees ";" as any other usual character, so, if you make a GET to /some/api;parameter=value it expects an api configured with a request path /some/api;parameter=value, and not just /some/api.
This is slightly annoying and does not match the spec, so, I'd hope it'd be fixed.
Steps To Reproduce
Additional Details & Logs
The text was updated successfully, but these errors were encountered: