You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2020/10/12 03:03:31 UTC

[GitHub] [apisix] idbeta opened a new issue #2392: request help: why this route not work

idbeta opened a new issue #2392:
URL: https://github.com/apache/apisix/issues/2392


   ### Issue description
   I have a strange idea about uri, so I tested it, but it didn't work, pls help me out
   ```shell
   $ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -i -d '{"uri": "/hello-_~!*();:@&=+$,/[]", "upstream": {"nodes": {"127.0.0.1:9666": 1}, "type": "roundrobin"}}'
   HTTP/1.1 200 OK
   ... ...
   ```
   test route
   ```shell
   $ curl -i 'http://127.0.0.1:9080/hello-_~%21%2a%28%29%3b%3a%40%26%3d%2b%24%2c/%5b%5d'
   HTTP/1.1 404 Not Found
   ... ...
   ```
   test the original address
   ```shell
   $ curl -i 'http://127.0.0.1:9666/hello-_~%21%2a%28%29%3b%3a%40%26%3d%2b%24%2c/%5b%5d'
   HTTP/1.1 200 OK
   ... ..
   ```
   ### Environment
   
   * apisix version (cmd: `apisix version`):  master
   * OS: macos
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix] moonming closed issue #2392: bug: return `400` for invalid uri `/hello-_~!*();:@&=+$,/[]` in route

Posted by GitBox <gi...@apache.org>.
moonming closed issue #2392:
URL: https://github.com/apache/apisix/issues/2392


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix] membphis commented on issue #2392: request help: why this route not work

Posted by GitBox <gi...@apache.org>.
membphis commented on issue #2392:
URL: https://github.com/apache/apisix/issues/2392#issuecomment-707423464


   > ```shell
   > "uri": "/hello-_~!*();:@&=+$,/[]",
   > ```
   
   that is an invalid path for APISIX. the `*` mainly use for prefix matching.
   
   the admin API should return `400` for this case.
   
   here is the link of the APISIX router: https://github.com/api7/lua-resty-radixtree#prefix-match


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix] moonming closed issue #2392: bug: return `400` for invalid uri `/hello-_~!*();:@&=+$,/[]` in route

Posted by GitBox <gi...@apache.org>.
moonming closed issue #2392:
URL: https://github.com/apache/apisix/issues/2392


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix] idbeta commented on issue #2392: request help: why this route not work

Posted by GitBox <gi...@apache.org>.
idbeta commented on issue #2392:
URL: https://github.com/apache/apisix/issues/2392#issuecomment-707026660


   ping @membphis 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org