Proxy doesn't always respect "strip_uri" with multiple "uris"

[sgillespie]

Summary

If I define an API with multiple URIs, and strip_uri = true, Kong doesn't always strip the matching prefix. When I first start Kong, it works with both URIs, then one of them stripping the prefix. Full example below.

Steps To Reproduce

1. Start a kong:0.10.2 docker image

   # Start Postgres
   docker run -d --name kong-database \
       -p 9042:9042 \
       cassandra:2.2
   
   # Start nginx
   docker run -d -p 80:80 --name nginx nginx
   
   # Start Kong
   docker run -d --name kong \
      --link kong-database:kong-database --link nginx:nginx \
      -e "KONG_DATABASE=postgres" \
      -e "KONG_PG_HOST=kong-database" \
      -p 8000:8000 \
      -p 8443:8443 \
      -p 8001:8001 \
      -p 7946:7946 \
      -p 7946:7946/udp \
      kong:0.10.2
   

2. Add an API with multiple URIs

   curl -H 'Content-Type: application/json' \
       -X POST \
       -d '{ "name": "nginx", "uris": ["/x/y/z", "/z/y/x"], "upstream_url": "http://nginx", "strip_uri": true}' \
       http://localhost:8001/apis           
   

3. Request the API in both forms multiple times (it will eventually start happening)

   curl http://localhost:8000/x/y/z/a/b/c
   curl http://localhost:8000/z/y/x/a/b/c
   curl http://localhost:8000/x/y/z/a/b/c
   curl http://localhost:8000/z/y/x/a/b/c
   curl http://localhost:8000/x/y/z/a/b/c
   curl http://localhost:8000/z/y/x/a/b/c
   

4. Check the nginx logs (404s are expected):

   docker logs -f nginx

   172.17.0.4 - - [25/May/2017:13:53:24 +0000] "GET /a/b/c
   172.17.0.4 - - [25/May/2017:13:53:33 +0000] "GET /a/b/c
   172.17.0.4 - - [25/May/2017:13:53:41 +0000] "GET /a/b/c
   172.17.0.4 - - [25/May/2017:13:53:45 +0000] "GET /a/b/c
   172.17.0.4 - - [25/May/2017:13:53:50 +0000] "GET /x/y/z/a/b/c

You can see above that it eventually stops stripping the prefix on /x/y/z. However, /z/y/x still works as expected

Additional Details & Logs

• Kong version: 0.10.2 (Also tested on 0.10.1 and "latest")
• Operating System: ArchLinux (vanilla kernel 4.10.13) and Docker 17.05.0-ce

[thibaultcha]

Hi,

Thanks for the report, we'll be looking into it.

[thibaultcha]

Fix proposed in #2582!

[thibaultcha]

This will be released in 0.10.4 or 0.11.0. Thanks again for the report!

[marccardinal]

Currently affected by this, do you have a nightly build of the kong container that would contain this fix?

[thibaultcha]

No, sorry.