Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mayan-EDMS - default max upload size #1273

Closed
wildgarden opened this issue Dec 5, 2018 · 4 comments
Closed

Mayan-EDMS - default max upload size #1273

wildgarden opened this issue Dec 5, 2018 · 4 comments

Comments

@wildgarden
Copy link

wildgarden commented Dec 5, 2018

Freshly installed Mayan-EDMS appliance (15) uses nginx default upload size of 1 MB. In practice this value is too low and a upload of a huge document results in a 413 (Request Entity Too Large) error.
The value has to be manually adjusted in the nginx conf (client_max_body_size).
https://nginx.org/en/docs/http/ngx_http_core_module.html#client_max_body_size

@JedMeister
Copy link
Member

JedMeister commented Dec 5, 2018

Great catch! Thanks for that.

Just for clarity, have you adjusted the client_max_body_size within the location block? Or somewhere else?

Also what do you think a reasonable default is? 50MB?

I propose that we add something like this:

    client_max_body_size 50M;  # Increase to upload bigger documents
    proxy_read_timeout 30s;  # Increase if your document uploads take more than 30 seconds

@JedMeister JedMeister added this to the 15.1 milestone Dec 5, 2018
@wildgarden
Copy link
Author

I tested both, first the global http in /etc/nginx/nginx.conf and later the server block in /etc/nginx/sites-available/mayan. I think location should work, too.

Regarding a sane default value: I think, in general 50MB might be enough for a typical user. But I have some PDFs > 100 MB. Thus I choose 200MB for my personal needs - just to be safe in future.

Adding proxy_read_timeout is a good advice, thanks.

@JedMeister
Copy link
Member

Ok thanks for that and thanks again for the initial notification. We'll aim to include a fix for this in the next release.

@JedMeister
Copy link
Member

Closed by turnkeylinux-apps/mayan-edms@43d5174 - applaince will be rebuilt shortly with this and some other minor adjustments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants