API and query parameters

[cedric....@motwin.com]

Hello, 

I am using Kong 4.2 and I've got a question about query parameters.

I have registered my API with the "path" option as follow:

curl -i -X POST --url http://localhost:8001/apis/ --data 'name=stockmarket' --data 'target_url=http://demo-streamdataio.rhcloud.com/' --data 'path=/stockmarket/prices'

When I curl with no query params, that's ok. I get my result. But if I curl with a query param, I've got an error:

curl -i -X GET --url "http://localhost:8000/stockmarket/prices?foo=bar" --header 'Host: demo-streamdataio.rhcloud.com'

HTTP/1.1 404 Not Found

Date: Fri, 21 Aug 2015 12:03:38 GMT

Content-Type: application/json; charset=utf-8

Connection: close

Server: kong/0.4.2

Via: kong/0.4.2

{"public_dns":["demo-streamdataio.rhcloud.com"],"message":"API not found with these values","path":"\/stockmarket\/prices?foo=bar"}

I didn't expect to get a "404 Not Found" when querying the API with a query param but my result. Am I wrong with my expectation or is it a bug?

Thanks in advance,

Cédric.

[cedric....@motwin.com]

Well, having a look at the code, I've managed to get something working by adding a '/':

curl -i -X GET --url "http://localhost:8000/stockmarket/prices/?toto=oto" --header 'Host: demo-streamdataio.rhcloud.com'

Nonetheless, I would tend to say it's a bug because most of the time you don't have a ended '/' before the '?' of the query param. Am I wrong?

Cédric.

PS: « faulty » code is in kong/resolver/access.lua ~ l. 156 (local m, err = ngx.re.match(request_uri, "^"..path.."/"))

[David Carr]

This feels like a bug to me.

[Ahmad Nassri]

opened an issue here for easy tracking: https://github.com/Mashape/kong/issues/495

thanks for reporting.

[Ahmad Nassri]

you can track this issue here: https://github.com/Mashape/kong/issues/495

thanks for reporting.

[Thibault Charbonnier]

Hi,

Here is the fix: https://github.com/Mashape/kong/pull/496

Thanks for reporting.

[Cédric Tran-Xuan]

Many thanks for the fix!

Cédric.