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

increase limits for big uploads #192

Closed
wants to merge 4 commits into from
Closed

increase limits for big uploads #192

wants to merge 4 commits into from

Conversation

dcornejo
Copy link

@dcornejo dcornejo commented Jan 4, 2017

I am using some complicated themes and big content files that exceed the defaults, this change allows for these.

@tianon
Copy link
Member

tianon commented Jan 5, 2017

There's some related discussion over in #10 and #49.

@yordis
Copy link

yordis commented Feb 5, 2018

@dcornejo IMHO I do not think this should belong to the repo for one reason

Why 128mb and no 200mb ?

You see,

based on your opinion it should be 128, based on the default it is something else and based on my opinion should be 200.

There is no final answer to who is right or wrong, because the is no right or wrong here, just personal use cases,

so no way to find an absolute answer to my question.

What people should be doing is using volumes and copy over your configurations.

Example,

volumes:
  - ./uploads.ini:/usr/local/etc/php/conf.d/uploads.ini

or using the command line instead of the docker-compose file.

@dcornejo dcornejo closed this Apr 18, 2018
@dcornejo
Copy link
Author

not useful

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

Successfully merging this pull request may close these issues.

None yet

3 participants