search

foundeo / ubuntu-nginx-lucee

Script for standing up a Lucee server using nginx and Tomcat on Ubuntu
Apache License 2.0
86 stars 47 forks source link

Exclude query string from path_info #46

Open cubiclabs opened 5 months ago

cubiclabs commented 5 months ago

The current regex for path_info includes the query string if one is present.

path_info should not include the query string and the regex tweak excludes it.

pfreitag commented 5 months ago

Doesn't $uri exclude the query string? https://stackoverflow.com/questions/48708361/nginx-request-uri-vs-uri

cubiclabs commented 5 months ago

Looking at the docs it says that the value of $uri can change during the request: https://nginx.org/en/docs/http/ngx_http_core_module.html#uri

For context, I am using Nginx v 1.22.1 and I have some rewrites in place that are essentially the same as the ColdBox one here: https://coldbox.ortusbooks.com/the-basics/routing/requirements/rewrite-rules#nginx

I wonder if this is affecting things in some way causing the query string to get included in the $uri and subsequently the path_info?