ubuntu-nginx-lucee icon indicating copy to clipboard operation
ubuntu-nginx-lucee copied to clipboard

Exclude query string from path_info

Open cubiclabs opened this issue 1 year ago • 2 comments

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.

cubiclabs avatar Apr 18 '24 20:04 cubiclabs

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

pfreitag avatar Apr 18 '24 20:04 pfreitag

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?

cubiclabs avatar Apr 19 '24 08:04 cubiclabs