[00:10:07]
<Yunohost Git/Infra notifications> [autoupdate_app_sources] App sources auto-update failed miserably!
[01:18:04]
<Yunohost Git/Infra notifications> [yunohost-admin] dependabot[bot] labeled dependencies on [pull request #590](https://github.com/YunoHost/yunohost-admin/pull/590): Bump vue from 2.7.16 to 3.0.0 in /app
[01:18:04]
<Yunohost Git/Infra notifications> [yunohost-admin] dependabot[bot] pushed 1 commit to dependabot/npm_and_yarn/app/vue-3.0.0: Bump vue from 2.7.16 to 3.0.0 in /app Bumps [vue](https://github.com/vuejs/core) from 2.7.16 to 3.0.0. - [Release notes... ([c6ac0023](https://github.com/YunoHost/yunohost-admin/commit/c6ac00233ae13917152bef25ab2d7a815ea36875))
[01:18:04]
<Yunohost Git/Infra notifications> [yunohost-admin] dependabot[bot] opened [pull request #590](https://github.com/YunoHost/yunohost-admin/pull/590): Bump vue from 2.7.16 to 3.0.0 in /app
[01:19:19]
<kayou> > <@yunohostinfra:matrix.org> [doc] Josue-T opened [pull request #2509](https://github.com/YunoHost/doc/pull/2509): Update sso_ldap_integration.md
This notification should be in the doc chan
[01:19:19]
<Yunohost Git/Infra notifications> [yunohost-admin] dependabot[bot] created new branch dependabot/npm_and_yarn/app/vue-3.0.0
[04:44:07]
<Yunohost Git/Infra notifications> [appstore] OniriCorpe approved [pull request #7](https://github.com/YunoHost/appstore/pull/7#pullrequestreview-2394155514) Format Python code with Black
[04:44:17]
<Yunohost Git/Infra notifications> [appstore] OniriCorpe pushed 1 commit to main: :art: Format Python code with Black ([93bcbaa6](https://github.com/YunoHost/appstore/commit/93bcbaa644730ebaf232fa32797142c0881243d5))
[04:44:18]
<Yunohost Git/Infra notifications> [appstore] OniriCorpe merged [pull request #7](https://github.com/YunoHost/appstore/pull/7): Format Python code with Black
[04:44:19]
<Yunohost Git/Infra notifications> [appstore] OniriCorpe deleted branch actions/black
[08:10:48]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/cpAqkUhPXngdMXWyqjxQfEEY
[08:10:57]
<Salamandar> The sub path test is buggy : it tests the root of the domain
[08:48:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] genma opened [issue #740](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/740): Nextcloud Hub 8 (29.0.8)
[08:53:10]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar closed [issue #740](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/740): Nextcloud Hub 8 (29.0.8)
[08:53:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/740#issuecomment-2437248816) on [issue #740](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/740) Nextcloud Hub 8 (29.0.8): The CI is having trouble validating all tests (probably due to changes in our infrastructure). In the meantime, you can ...
[08:54:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to 30.0.0: Update manifest.toml ([68a855d5](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/68a855d58380fef7ae071a3f39bd58fd177470d7))
[08:54:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh] genma opened [issue #741](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/741): Nextcloud 30.0.1 & MariaDB >=10.6
[09:02:22]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL_helpers_2.1: cleaning ([2b4c8f56](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/2b4c8f567a97839a0b50d491cc29ed84105a8b8b))
[09:02:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL_helpers_2.1: Update install ([14280225](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/14280225638ff21550847b96e8df67d7c1fa3fad))
[09:03:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL: Update install ([12737f21](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/12737f21d6fbf90ec8be88ee4caba550d90d996b))
[09:05:11]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL: Update install ([1b9777c3](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/1b9777c3267d49d5c82fd92de3d3c0890114bf4d))
[09:06:56]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL_helpers_2.1: Update manifest.toml ([3bd936cf](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/3bd936cf60bc5b67ccd37acf0f86b6c3f111464d))
[09:06:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot pushed 1 commit to v2_PostgreSQL_helpers_2.1: Auto-update READMEs ([51fb1c14](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/51fb1c1426077599ad071d711712794ddcfb5539))
[09:09:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL_helpers_2.1] cleaning - Éric Gaspar
[09:09:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 2 commits to v2_PostgreSQL_helpers_2.1 ([51fb1c142607...cb805db7d821](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/51fb1c142607...cb805db7d821))
[09:09:05]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL_helpers_2.1] Merge branch v2_PostgreSQL_helpers_2.1 of https://github.com/YunoHost-Apps/nextcloud_ynh into v2_PostgreSQL_helpers_2.... - Éric Gaspar
[09:18:21]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL_helpers_2.1: cleaning ([aa627be2](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/aa627be2fb98df06a9f88cda3fd9a3a0cbe09c47))
[09:37:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar closed [issue #741](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/741): Nextcloud 30.0.1 & MariaDB >=10.6
[09:37:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/741#issuecomment-2437338755) on [issue #741](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/741) Nextcloud 30.0.1 & MariaDB >=10.6: We are trying to follow Nextcloud requirements for DB and PHP version. and I think version 30.0.x will be pushed to mast...
[09:44:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar created new branch remove-duplicate-JS
[09:44:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to remove-duplicate-JS: remove duplicate extension "js" ([0441e40d](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/0441e40db415b09e26840ce9e617402da8c89ec4))
[09:45:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar opened [pull request #742](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/742): Remove duplicate js in NGINX config
[10:07:33]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL: Update config_panel.toml ([53012e45](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/53012e454d67604c53f64d7c3fb6c5af019358b6))
[10:13:56]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: Add fr translation ([cc04eb60](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/cc04eb60034cbf633190bd746d35a1d284486bac))
[10:14:55]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar edited [pull request #735](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/735): Testing
[10:16:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: remove duplicate extension "js" (#742) ([e8e04de9](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/e8e04de9d777486d88a1620cc230e7cda7d02ec5))
[10:16:11]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar merged [pull request #742](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/742): Remove duplicate js in NGINX config
[10:16:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar deleted branch remove-duplicate-JS
[10:16:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar edited [pull request #735](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/735): Testing
[10:16:38]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar edited [pull request #735](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/735): Testing
[10:19:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/667#issuecomment-2437420113) on [issue #667](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/667) Nginx fails to start at server boot because of the nextcloud nginx config: closing with #735
[10:19:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar closed [issue #667](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/667): Nginx fails to start at server boot because of the nextcloud nginx config
[10:19:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/667#issuecomment-2437420113) on [issue #667](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/667) Nginx fails to start at server boot because of the nextcloud nginx config: closing with #742
[10:22:37]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 8 commits to v2_PostgreSQL ([53012e454d67...d1b1b97a89ae](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/53012e454d67...d1b1b97a89ae))
[10:22:37]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Add fr translation - Éric Gaspar
[10:22:38]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] remove duplicate extension "js" (#742) - eric_G
[10:22:39]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - Éric Gaspar
[11:13:21]
<kayou> from package_check readme:
> if using LXD, run this command to add the Yunohost image repository: `lxc remote add yunohost https://repo.yunohost.org/incus --protocol simplestreams --public;` at the time this README is written, fingerprint is d9ae6e76c374e3c58c3c20a881cffe7435809adb3b222ec393805f5bd01bb522
works on my server, but the remote doesn't contain any image
[11:13:29]
<kayou> > $ lxc image list yunohost:
+-------+-------------+--------+-------------+--------------+------+------+-------------+
| ALIAS | FINGERPRINT | PUBLIC | DESCRIPTION | ARCHITECTURE | TYPE | SIZE | UPLOAD DATE |
+-------+-------------+--------+-------------+--------------+------+------+-------------+
[11:13:54]
<kayou> (i know I should migrate to incus, but i don't want to do it rn)
[11:41:53]
<Salamandar> probably cus it's lxc and not incus
[11:41:59]
<Salamandar> let me c heck
[11:42:16]
<Salamandar> yeah via incus it's ok
[11:45:02]
<Salamandar> https://repo.yunohost.org/incus/streams/v1/images.json
[11:45:02]
<Salamandar> ah yes
[11:45:15]
<Salamandar> incus_combined.tar.g
[11:45:15]
<Salamandar> incus_combined.tar.gz
[11:45:23]
<Salamandar> that's a new feature provided by incus but not lxc
[15:00:37]
<Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch update_app_levels
[15:00:37]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to update_app_levels: Update app levels according to CI results ([39c53b46](https://github.com/YunoHost/apps/commit/39c53b46c6be3fe2f3b70039c6f94a32d1fc2fef))
[15:00:38]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2660](https://github.com/YunoHost/apps/pull/2660): Update app levels according to CI results
[15:00:54]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] created new branch actions/toml
[15:00:55]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/toml: :art: Format TOML with Taplo ([01d1cc2e](https://github.com/YunoHost/apps/commit/01d1cc2ef608c862e5132683ad052dfb9e1fe5c2))
[15:00:57]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] opened [pull request #2661](https://github.com/YunoHost/apps/pull/2661): Format TOML with Taplo
[15:04:08]
<Aleks (he/him/il/lui)> zomg only 2 major regression this week
[15:04:12]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2661](https://github.com/YunoHost/apps/pull/2661#pullrequestreview-2395573233) Format TOML with Taplo
[15:04:18]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2661](https://github.com/YunoHost/apps/pull/2661): Format TOML with Taplo
[15:04:19]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: :art: Format TOML with Taplo (#2661) Co-authored-by: yunohost-bot <14998418+yunohost-bot@users.noreply.github.com> ([bee63373](https://github.com/YunoHost/apps/commit/bee63373d580d1cea4d9cb1b7fac0d739547cd19))
[15:04:19]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch actions/toml
[15:05:17]
<Aleks (he/him/il/lui)> and seven None -> 7 🤯
[15:05:45]
<Aleks (he/him/il/lui)> six*
[15:39:44]
<eric_G> 🤔 should be 8...
[15:41:46]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([d420738a](https://github.com/YunoHost/apps/commit/d420738aba18d8e718ad53d01ef6af1e108b67be))
[15:41:46]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2660](https://github.com/YunoHost/apps/pull/2660): Update app levels according to CI results
[15:42:10]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([01ccec6a](https://github.com/YunoHost/apps/commit/01ccec6a7617a3253ace0c212d6c791575190ba9))
[15:42:14]
<Yunohost Git/Infra notifications> [apps] ericgaspar edited [pull request #2660](https://github.com/YunoHost/apps/pull/2660): Update app levels according to CI results
[16:16:23]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([325655d1](https://github.com/YunoHost/apps/commit/325655d1e365957adf345c026ce02efe437b5914))
[16:17:06]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2660](https://github.com/YunoHost/apps/pull/2660#pullrequestreview-2395784115) Update app levels according to CI results
[16:44:29]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: Add some documentation for curl test properties ([8b765c01](https://github.com/YunoHost/package_check/commit/8b765c01b9b07a142589780a0039682698971d50))
[16:49:17]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2660](https://github.com/YunoHost/apps/pull/2660): Update app levels according to CI results
[16:49:17]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Update app levels according to CI results (#2660) * Update app levels according to CI results * :art: Format TOML with... ([1044daa2](https://github.com/YunoHost/apps/commit/1044daa2489268d8490479f35dd0bf8036efb167))
[16:49:17]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch update_app_levels
[18:48:03]
<Yunohost Git/Infra notifications> [ynh-dev] alexAubin merged [pull request #83](https://github.com/YunoHost/ynh-dev/pull/83): Add information about prerequisites when using use-git
[18:48:04]
<Yunohost Git/Infra notifications> [ynh-dev] alexAubin pushed 2 commits to master ([c13cce70d412...f16dcde24888](https://github.com/YunoHost/ynh-dev/compare/c13cce70d412...f16dcde24888))
[18:48:04]
<Yunohost Git/Infra notifications> [ynh-dev/master] Add information about prerequisites when using use-git - yorffuoj
[18:48:05]
<Yunohost Git/Infra notifications> [ynh-dev/master] Merge pull request #83 from yorffuoj/patch-1 Add information about prerequisites when using use-git - Alexandre Aubin
[18:50:48]
<Yunohost Git/Infra notifications> [yunorunner] alexAubin merged [pull request #69](https://github.com/YunoHost/yunorunner/pull/69): Format Python code with Black
[18:50:49]
<Yunohost Git/Infra notifications> [yunorunner] alexAubin pushed 2 commits to master ([9f5fbba7b879...a1f48c8984ac](https://github.com/YunoHost/yunorunner/compare/9f5fbba7b879...a1f48c8984ac))
[18:50:49]
<Yunohost Git/Infra notifications> [yunorunner/master] :art: Format Python code with Black - Salamandar
[18:50:50]
<Yunohost Git/Infra notifications> [yunorunner/master] Merge pull request #69 from YunoHost/actions/black Format Python code with Black - Alexandre Aubin
[18:50:52]
<Yunohost Git/Infra notifications> [yunorunner] alexAubin deleted branch actions/black
[18:51:21]
<Yunohost Git/Infra notifications> [yunohost-portal] alexAubin merged [pull request #18](https://github.com/YunoHost/yunohost-portal/pull/18): Translations update from Weblate
[18:51:40]
<Yunohost Git/Infra notifications> [yunohost-portal] alexAubin pushed 2 commits to dev ([315c743623f4...0ae5d3065168](https://github.com/YunoHost/yunohost-portal/compare/315c743623f4...0ae5d3065168))
[18:55:31]
<Yunohost Git/Infra notifications> [package_check] alexAubin merged [pull request #179](https://github.com/YunoHost/package_check/pull/179): Fix all shellcheck issues
[18:55:32]
<Yunohost Git/Infra notifications> [package_check] alexAubin pushed 3 commits to master ([8b765c01b9b0...01afe49c87b5](https://github.com/YunoHost/package_check/compare/8b765c01b9b0...01afe49c87b5))
[18:55:33]
<Yunohost Git/Infra notifications> [package_check/master] Fix all shellcheck issues - Félix Piédallu
[18:55:34]
<Yunohost Git/Infra notifications> [package_check/master] shellcheck workflow: external sources - Félix Piédallu
[18:55:34]
<Yunohost Git/Infra notifications> [package_check/master] Merge pull request #179 from YunoHost/shellcheck Fix all shellcheck issues - Alexandre Aubin
[18:57:50]
<Yunohost Git/Infra notifications> [package_check] alexAubin deleted branch shellcheck
[18:59:39]
<Yunohost Git/Infra notifications> [doc] alexAubin merged [pull request #2508](https://github.com/YunoHost/doc/pull/2508): [CI] Update app helpers/resources for debian/11.2.31
[18:59:40]
<Yunohost Git/Infra notifications> [doc] alexAubin pushed 1 commit to master: [CI] Update app helpers/resources for debian/11.2.31 (#2508) ([894bb761](https://github.com/YunoHost/doc/commit/894bb76191b5506393fafd32896633839ae9d7c2))
[18:59:42]
<Yunohost Git/Infra notifications> [doc] alexAubin deleted branch debian/11.2.31
[19:00:14]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost-portal/12.0.4+202410252100 for bookworm/unstable/all ...
[19:05:54]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost-portal/12.0.4+202410252100 for bookworm/unstable/all.
[19:20:43]
<Josué> Hello, Aleks (he/him/il/lui) @Alekswag:matrix.org: do you have some time to review https://github.com/YunoHost/yunohost/pull/1981 ?
[19:22:17]
<Aleks (he/him/il/lui)> not really, i was at $dayjob during 3 days, i need to rest and dig in it hopefully this weekend
[19:28:24]
<Aleks (he/him/il/lui)> to me it feels funky that the headers to inject should be stored in the cookie rather than having general keys in the cookie (e.g. the username is already there), containing the display name and email address ... and then hardcoding in ssowat `YNH_FOO = cookie.foo`
[19:28:44]
<Aleks (he/him/il/lui)> to me it feels funky that the headers to inject should be stored in the cookie rather than having general keys in the cookie (e.g. the username is already there), containing the display name and email address ... and then hardcoding in ssowat `set_header("YNH_FOO") = cookie.foo`
[19:29:03]
<Aleks (he/him/il/lui)> also i'm not sure to understand why an app would need the display name
[19:29:53]
<Aleks (he/him/il/lui)> (and i'm not sure either why it would need the email address ... we discussed a few use case the other day and it seemed like the fact that the app was using the email address as login could be configured in the app conf)
[19:44:59]
<Josué> > <@Alekswag:matrix.org> (and i'm not sure either why it would need the email address ... we discussed a few use case the other day and it seemed like the fact that the app was using the email address as login could be configured in the app conf)
Well, as I said, for some app like seafile, it just not possible to use the usename instead as it's hard coded in the in app that a username is an email. So putting a username which is not an email just don't work at all.
[19:47:06]
<Josué> > <@Alekswag:matrix.org> also i'm not sure to understand why an app would need the display name
The display name is more, in case of the app populate all user account data from the header (in case of the app don't support LDAP). By example, pgadmin manage the account with the Header but is not linked to LDAP
[19:48:21]
<Josué> > <@Alekswag:matrix.org> to me it feels funky that the headers to inject should be stored in the cookie rather than having general keys in the cookie (e.g. the username is already there), containing the display name and email address ... and then hardcoding in ssowat `set_header("YNH_FOO") = cookie.foo`
Yes I can understand but well, the issue is that either we need to do this way or we need to connect ssowat to ldap to retrieve the other info (email, user fullname...)
[19:49:10]
<Josué> > <@Alekswag:matrix.org> not really, i was at $dayjob during 3 days, i need to rest and dig in it hopefully this weekend
yes I understand, no problem 😉
[19:49:59]
<Aleks (he/him/il/lui)> i was meaning just adding directly extra keys in the JWT for each info we want rather than a dict of header (that's sort of a detail, but at the same time cookie length has a finite limit so not a big fan of repeating HTTP header keys if we can avoid them, and also the username is already one of the existing keys so)
[19:51:57]
<Aleks (he/him/il/lui)> are there other apps that do need the user display name ? Imho if we have features we should be able to justify it by at least 3-4 apps that do need it and can't avoid it, otherwise we end up in the same situation of "this feature exist but we don't know which app actually use it or if it is even used at all" (or "it is used by a single app but nobody knows about it" etc)
[19:52:18]
<Aleks (he/him/il/lui)> feature creep / yagni / etc
[19:53:06]
<Josué> ah year, I did this way as it would be easier one day if we want to add an other header, we just need to change at one place. But yes I agree about the cookie size which is not ideal.
[19:53:43]
<Josué> so did you think about just using a list instead of a dict to transmit the header ?
[19:56:33]
<Aleks (he/him/il/lui)> just ... currently "infos" (the actual payload of the cookie) is something like
```
{"id": <sessionid>, "user": "alice", "pwd": "password", "host": "domain.tld"}
```
if we need the email, let's just add a key `"email": "alice@domain.tld"`
[19:58:46]
<Josué> > <@Alekswag:matrix.org> are there other apps that do need the user display name ? Imho if we have features we should be able to justify it by at least 3-4 apps that do need it and can't avoid it, otherwise we end up in the same situation of "this feature exist but we don't know which app actually use it or if it is even used at all" (or "it is used by a single app but nobody knows about it" etc)
well, I don't really have a list in mind. I know it for pgadmin as I'm the maintainer. For the 200 others app I've no idea, maybe someone but not a lot. But well to me currently the issue until now was also that the http header was not really fully documented, so for a packager it's quite difficult to use a not documented feature.
[20:18:37]
<Aleks (he/him/il/lui)> Josué: can you point to where exactly in pgadmin it interfaces with the sso display name header ? x_x i can't find anything in the config ?
[20:21:39]
<Josué> > <@Alekswag:matrix.org> Josué: can you point to where exactly in pgadmin it interfaces with the sso display name header ? x_x i can't find anything in the config ?
In currently in pgadmin we use only the username for now, but for this app it could be an improvement to also set the email and the user fullname from the header.
[20:24:25]
<Aleks (he/him/il/lui)> @_@
[20:25:54]
<Josué> And for XWiki, I saw somewhere that we can set a lot of account info from the HTTP header. Currently the app is connected with LDAP so only the remote_user header is used, but it could be also an example of usage. And also on gitea/forgejo, currently we can't have the header authentification and LDAP at the same time. So we need to chose between the LDAP connection or the SSO feature, in this case it could be also usefull to use the header to populate the account information.
[20:29:41]
<Josué> maybe it could be also great to have in the header the information about the list of the group the user is member of. In dex by example we can use it: https://dexidp.io/docs/connectors/authproxy/
[20:38:32]
<Yunohost Git/Infra notifications> [yunohost] Josue-T pushed 1 commit to auth_headers: Avoid to pass the full headers keys in the cookie ([415b5a67](https://github.com/YunoHost/yunohost/commit/415b5a67765910923ee2da30166f4b8428e24721))
[20:38:43]
<Yunohost Git/Infra notifications> [SSOwat] Josue-T pushed 1 commit to auth_headers: Avoid to pass the full headers keys in the cookie ([c811c9ee](https://github.com/YunoHost/SSOwat/commit/c811c9ee67e492dcdbbef63a01c8df4ed56ff85d))
[20:40:59]
<Aleks (he/him/il/lui)> but like are there really existing mechanism in those apps that allow to populate the create account using HTTP headers or would this be implemented using custom code ... in which case this custom code could very well as well poke the LDAP db in anonymous mode to fetch these infos @_@
[20:42:01]
<Yunohost Git/Infra notifications> MadMan247 forked yunohost-admin to [MadMan247/yunohost-admin](https://github.com/MadMan247/yunohost-admin)
[20:43:36]
<Josué> For gitea it seem yes: https://docs.gitea.com/usage/authentication#reverse-proxy
[20:49:17]
<Josué> for pgadmin it will be custom code, but well for this app it's much easier to do from the header. In most of app it's probably easier to get the info form the header than from LDAP as the header is available everywhere in the application. Sending a LDAP request might be really a pain in case of a customization of an app which don't support LDAP by default.
[21:16:24]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: Update config_panel.toml ([8f8a3afc](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/8f8a3afc25f15a471afc053a3dbd1f7484829375))
[21:21:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL_helpers_2.1: Update post_user_delete ([4fea2f34](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/4fea2f3479aac84d1430dfe2328d3ff9c82dcdd1))
[21:24:05]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL_helpers_2.1: cleaning ([54d013d8](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/54d013d8ebf826963f25c7e81c44226b755d985b))
[21:25:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to v2_PostgreSQL_helpers_2.1: cleaning ([866f5b0f](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/866f5b0fd49917acf5a24335ac9a5813fdc05e2d))