nystudio107 / nginx-craft

An Nginx virtual host configuration for Craft CMS that implements a number of best-practices.
MIT License
310 stars 37 forks source link

client_max_body_size #10

Closed samuelbirch closed 6 years ago

samuelbirch commented 6 years ago

it would be good to include client_max_body_size to match craft's default of 16M

khalwat commented 6 years ago

There's a note for it in the docs, but you're right, I should include it in the config as well.

khalwat commented 6 years ago

Done in 1.0.13:

https://github.com/nystudio107/nginx-craft/releases/tag/1.0.13

I actually ended up doing:

    # Disable limits on the maximum allowed size of the client request body
    client_max_body_size 0;

...so we can just let Craft decide what the limits should be.

http://nginx.org/en/docs/http/ngx_http_core_module.html#client_max_body_size