Issue with Environment Variable NGINX_CLIENT_MAX_BODY_SIZE about humhub-docker HOT 7 CLOSED

Missed Spacing in Variable

from humhub-docker.

hi, how did you solve it, do you need a space between number and unit?
100 m indstead of 100m

from humhub-docker.

If you follow the NGINX docs shouldn't need a space.
http://nginx.org/en/docs/http/ngx_http_core_module.html#client_max_body_size

Edit:
Looking more into this here's a post similar.
https://stackoverflow.com/questions/2056124/nginx-client-max-body-size-has-no-effect

from humhub-docker.

@olekristensen have you tried the value in humhub-settings like ArchBlood in issue #92 suggested?
#92 (comment)
It could be possible to set this by manipulating the config file directly, but I do not have any time at the moment. A PR would be highly appreciated :)

from humhub-docker.

I don't know much about NGINX so I can give an example. 🤔

    client_max_body_size size;
    client_max_body_size 100m;

As NGINX_CLIENT_MAX_BODY_SIZE is seen as 1m as this is the default value.

from humhub-docker.

Just tested it. To raise the upload-limit you have to set all 3 env-vars NGINX_CLIENT_MAX_BODY_SIZE, PHP_POST_MAX_SIZE and PHP_UPLOAD_MAX_FILESIZE. Also use M for PHP and m for nginx do avoid problems. Do not seperate the number from the unit (e.g. 100 m use 100m). You than can raise the upload-limit at the administration-page inside Humhub (Administration > Settings > Advanced > Files > max. upload file size).

version: '3.1'
services:
  humhub:
    image: mriedmann/humhub:latest
    ports:
      - "8080:80"
    environment:
      NGINX_CLIENT_MAX_BODY_SIZE: 50m
      PHP_UPLOAD_MAX_FILESIZE: 50M
      PHP_POST_MAX_SIZE: 50M

I can totally see why this is confusing. If I find some time I can introduce a variable that can be used to change all 3 values at once.

from humhub-docker.

@mriedmann Thanks a lot 🙏 - i also just had to lean that it is only set on build and not on run.
Thanks for making a dockerized version, it spins beautifully !

from humhub-docker.