Friday, December 01, 2023
apps@conference.yunohost.org
December
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
31
             

[03:43:59] <Yunohost Git/Infra notifications> App gitlab goes down from level 8 to 6 in job [#21002](https://ci-apps.yunohost.org/ci/job/21002)
[05:38:57] <Yunohost Git/Infra notifications> App samba goes down from level 8 to 6 in job [#21007](https://ci-apps.yunohost.org/ci/job/21007)
[06:39:25] <Yunohost Git/Infra notifications> App ssh_chroot_dir goes down from level 8 to 6 in job [#21010](https://ci-apps.yunohost.org/ci/job/21010)
[07:45:32] <Thomas> Latest Jellyfin version fixes two serious security vulnerabilities : https://jellyfin.org/posts/jellyfin-security-and-you
[07:48:17] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar merged [pull request #635](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/635): Edit ADMIN.md
[07:48:17] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar approved [pull request #635](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/635#pullrequestreview-1759205398) Edit ADMIN.md
[07:48:19] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 2 commits to testing ([0e79a6449d0d...5ab5ce3b95d2](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/0e79a6449d0d...5ab5ce3b95d2))
[07:48:27] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Update ADMIN.md - narF
[07:48:36] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar approved [pull request #634](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/634#pullrequestreview-1759205881) Update ADMIN_fr.md
[07:48:42] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 2 commits to testing ([5ab5ce3b95d2...90fa4e7cc1b7](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/5ab5ce3b95d2...90fa4e7cc1b7))
[07:48:44] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar merged [pull request #634](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/634): Update ADMIN_fr.md
[07:48:46] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Update ADMIN_fr.md - narF
[07:48:49] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Merge pull request #634 from narFnarF/patch-2 Update ADMIN_fr.md - eric_G
[08:07:50] <Yunohost Git/Infra notifications> [apps] @tituspijean approved [pull request #1889](https://github.com/YunoHost/apps/pull/1889#pullrequestreview-1759234005) Add Claper to wishlist
[08:08:03] <Yunohost Git/Infra notifications> [apps] @tituspijean merged [pull request #1889](https://github.com/YunoHost/apps/pull/1889): Add Claper to wishlist
[08:08:04] <Yunohost Git/Infra notifications> [apps] @tituspijean deleted branch add-to-wishlist-claper
[08:08:06] <Yunohost Git/Infra notifications> [apps] @tituspijean pushed 2 commits to master ([98aa48c48149...671e361ee9b1](https://github.com/YunoHost/apps/compare/98aa48c48149...671e361ee9b1))
[10:07:08] <thatoo> > Ah pas simple si c'est du docker :(

J'ai trouvé ce script d'install : https://github.com/flexcomng/erpnext_quick_install/blob/main/erpnext_install.sh
A savoir que DOkos est la francisation d'ERPnext
[10:07:21] <thatoo> Ca me parait une base plus saine pour travailler, non?
[10:10:49] <thatoo> est-ce un soucis si db_user=root ?
```
#=================================================
ynh_script_progression --message="Creating a MySQL database..."

db_name=$(ynh_sanitize_dbid --db_name=$app)
db_user=root
ynh_app_setting_set --app=$app --key=db_name --value=$db_name
ynh_mysql_setup_db --db_user=$db_user --db_name=$db_name
```

Je ne trouve pas d'option pour db_user dans le fichier json de conf de l'app Dokos et dans les scripts d'install, c'est toutjours root.
[11:58:39] <Yunohost Git/Infra notifications> @leonarf forked apps to [leonarf/apps](https://github.com/leonarf/apps)
[13:03:00] <Yunohost Git/Infra notifications> [apps] @leonarf opened [pull request #1890](https://github.com/YunoHost/apps/pull/1890): add application my_fastapi_app to the catalog
[13:43:26] <tituspijean> @leolecolo:matrix.org welcome! to properly assess if the app is working, we should run an automated test. Can you find a small alteration to make with a PR and add a comment with `!testme` to run the CI?
[13:48:44] <Yunohost Git/Infra notifications> [apps] @leonarf [commented](https://github.com/YunoHost/apps/pull/1890#issuecomment-1836148867) on [issue #1890](https://github.com/YunoHost/apps/pull/1890) add application my_fastapi_app to the catalog: testme
[13:56:59] <Leonard> Aleks (he/him/il/lui): to continue our discussion on the correct channel : I was planning to do as my_webapp does : create a sftp access so the user can upload it's own code. Whereas having the possibility to give an github url at during the installation
[13:58:20] <Mateusz Szymański> > <@yunohostinfra:matrix.org> [apps] @leonarf [commented](https://github.com/YunoHost/apps/pull/1890#issuecomment-1836148867) on [issue #1890](https://github.com/YunoHost/apps/pull/1890) add application my_fastapi_app to the catalog: testme

that `!testme` was supposed to go in new PR created in your app's repo ;)
[13:59:47] <Mateusz Szymański> intent being like so: https://github.com/YunoHost-Apps/gitea_ynh/pull/84#issuecomment-1835838328
[14:01:12] <Yunohost Git/Infra notifications> [apps] @alexAubin [commented](https://github.com/YunoHost/apps/pull/1890#discussion_r1412138164) on pull request #1890 add application my_fastapi_app to the catalog: Beware if your app id is my_fastapi_app, your repo should be named my_fastapi_app_ynh
[14:02:47] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-octoprint: Add octoprint to wishlist ([acd49aad](https://github.com/YunoHost/apps/commit/acd49aad7c808cf7b8384cbf25c5cd5bda53c746))
[14:02:47] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch add-to-wishlist-octoprint
[14:02:51] <Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1891](https://github.com/YunoHost/apps/pull/1891): Add octoprint to wishlist
[14:16:36] <Mateusz Szymański> > <@titus:pijean.ovh> @leolecolo:matrix.org welcome! to properly assess if the app is working, we should run an automated test. Can you find a small alteration to make with a PR and add a comment with `!testme` to run the CI?

aside from usual `pip` complaints about cache folder (wtf is up with that, saw that all over catalog) seems to be working just fine https://ci-apps-dev.yunohost.org/ci/job/11453
[14:18:44] <Aleks (he/him/il/lui)> maybe gotta make sure that `pip` command are consistently called as the app user (`ynh_exec_as $app` etc)
[14:20:48] <Mateusz Szymański> Wasn't touching `manifest.toml` enough to trigger README bot?
[14:22:07] <Mateusz Szymański> > <@orhtej2:matrix.org> aside from usual `pip` complaints about cache folder (wtf is up with that, saw that all over catalog) seems to be working just fine https://ci-apps-dev.yunohost.org/ci/job/11453

nope, backup is dead Leonard
[14:24:04] <Leonard> I've only done install and remove. I think I need some explanation about backup/restore and uprade
[14:30:52] <Mateusz Szymański> > <@leolecolo:matrix.org> I've only done install and remove. I think I need some explanation about backup/restore and uprade

you can back up the app from web admin and then attempt to restore it after uninstall. you can also `sudo yunohost app upgrade <appid> -u <some_url> --force` to see if it updates properly, currently from the same version given there's only one available
[14:31:32] <Mateusz Szymański> can someone create `testing` branch [here](https://github.com/YunoHost-Apps/FastAPI_ynh) and mark both `master` and `testing` protected, requiring PR?
[15:23:45] <lapineige> > <@thatoo:defis.info> est-ce un soucis si db_user=root ?
> ```
> #=================================================
> ynh_script_progression --message="Creating a MySQL database..."
>
> db_name=$(ynh_sanitize_dbid --db_name=$app)
> db_user=root
> ynh_app_setting_set --app=$app --key=db_name --value=$db_name
> ynh_mysql_setup_db --db_user=$db_user --db_name=$db_name
> ```
>
> Je ne trouve pas d'option pour db_user dans le fichier json de conf de l'app Dokos et dans les scripts d'install, c'est toutjours root.

je pense que ça va poser problème oui, niveau sécurité surtout
[15:31:16] <Aleks (he/him/il/lui)> >db_user=root

keuwa 🙀
[15:31:45] <Aleks (he/him/il/lui)> nopenopenope.gif
[16:35:59] <eric_G> Leonard: why is the app installed in `$data_dir` directory and not `$install_dir`?
[17:00:44] <Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1892](https://github.com/YunoHost/apps/pull/1892): Update app levels according to CI results
[17:00:44] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch update_app_levels
[17:00:44] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to update_app_levels: Update app levels according to CI results ([2e3351eb](https://github.com/YunoHost/apps/commit/2e3351eb458df9952ed87ebe6133b9c1ab62f607))
[17:11:35] <Leonard> > <@ericg:matrix.org> Leonard: why is the app installed in `$data_dir` directory and not `$install_dir`?

Hello, I started the app from the django_example app. But I have prepared a commit to install it in install_dir, and put only custom app into data_dir.
[17:12:12] <Leonard> And I want to create a SFTP access to the data_dir so users quand easily add their custom fastapi code
[17:24:04] <Yunohost Git/Infra notifications> App ethercalc failed all tests in job [#20935](https://ci-apps.yunohost.org/ci/job/20935) :(
[17:27:03] <eric_G> https://ci-apps.yunohost.org/ci/job/20935 😑
[17:28:13] <eric_G> corrupt npm package? 😷
[18:26:08] <Yunohost Git/Infra notifications> App sonarr failed all tests in job [#20970](https://ci-apps.yunohost.org/ci/job/20970) :(
[19:41:13] <Charles P.> Well
[19:41:15] <Charles P.> Yeah
[19:41:18] <Charles P.> Failed all tests
[19:41:22] <Charles P.> https://aria.im/_matrix/media/v1/download/stratus.family/ZPyILKNYFonQFiFhbyYgIaoK
[19:45:03] <Charles P.> Linter failed, thus making app level 0, but it isn't indicated in summary
[19:46:01] <Aleks (he/him/il/lui)> ah yes meh critical issues are not reported for some reason, only "errors"
[20:08:19] <Leonard> > <@leolecolo:matrix.org> And I want to create a SFTP access to the data_dir so users quand easily add their custom fastapi code

Hello, I try to change the user create for my app to make its home folder = data_dir and enabling sftp connection. I'm using the command 'ynh_system_user_create' but linter tells me it's deprecated. How can I enable sftp and change default home folder?
[20:17:42] <Raphael> > <@ashledombos:matrix.org> 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?

After reading app managing script, I see that hooks are copied after install, so doing regen-conf during install is not the good solution. Meanwhile I’ve seen than mailman package maintainer is using another solution https://github.com/YunoHost-Apps/mailman3_ynh/blob/16a134e53aad2e32c6689e971aaed6e0166d0143/scripts/install#L168
[21:11:17] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-wishthis: Add wishthis to wishlist ([06116721](https://github.com/YunoHost/apps/commit/061167218ecfd3f20f812466d589b46666dad25f))
[21:11:18] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch add-to-wishlist-wishthis
[21:11:20] <Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1893](https://github.com/YunoHost/apps/pull/1893): Add wishthis to wishlist