[07:48:59]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T merged [pull request #560](https://github.com/YunoHost-Apps/synapse_ynh/pull/560): Testing
[10:00:16]
<Gildas GH> Hi, I'd like to have a PR merged on Restic to fix bugs, if anyone has the right permissions https://github.com/YunoHost-Apps/restic_ynh/pull/36
[11:48:41]
<lapineige> > I'm trying here fyi : https://github.com/YunoHost-Apps/plakar_ynh/pull/12
Solved ! 🎉
[11:50:41]
<lapineige> Thanks both eric_G @ericg:matrix.org and Mateusz Szymański:
[11:51:23]
<lapineige> I wouldn't be able to polish that package without your help (and/or loosing my menthal health points 😅)
[12:05:44]
<Yunohost Git/Infra notifications> App chitchatter goes down from level 6 to 3 in job [#20687](https://ci-apps.yunohost.org/ci/job/20687)
[14:33:02]
<Cippr> eric_G: Bonjour, j'ai tenté l'upgrade mais ça n'a pas fonctionné (une histoire de domain, path and URL for permission) :
sudo yunohost app upgrade simplex -u https://github.com/YunoHost-Apps/simplex_ynh/tree/rework --force
Info: Now upgrading simplex…
Info: Creating a safety backup prior to the upgrade
Info: Collecting files to be backed up for simplex…
Info: Declaring files to be backed up...
Info: /home/yunohost.app/simplex will not be saved, because 'BACKUP_CORE_ONLY' is set.
Info: Backup script completed for simplex. (YunoHost will then actually copy those files to the archive).
Info: Creating a backup archive from the collected files…
Info: The archive will contain about 238.4MB of data.
Success! Backup created: simplex-pre-upgrade2
Success! Backup deleted: simplex-pre-upgrade1
Info: Updating sources...
Info: Updating system_user...
Info: Updating install_dir...
Warning: Moving /usr/local/bin/simplex to /var/www/simplex... (this may take a while)
Info: Updating data_dir...
Info: Updating permissions...
Warning: You cannot enable 'show_tile' right now, because you must first define an URL for the permission 'simplex.main'
Warning: Failed to update permissions : Unknown domain or path for 'simplex'. You need to specify a domain and a path to be able to specify a URL for permission.
Info: Reverting sources...
Info: Reverting system_user...
Info: Reverting install_dir...
Warning: Moving /var/www/simplex to /usr/local/bin/simplex... (this may take a while)
Info: Reverting data_dir...
Info: Reverting permissions...
Error: Provisioning, deprovisioning, or updating resources for simplex failed: Unknown domain or path for 'simplex'. You need to specify a domain and a path to be able to specify a URL for permission.
Info: The operation 'Upgrade the 'simplex' app' could not be completed. Please share the full log of this operation using the command 'yunohost log share 20250805-142556-app_upgrade-simplex' to get help
[14:39:45]
<Yunohost Git/Infra notifications> [searxng_ynh] ewilly merged [pull request #414](https://github.com/YunoHost-Apps/searxng_ynh/pull/414): Upgrade master from testing
[14:39:48]
<Yunohost Git/Infra notifications> [searxng_ynh] ewilly deleted branch master-promotion
[14:43:57]
<Yunohost Git/Infra notifications> [apps] DeMiro5001 opened [pull request #3073](https://github.com/YunoHost/apps/pull/3073): Add open-webui to catalog
[14:46:27]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #3073](https://github.com/YunoHost/apps/pull/3073): Add open-webui to catalog
[14:46:28]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to main: Add open-webui to catalog (#3073) * Add open-webui to apps list * Add open-webui logo ([5fc289c5](https://github.com/YunoHost/apps/commit/5fc289c5529209b2484c862f69a537026fada7aa))
[14:54:32]
<Cippr> Le log est ici et je vais tenter une vraie installation après avoir désinstallé l'application :
https://paste.yunohost.org/raw/wimokamogi
[15:04:08]
<Cippr> eric_G: l'installation via la commande
sudo yunohost app install https://github.com/YunoHost-Apps/simplex_ynh/tree/rework
fonctionne.
Cependant il m'a fallu utiliser un nouveau sous-domaine car SimpleX n'était plus la seule application sur le domaine racine.
Je vais continuer de tester les jours qui viennent mais les photos peuvent maintenant être envoyées et reçues : merci à toi !
[16:37:07]
<Thomas> Could someone give me the rights for https://github.com/YunoHost-Apps/tandoor_ynh/tree/master? There are some issues/pr I’d like to take over
[16:38:04]
<Aleks (he/him/il/lui)> urrrgh i don't have my TOTP right now T_T
[16:50:57]
<tituspijean[m]> > <@thovi98:matrix.org> Could someone give me the rights for https://github.com/YunoHost-Apps/tandoor_ynh/tree/master? There are some issues/pr I’d like to take over
Done!
[16:51:11]
<Thomas> Thank you!
[17:21:07]
<eric_G> Ah, yes. That's because we're now using a domain name instead of the server's IP address. 😵💫
[17:47:48]
<eric_G> Is there a way to specify a domain during the upgrade process? (From a previous release where no domain was set.)
[17:49:51]
<Aleks (he/him/il/lui)> not interactively but if you can magically know which value then you can use `ynh_setting_default_set` ..
[17:50:38]
<Aleks (he/him/il/lui)> alternatively if i understand the context the app is pretty young so maybe you can also "just" add a PRE_UPGRADE specific to this version to tell people to manually define the domain setting with `yunohost app setting <app> domain -v <the.domain.tld>`
[17:50:56]
<Aleks (he/him/il/lui)> and check at the beginning of the upgrade that the setting is define, otherwise call ynh_die
[17:59:41]
<eric_G> https://github.com/YunoHost-Apps/simplex_ynh/blob/8cd52675b11ae82ca63813a34907e0d827c403df/scripts/upgrade#L22-L24
[18:06:01]
<Aleks (he/him/il/lui)> you probably want `[ -z "${domain:-}" ]` otherwise it'll crash if the var doesnt exist yet which is what you're trying to check (yeah .. #bash ...)
[18:13:04]
<Yunohost Git/Infra notifications> [apps] Thovi98 created new branch open-webui
[18:13:05]
<Yunohost Git/Infra notifications> [apps] Thovi98 pushed 1 commit to open-webui: main branch of open-webui is main ([71d5e544](https://github.com/YunoHost/apps/commit/71d5e5445b6d7029d8794efd6f6999e7cf4fa2dc))
[18:13:07]
<Yunohost Git/Infra notifications> [apps] Thovi98 opened [pull request #3074](https://github.com/YunoHost/apps/pull/3074): main branch of open-webui is main
[18:14:35]
<Yunohost Git/Infra notifications> [apps] Thovi98 pushed 1 commit to main: main branch of open-webui is main (#3074) ([4dc4a34e](https://github.com/YunoHost/apps/commit/4dc4a34e1a2a0216f1b9aef0953d29164879a6a1))
[18:14:37]
<Yunohost Git/Infra notifications> [apps] Thovi98 merged [pull request #3074](https://github.com/YunoHost/apps/pull/3074): main branch of open-webui is main
[18:14:37]
<Yunohost Git/Infra notifications> [apps] Thovi98 deleted branch open-webui
[19:19:12]
<Cippr> eric_G: La parie appel de SimpleX en émission et en réception fonctionne, que ce soit avec ou sans vidéo. C'est vraiment cool ce que tu as pu faire !
[20:32:22]
<Yunohost Git/Infra notifications> ericgaspar created repository filegator_ynh https://github.com/YunoHost-Apps/filegator_ynh
[20:34:57]
<Yunohost Git/Infra notifications> ericgaspar edited repository filegator_ynh: Filgator package for YunoHost https://github.com/YunoHost-Apps/filegator_ynh
[20:34:58]
<Yunohost Git/Infra notifications> ericgaspar edited repository filegator_ynh: Filgator package for YunoHost https://github.com/YunoHost-Apps/filegator_ynh
[20:58:37]
<Yunohost Git/Infra notifications> [apps] ericgaspar created new branch filegator
[20:58:37]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to filegator: Update apps.toml ([bda5c612](https://github.com/YunoHost/apps/commit/bda5c61272f0f966e7181644e71bba75e4381c5b))
[21:01:40]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to filegator: Create filegator.png ([0f13416e](https://github.com/YunoHost/apps/commit/0f13416e76c9f646d05c379baf244acd2eff3d10))
[21:01:46]
<Yunohost Git/Infra notifications> [apps] ericgaspar opened [pull request #3075](https://github.com/YunoHost/apps/pull/3075): Filegator
[21:02:27]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #3075](https://github.com/YunoHost/apps/pull/3075): Filegator
[21:02:29]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to main: Filegator (#3075) * Update apps.toml * Create filegator.png ([d5582cfb](https://github.com/YunoHost/apps/commit/d5582cfb99c65d093d166cfff1182ef26e3e1bd4))
[21:13:29]
<orhtej2> Could we flip [auth_header](https://github.com/YunoHost/yunohost/blob/3189c64ad8b7750ac3c578ee1bbefc6d6991abd3/src/utils/resources.py#L584) to false? I understand that would break the 10 or so apps that actually benefit from it but....
[21:14:40]
<Yunohost Git/Infra notifications> ericgaspar edited repository filegator_ynh: Filegator package for YunoHost https://github.com/YunoHost-Apps/filegator_ynh
[21:14:40]
<Yunohost Git/Infra notifications> ericgaspar edited repository filegator_ynh: Filegator package for YunoHost https://github.com/YunoHost-Apps/filegator_ynh
[21:15:04]
<Yunohost Git/Infra notifications> ericgaspar edited repository filegator_ynh: Filegator package for YunoHost https://github.com/YunoHost-Apps/filegator_ynh
[21:15:45]
<orhtej2> (based on super scientific [sso = true](https://github.com/search?q=org%3AYunoHost-Apps+%22sso+%3D+true%22&type=code) vs [auth_header](https://github.com/search?q=org%3AYunoHost-Apps%20auth_header&type=code) search
[21:20:36]
<Yunohost Git/Infra notifications> App filegator failed all tests in job [#20713](https://ci-apps.yunohost.org/ci/job/20713) :(
[21:25:27]
<Aleks (he/him/il/lui)> hmmmmmmmm
[21:25:42]
<Aleks (he/him/il/lui)> i never remember what it does exactly it's totally not straightforward
[21:26:57]
<Aleks (he/him/il/lui)> are you meaning that the majority of sso-compatible apps have to set it to false ?
[21:27:56]
<Aleks (he/him/il/lui)> 🤔
[21:28:37]
<orhtej2> what it does is between nginx and proxied app it (re)sets the header `Authorization` to `user:pass_plaintext` IIRC
[21:28:50]
<orhtej2> which breaks the apps that set said header themselves
[21:29:08]
<Aleks (he/him/il/lui)> yeah that's like the most common way to support the SSO (i mean, until we integrate ... anything else such as OIDC etc) ?
[21:29:21]
<Aleks (he/him/il/lui)> but like, setting it to false should be the exception ?
[21:29:22]
<orhtej2> or, in my iritating case, breaks HTTP Basic auth if you go to `/` but not to `/file.html` 🤷
[21:29:58]
<orhtej2> most of the apps in catalog don't use SSO
[21:30:10]
<Aleks (he/him/il/lui)> wut 😬
[21:31:01]
<orhtej2> seriously I doubt they do, they either have their own auth unrelated to YNH or don't use authorization at all I guess?
[21:31:25]
<Aleks (he/him/il/lui)> 😵💫
[21:31:30]
<orhtej2> what I'm proposing is a breaking change of course, more systematic approach is needed
[21:35:10]
<eric_G> Nextcloud do 😶🌫️
[21:35:45]
<Aleks (he/him/il/lui)> wtf
[21:35:45]
<Aleks (he/him/il/lui)> ```
# grep -nr "auth_header.*false" */manifest.toml --color | wc -l
131
```
[21:36:56]
<Aleks (he/him/il/lui)> well the ones on "api" route make sense usually
[21:37:52]
<Aleks (he/him/il/lui)> still 74 for the non-api ones ... and 46 set it on `main` meh
[21:38:57]
<eric_G> is `main` the new `master`?
[21:39:31]
<Aleks (he/him/il/lui)> urhg yes in terms of git default branch but here i'm talking about the `main` permission 😅
[21:40:46]
<eric_G> chapeau de paille...
[21:43:14]
<orhtej2> a) I find this surprising, does it not issue secure `Bearer` token after login?
b) it can opt in, we shouldn't force apps to opt out (IMO)
[21:45:55]
<orhtej2> it is, I like the recently merged PR that reverses the main vs master default TBH ([here](https://github.com/YunoHost/apps/pull/3055))
[21:46:35]
<orhtej2> waait there was a paired PR in apps_tools, wasn't there? why one has to define `main` as... main branch these days?
[21:46:52]
<Aleks (he/him/il/lui)> the vast majority of apps that set auth_header = false on the `main` perm are apps with `sso = false` (which is not useless because it means the app doesn't receive the credentials since it doesn't need to ... which should in fact be the default and only happen when sso = true witou having to specify auth_header = false)
[21:47:55]
<orhtej2> again, a lot of apps use auth mechanism super incompatible with YNH, there's a ticket do support industry-standard OIDC somewhere, the current solution doesn't cut it IMO
[21:49:14]
<orhtej2> to your point, [piped](https://github.com/YunoHost-Apps/piped_ynh/blob/05d0b68bb36ef825b8ef3597b65ae1d02673b5a5/manifest.toml#L28) declares `sso = false` yet still I had to explicitly opt out of `auth_header`
[21:49:57]
<Aleks (he/him/il/lui)> the real question is "is the majority of apps integrated with YunoHost SSO using the basic HTTP auth headers (auth_header = true, the default)" and i would since yes, because i don't quite understand how you can be auto-logged on the app other than via the basic HTTP auth headers since it's the only supported method of integrating with yunohost sso x_x
[21:52:00]
<orhtej2> (my point being `ldap` is the typical way of interfacing with YNH's auth system)
[21:52:40]
<orhtej2> again, not true penpot uses YNH's `ldap` yet opts out of [ldap integration somehow](https://github.com/YunoHost-Apps/penpot_ynh/blob/be7ef1e7b19c51f130696623864a513af41dac62/manifest.toml#L27)) what is happening I don't anymore
[21:53:33]
<orhtej2> ah I get it, SSO means you logged into portal=you're logged into the app, fair
[21:54:53]
<Aleks (he/him/il/lui)> yeah ... ldap is another thing ... but what makes everything a mess is that the `ldap` and `sso` info in the manifest is purely ... "declarative from the packager" and doesn't impact any code logic anywhere ... (though in the mid/long term it should ...)
[21:58:47]
<Aleks (he/him/il/lui)> in an ideal world, we would have
- sso = true : means ... obviously that SSO works ... and it should be backed by a corresponding `curl_test` [like there's for Nextcloud](https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/tests.toml#L22)
- sso = "not_relevant" should mean that the app has no account mechanism whatsoever so SSO doesn't apply, and it should imply that no credentials is passed to the app from yunohost SSO (equivalent to auth_header=false)
- sso = false should in fact correspond to some warning somewhere to inform people that the packaging is "not ideal", and either the SSO should be integrated somehow in the future, or there's a fundamental issue prenventing sso from being integrated
[22:01:58]
<Aleks (he/him/il/lui)> (historically the `sso` and `ldap` keys were introduced in packaging v2 as a formal, declarative way for packagers to express that the app is supposed to support `sso` / `ldap` instead of manually writing a note in the app's README.md or DISCLAIMER.md)
[22:16:50]
<Yunohost Git/Infra notifications> [mastodon_ynh] yunohost-bot opened [pull request #507](https://github.com/YunoHost-Apps/mastodon_ynh/pull/507): Upgrade to v4.4.3
[22:30:51]
<Yunohost Git/Infra notifications> App plakar rises from level 0 to 7 in job [#20714](https://ci-apps.yunohost.org/ci/job/20714) !
[22:31:35]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 45 pending update PRs
- 18 new apps PRs
- 5 failed apps updates: litechat, openemr, passed, plakar, prestashop
See the full log here: https://paste.yunohost.org/raw/ugubokobuk
Autoupdate dashboard: https://apps.yunohost.org/dash?filter=autoupdate
[22:32:51]
<orhtej2> > <@yunohostinfra:matrix.org> Autoupdater just ran, here are the results:
>
> - 45 pending update PRs
> - 18 new apps PRs
> - 5 failed apps updates: litechat, openemr, passed, plakar, prestashop
>
> See the full log here: https://paste.yunohost.org/raw/ugubokobuk
> Autoupdate dashboard: https://apps.yunohost.org/dash?filter=autoupdate
Stfu dokos
[22:33:45]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/Dokos\_ynh/pulls 😬
[22:34:02]
<orhtej2> Interesting thing is Luanti should have updated VoxelLibre as well but it didn't 🤔
[22:37:40]
<orhtej2> > <@Alekswag:matrix.org> https://github.com/YunoHost-Apps/Dokos_ynh/pulls?q=sort%3Aupdated-desc+is%3Apr+is%3Aopen 😬
Perks of not having `main` source 😵💫
[22:40:56]
<Aleks (he/him/il/lui)> sounds like erpnext should be the main source ?
[22:51:19]
<orhtej2> > <@Alekswag:matrix.org> sounds like erpnext should be the main source ?
Probably
[23:10:30]
<Yunohost Git/Infra notifications> App airsonic failed all tests in job [#20715](https://ci-apps.yunohost.org/ci/job/20715) :(
[23:59:24]
<Yunohost Git/Infra notifications> App nomad stays at level 1 in job [#20720](https://ci-apps.yunohost.org/ci/job/20720)