Andrew Clayton
Andrew Clayton
> 1. It's better to keep the order: header, body, etc. Move the related "header" in front of "body" options. Sure. I'll stick it before 'body_buffer_size'. > 2. Is "max_header_size"...
Changes :- - Fixed typo in nxt_router.c - Re-ordered config setting - Added a couple of tests
Added review requests for @andrey-zelenkov to check over the tests. @artemkonev as this will require documentation updates.
Changes :- - Tweak to the changelog
Needing to terminate the URL with a '/' seems to be intended, from the docs... ``` index | Filename appended to any URI paths ending with a slash; applies if...
I agree, this could probably be handled better. IIRC (memory is likely a little hazy) in the early days of the web if the last component of a URL was...
@mwoodpatrick Here's a patch against current unit that allows to skip the trailing '/' on PHP URLs that are directories, if you'd like to give it a spin... (updated to...
@mwoodpatrick Here's an preliminary version of an alternative fix for this issue. It uses a 301 redirect to point to the path with a trailing '/', if it's a directory....
The fix for this has been merged. [a032744](https://github.com/nginx/unit/commit/a03274456b54cbc39e220b9dd73c3fc3fb935e46)
So, if I'm reading the above correctly, this issue was caused by container resource limits being hit?