Sitelet https://web.archive.org/web/20201113063027/https://github.com/zulip/zulip/issues/16741
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

Setting MAX_FILE_UPLOAD_SIZE > 25m requires a change to nginx as well #16741

Open
alexmv opened this issue Nov 13, 2020 · 4 comments
Open

Setting MAX_FILE_UPLOAD_SIZE > 25m requires a change to nginx as well #16741

alexmv opened this issue Nov 13, 2020 · 4 comments

Comments

@alexmv
Copy link
Contributor

@alexmv alexmv commented Nov 13, 2020

The default MAX_FILE_UPLOAD_SIZE size is 25 megabytes -- and while we mention not setting it above 80, we don't clarify that the cap of 25 is also currently enforced at the nginx level.

We should probably increase the cap to 80 at the nginx level, and update the documentation in settings.py to mention that limit.

@zulipbot
Copy link
Member

@zulipbot zulipbot commented Nov 13, 2020

Hello @zulip/server-production members, this issue was labeled with the "area: documentation (production)" label, so you may want to check it out!

@alexmv
Copy link
Contributor Author

@alexmv alexmv commented Nov 13, 2020

@Sourabh025
Copy link
Collaborator

@Sourabh025 Sourabh025 commented Nov 13, 2020

@zulipbot claim

@zulipbot
Copy link
Member

@zulipbot zulipbot commented Nov 13, 2020

Welcome to Zulip, @Sourabh025! We just sent you an invite to collaborate on this repository at https://github.com/zulip/zulip/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip!

Here's some tips to get you off to a good start:

As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site.

See you on the other side (that is, the pull request side)!

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

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.