change UI upload size limit about funkwhale_ynh HOT 9 CLOSED

from funkwhale_ynh.

Solved, here is the file to edit:

/etc/nginx/conf.d/yourdomain.d/funkwhale.conf

from funkwhale_ynh.

I've tried to nano this file:

funkwhale_ynh/conf/nginx.conf

and to edit this line:

client_max_body_size 30M; to put a higher value,

but it doesn't work.
Help!

from funkwhale_ynh.

please remember your change will be erased at each update of the funkwhale package

@EliotBerriot : do you think using a higher value is safe? does it makes sense to you?

from funkwhale_ynh.

We could probably increase this, the main risk is having user accidentally (or voluntarily in case of an attack) exhausting the server resources by uploading large files. Setting a higher limit (let's say 100MB) will not break anything, it's just a tradeof to consider between security/stability and functionality.

That being said, I'd probably increase the default limit to 50 or 100MB in our sample files because we've got plenty of support tickets and questions about this :)

from funkwhale_ynh.

I created a ticket for that: https://dev.funkwhale.audio/funkwhale/funkwhale/issues/654

from funkwhale_ynh.

OK, I see it is now upstream, I will update it to 100 MB

from funkwhale_ynh.

👌 @Jibec, sorry, I forgot to ping you back once it was done!

from funkwhale_ynh.

fixed by 9c6ee6e

from funkwhale_ynh.