Thursday, November 30, 2023
apps@conference.yunohost.org
November
Mon Tue Wed Thu Fri Sat Sun
    1
2
3
4
5
6 7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
     
             

[00:08:16] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur edited [pull request #541](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/541): augment "opcache.revalidate_freq" from 1 to 60
[00:09:18] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur merged [pull request #541](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/541): augment "opcache.revalidate_freq" from 1 to 60
[00:09:19] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to testing: augment "opcache.revalidate_freq" from 1 to 60 (#541) * [enh] revalidate_freq to 60 --------- Co-authored-by: Éric Ga... ([beb3eacc](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/beb3eacc9a64be29ef446469f2ffe05b8b4f06ad))
[00:10:09] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur edited [pull request #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622): Testing / switch to packagingv2 + proper mail config
[02:17:01] <Aleks (he/him/il/lui)> https://old.reddit.com/r/selfhosted/comments/186g3ak/owncloud_vulnerability_with_maximum_10_severity/
[02:17:02] <Aleks (he/him/il/lui)> ugh
[02:17:08] <Aleks (he/him/il/lui)> that sound uuuh not good
[04:29:21] <thatoo> > Donc une app basée sur NodeJS ? Le reste est assez classique. Pour redis, tu peux regarder ce que fait l'app Wallabag par exemple

Je regarderais wallabag pour redis. Et pour NodeJS, un bon exemple d'app bien packagé serait?
[05:43:10] <thatoo> Si je comprends bien la façon d'installer Dokos, au moment de lancer la commande `` python3 -m pip install --upgrade --quiet dokos-cli ``, j'execute ce script https://gitlab.com/dokos/docli/-/blob/master/easy-install.py
[05:47:47] <thatoo> lequel script utilise docker compose donc c'est ce script qu'il faudrait "yunohostisé" en install.sh? 
[06:49:53] <lapineige> Ah pas simple si c'est du docker :(
[07:48:22] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @tituspijean merged [pull request #105](https://github.com/YunoHost-Apps/dokuwiki_ynh/pull/105): Require php8.1-mbstring
[07:48:22] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @tituspijean pushed 2 commits to testing ([2411007da35c...f99d9599614e](https://github.com/YunoHost-Apps/dokuwiki_ynh/compare/2411007da35c...f99d9599614e))
[07:48:28] <Yunohost Git/Infra notifications> [dokuwiki_ynh/testing] Require php8.1-mbstring - tituspijean
[07:48:28] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @tituspijean deleted branch tituspijean-patch-1
[07:48:28] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @yunohost-bot pushed 1 commit to testing: Auto-update README ([d5d473e6](https://github.com/YunoHost-Apps/dokuwiki_ynh/commit/d5d473e6afaa6abcd92d4d79598718d9c097ab3c))
[07:49:01] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @tituspijean opened [pull request #106](https://github.com/YunoHost-Apps/dokuwiki_ynh/pull/106): Require PHP mbstring extension
[07:49:27] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @tituspijean pushed 1 commit to testing: Update manifest.toml ([d2c7ed11](https://github.com/YunoHost-Apps/dokuwiki_ynh/commit/d2c7ed11d7f45752bee7b574bc87cb2c1a9ebc68))
[07:49:34] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @yunohost-bot pushed 1 commit to testing: Auto-update README ([f3efd1f6](https://github.com/YunoHost-Apps/dokuwiki_ynh/commit/f3efd1f641469829b21a494939f98e357b75028f))
[07:50:33] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @tituspijean edited [pull request #106](https://github.com/YunoHost-Apps/dokuwiki_ynh/pull/106): Require PHP mbstring extension
[08:32:54] <eric_G> would `POST_REMOVE.md` file in app package usefull to have?
[08:35:12] <eric_G> (Peertube has a post remove message, but that is the only app I see using this)
[08:39:16] <tituspijean> Why not, I'm thinking about Nextcloud which does not delete its /home/yunohost.app directory to keep user data. Mentioning that to the admins would be nice.
[08:45:26] <Maranda> .
[10:16:14] <Aleks (he/him/il/lui)> if both peertube / nextcloud use case are about (not)purging the data dir, we could also decide that we print this tip about "datadir was not removed, please run "rm -rf /home/yunohost.app/$app" if you want to free the data"
[10:46:02] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to fix-overwrite-cli-url: [fix] overwrite.cli.url should contains path ([8efdae92](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/8efdae92176cc29d27291db752c6c97709b5f86b))
[10:46:48] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur opened [pull request #632](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/632): [fix] overwrite.cli.url should contains path
[10:46:55] <Yunohost Git/Infra notifications> WARNING: unknown pull_request action: converted_to_draft
[10:48:25] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to fix-overwrite-cli-url: [fix] Upgrade the bad overwrite.cli.url with path ([ac91d3a6](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/ac91d3a6769747fd3819484ae8c02446e5cc0a30))
[10:48:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur edited [pull request #632](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/632): [fix] overwrite.cli.url should contains path
[10:48:40] <Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur just made [pull request #632](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/632) ready for review: [fix] overwrite.cli.url should contains path
[10:50:20] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/632#issuecomment-1833519919) on [issue #632](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/632) [fix] overwrite.cli.url should contains path: testme
[10:50:23] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/632#issuecomment-1833519988) on [issue #632](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/632) [fix] overwrite.cli.url should contains path: :v:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/11424/badge)](https:/...
[11:51:25] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar approved [pull request #632](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/632#pullrequestreview-1757274059) [fix] overwrite.cli.url should contains path
[11:51:31] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar merged [pull request #632](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/632): [fix] overwrite.cli.url should contains path
[11:51:31] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 3 commits to testing ([beb3eacc9a64...0e79a6449d0d](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/beb3eacc9a64...0e79a6449d0d))
[11:51:33] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar deleted branch fix-overwrite-cli-url
[11:51:34] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] [fix] overwrite.cli.url should contains path - ljf (zamentur)
[11:51:39] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] [fix] Upgrade the bad overwrite.cli.url with path - ljf (zamentur)
[11:53:05] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622): Testing / switch to packagingv2 + proper mail config
[11:54:47] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2_PostgreSQL: fix ([79e4cdf4](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/79e4cdf4012d663f77e0d99bd58755600f61ef52))
[11:55:28] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 11 commits to v2_PostgreSQL ([79e4cdf4012d...b41bfba45488](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/79e4cdf4012d...b41bfba45488))
[11:55:32] <Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] [fix] Upgrade the bad overwrite.cli.url with path - ljf (zamentur)
[11:55:36] <Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge pull request #632 from YunoHost-Apps/fix-overwrite-cli-url [fix] overwrite.cli.url should contains path - eric_G
[12:01:19] <eric_G> How is permissions handled in `change_url` script. we have this issue for Nextcloud: https://github.com/YunoHost-Apps/nextcloud_ynh/issues/626
[12:02:06] <eric_G> we need to regenerat the API permission with new_domain https://github.com/YunoHost-Apps/nextcloud_ynh/blob/0e79a6449d0ddf80f464b03b566ed97db6000556/manifest.toml#L69
[12:02:45] <Aleks (he/him/il/lui)> ogod
[12:03:15] <Aleks (he/him/il/lui)> it's cool if you can create a ticket for this otherwise i'll totally forget about it
[12:03:27] <Aleks (he/him/il/lui)> i mean an issue on yunohost's issue tracker
[12:04:39] <Aleks (he/him/il/lui)> in the meantime maybe you can call `ynh_permission_update` manually in the change_url script, though maybe the linter won't like it
[12:06:28] <eric_G> also, mentioned already but I forgot the meaning of this warning:
```
- Error validating manifest using schema: in key resources > permissions > api > url
're:__DOMAIN__\\/.well-known\\/.*' does not match '^(__[A-Z_]*__)?/.*$'
```

[12:08:21] <Aleks (he/him/il/lui)> hmyeah that's a special case that should be integrated in Salam andar's schema but hmpf
[12:08:58] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar created new branch API-permission
[12:09:14] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to API-permission: Update change_url ([a8f0324b](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/a8f0324b87c67f2112b51690509b966a6951b1aa))
[12:09:46] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to API-permission: Update change_url ([f53eede4](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/f53eede403e123f9daa5af4d4ba3ba814558dbb0))
[12:13:02] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622#issuecomment-1833651178) on [issue #622](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/622) Testing / switch to packagingv2 + proper mail config: @zamentur following #632, we could also update https://github.com/YunoHost-Apps/nextcloud_ynh/blob/0e79a6449d0ddf80f464b...
[12:13:22] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar opened [pull request #633](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/633): Api permission
[12:13:31] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/633#issuecomment-1833651845) on [issue #633](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/633) Api permission: testme
[12:13:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/633#issuecomment-1833651939) on [issue #633](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/633) Api permission: Fingers crossed
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/11425/ba...
[12:14:11] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #633](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/633): Api permission
[12:16:21] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to API-permission: Update change_url ([64b735ff](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/64b735ffd5636c448b3a8352a81200d4fe6cc6bd))
[12:16:59] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to API-permission: Update change_url ([8181c337](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/8181c3376ca56147f4257bedce81d9503257e049))
[13:38:37] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @Tagadda approved [pull request #106](https://github.com/YunoHost-Apps/dokuwiki_ynh/pull/106#pullrequestreview-1757490075) Require PHP mbstring extension
[13:38:55] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @Tagadda merged [pull request #106](https://github.com/YunoHost-Apps/dokuwiki_ynh/pull/106): Require PHP mbstring extension
[13:38:55] <Yunohost Git/Infra notifications> [dokuwiki_ynh] @Tagadda pushed 6 commits to master ([9962a1603f39...fcc3a6e0171d](https://github.com/YunoHost-Apps/dokuwiki_ynh/compare/9962a1603f39...fcc3a6e0171d))
[13:39:00] <Yunohost Git/Infra notifications> [dokuwiki_ynh/master] Update manifest.toml - tituspijean
[14:17:57] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-claper: Add Claper to wishlist ([55d3424b](https://github.com/YunoHost/apps/commit/55d3424b5d01974921678b9bc07573ff17a52056))
[14:17:58] <Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1889](https://github.com/YunoHost/apps/pull/1889): Add Claper to wishlist
[14:18:15] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch add-to-wishlist-claper
[14:24:54] <Yunohost Git/Infra notifications> [apps] @AlexioTROM approved [pull request #1889](https://github.com/YunoHost/apps/pull/1889#pullrequestreview-1757587690) Add Claper to wishlist
[14:27:06] <Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1889#issuecomment-1833881510) on [issue #1889](https://github.com/YunoHost/apps/pull/1889) Add Claper to wishlist: @AlexioTROM eh eh, you are not supposed to approve your own PR 😅
[14:30:47] <Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1889#issuecomment-1833887856) on [issue #1889](https://github.com/YunoHost/apps/pull/1889) Add Claper to wishlist: [](https://i.imgflip.com/87rhrh.jpg)
[14:52:06] <eric_G> oh no. https://ci-apps-dev.yunohost.org/ci/job/11425
[14:53:37] <eric_G> is `--clear_urls` allowed for `ynh_permission_create`?
[14:54:12] <eric_G> ah, no
[15:00:27] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to API-permission: Update change_url ([d5f77985](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/d5f77985e4c56614252b5229f9a0b96be22b6cb5))
[15:00:27] <eric_G> Ok, I have to use `ynh_permission_url`.
[19:24:56] <Raphael> hello, I did not find thir information, I made a hook modifying postfix:
At which time does the install process copy the hook on the server, asking because modifying settings with regen-conf works after installation, but not during install script process (in my case)
Can I specifically call my hook without calling for a full regen-conf?
[19:24:57] <Yunohost Git/Infra notifications> @narFnarF forked nextcloud_ynh to [narFnarF/nextcloud_ynh](https://github.com/narFnarF/nextcloud_ynh)
[19:28:46] <Yunohost Git/Infra notifications> [nextcloud_ynh] @narFnarF opened [pull request #634](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/634): Update ADMIN_fr.md
[19:29:11] <Yunohost Git/Infra notifications> [nextcloud_ynh] @narFnarF opened [pull request #635](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/635): Patch 1
[19:30:59] <Yunohost Git/Infra notifications> [nextcloud_ynh] @narFnarF edited [pull request #635](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/635): Edit ADMIN.md
[19:31:49] <Yunohost Git/Infra notifications> [nextcloud_ynh] @narFnarF edited [pull request #635](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/635): Edit ADMIN.md
[19:32:22] <Yunohost Git/Infra notifications> [nextcloud_ynh] @narFnarF edited [pull request #634](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/634): Update ADMIN_fr.md
[19:33:49] <Yunohost Git/Infra notifications> [nextcloud_ynh] @narFnarF [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/624#issuecomment-1834425890) on [issue #624](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/624) YNH Documentation about OnlyOffice could be reordered for clarity: Im not super familiar with Githubs pull-requests. I hope I did it properly?
[19:42:43] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/624#issuecomment-1834437575) on [issue #624](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/624) YNH Documentation about OnlyOffice could be reordered for clarity: #634 sounds good, Im not sure about #635 (it not targeting testing, hence there is a lot of patches from the testing br...
[19:47:33] <Yunohost Git/Infra notifications> [nextcloud_ynh] @narFnarF edited [pull request #635](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/635): Edit ADMIN.md
[19:47:53] <Yunohost Git/Infra notifications> [nextcloud_ynh] @narFnarF [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/624#issuecomment-1834444224) on [issue #624](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/624) YNH Documentation about OnlyOffice could be reordered for clarity: I think I fixed it?
[19:58:25] <Yunohost Git/Infra notifications> [nextcloud_ynh] @lapineige [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/624#issuecomment-1834457275) on [issue #624](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/624) YNH Documentation about OnlyOffice could be reordered for clarity: Perfect :)