[05:07:18]
<Yunohost Git/Infra notifications> App endi failed all tests in job [#15252](https://ci-apps.yunohost.org/ci/job/15252) :(
[06:30:14]
<Thomas> Default nginx client_max_body_size is 1m
Does increasing this by default in a package is good or does it lead to security issues?
Is it better to let the user configure it in the config_panel?
[07:02:57]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @eldertek approved [pull request #564](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/564#pullrequestreview-1403328503) 26.0.1: Tested on my side Working
[07:03:31]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @eldertek edited review [pull request #564](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/564#pullrequestreview-1403328503): 26.0.1
[07:03:49]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @eldertek edited review [pull request #564](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/564#pullrequestreview-1403328503): 26.0.1
[07:05:33]
<Yunohost Git/Infra notifications> App question2answer stays at level 2 in job [#15161](https://ci-apps.yunohost.org/ci/job/15161)
[07:25:27]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107#issuecomment-1524963659) on [issue #107](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107) V2: testme
[07:25:28]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107#issuecomment-1524963723) on [issue #107](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107) V2: :stuck_out_tongue_winking_eye:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/ap...
[08:44:38]
<Aleks (he/him/il/lui)> > <@thovi98:matrix.org> Default nginx client_max_body_size is 1m
> Does increasing this by default in a package is good or does it lead to security issues?
> Is it better to let the user configure it in the config_panel?
i don't think there's security issue with this except if you start getting to crazy amounts
[08:45:07]
<Aleks (he/him/il/lui)> imho setting a new default value for a given app vs. adding a setting in the config panel is a dependent on what the app does exactly
[08:46:25]
<Aleks (he/him/il/lui)> e.g. if this is some sort of usually-private app where you expect users to very often have to upload 10+ MB files, then it's fine to set it to e.g. 100 MB
[08:47:42]
<Aleks (he/him/il/lui)> if this is a "public"-ish app like Element/Synapse where you want people to be able to upload files, i'd set it to 10 M by default + maybe a setting to change it
[08:48:06]
<Aleks (he/him/il/lui)> similar stuff for apps like Lufi / WeTransfer-like apps
[08:56:18]
<Thomas> Okay thank you for your exhaustive answer!
[08:56:32]
<Thomas> Will change the default for FitTrackee then
[11:33:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @kay0u edited [pull request #564](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/564): 26.0.1
[11:34:57]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @kay0u pushed 40 commits to 26.0.0 ([4ea9edf101d9...2675a992d81e](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/4ea9edf101d9...2675a992d81e))
[11:35:01]
<Yunohost Git/Infra notifications> [nextcloud_ynh/26.0.0] Update nginx.conf - Éric Gaspar
[11:35:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to 26.0.0: Auto-update README ([72adb7d2](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/72adb7d21aaefa10fff7fdecf0843334313a5e70))
[11:35:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh/26.0.0] Merge branch master into testing - Éric Gaspar
[11:35:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh/26.0.0] Merge branch testing into 26.0.0 - Kayou
[15:50:39]
<Yunohost Git/Infra notifications> App borgserver failed all tests in job [#15260](https://ci-apps.yunohost.org/ci/job/15260) :(
[16:05:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @eldertek [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/564#issuecomment-1525961814) on [issue #564](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/564) 26.0.1: testme
[16:14:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @eldertek [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/564#issuecomment-1525961814) on [issue #564](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/564) 26.0.1: testme
[17:23:50]
<Yunohost Git/Infra notifications> App digiscreen failed all tests in job [#15261](https://ci-apps.yunohost.org/ci/job/15261) :(
[17:58:51]
<Yunohost Git/Infra notifications> App django-fritzconnection failed all tests in job [#15262](https://ci-apps.yunohost.org/ci/job/15262) :(
[19:25:25]
<Yunohost Git/Infra notifications> App facilmap goes down from level 8 to 1 in job [#15263](https://ci-apps.yunohost.org/ci/job/15263)