[01:38:22]
<m606> > <@Alekswag:matrix.org> idk, your getter is just super simple, there something obvious we're missing ...
As I added no `bind` property, settings should be saved in `/etc/yunohost/apps/fontcompare/settings.yml` as per the doc.
I do have that file there but here's what it contains (not my custom options)
```
app: fontcompare
checksum__etc_nginx_conf.d_mydomain.tld.d_fontcompare.conf: 4f9e2d7a239f91769c478e53cfdbb3fe
current_revision: c436f5bec7e08bc44e9e464f832f77402af2d354
domain: mydomain.tld
id: fontcompare
install_dir: /var/www/fontcompare
install_time: 1733937297
path: /fontcompare
```
[01:44:46]
<m606> Also, I notice the getter script is adding a space before the value:
```
861 DEBUG + for short_setting in "${!old[@]}"
861 DEBUG + [[ #b514e1 != YNH_NULL ]]
862 DEBUG + [[ yaml == \y\a\m\l ]]
863 DEBUG + ynh_return background:
863 DEBUG + echo background:
864 DEBUG ++ echo '#b514e1'
864 DEBUG ++ sed 's/^/ /g'
867 DEBUG + ynh_return ' #b514e1'
867 DEBUG + echo ' #b514e1'
```
Probably not an issue as I guess it does it on other working config panel, but I mention it just it case it could somehow make the yaml/json property invalid
[01:46:45]
<Aleks (he/him/il/lui)> ah now that you mention it
[01:46:49]
<Aleks (he/him/il/lui)> well it's unrelated but
[01:46:57]
<Aleks (he/him/il/lui)> the fact that it starts with "#"
[01:47:06]
<Aleks (he/him/il/lui)> probably is interpreted as a comment by the yaml parser...
[01:47:19]
<m606> uhh
[01:47:31]
<m606> let me try without #
[01:54:24]
<m606> > <@Alekswag:matrix.org> probably is interpreted as a comment by the yaml parser...
looks like this is it!
I modified the getter to echo `echo ${current_color:1}` so that it removes the starting `#`
Now in `yunohost app config get fontcompare --full --debug` I do find the value:
```
options:
0:
ask: Default background color
bind: settings
default: #cccccc
id: background
mode: bash
optional: True
readonly: False
redact: False
type: color
value: 78b7de
visible: True
```
But webadmin shows an error because as this is not a valid color code.
[01:55:05]
<Aleks (he/him/il/lui)> :psyduck:
[01:58:53]
<m606> but.. strange things is that `default : #cccccc` as set in `config-panel.toml` is accepted and working
[01:59:37]
<m606> the issue is between the getter output and the yaml file
[02:04:15]
<m606> just to be sure, if I `yunohost app config get fontcompare --full --debug --output-as json` I get the same behavior than with yaml. Here is without removing starting `#` from the getter output:
```
[
{
"options": [
{
"type": "color",
"id": "background",
"mode": "bash",
"ask": "Default background color",
"readonly": false,
"visible": true,
"bind": "settings",
"redact": false,
"optional": true,
"default": "#cccccc",
"value": null
}
```
[08:18:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] utzer [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2540844555) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: Here it worked without any problem as well.
[08:18:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] utzer [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2540844555) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: Here it worked without any problem as well. Thanks for the great work, not only now, but all the time
[08:26:06]
<Yunohost Git/Infra notifications> [borg_ynh] utzer [commented](https://github.com/YunoHost-Apps/borg_ynh/pull/176#issuecomment-2540856787) on [issue #176](https://github.com/YunoHost-Apps/borg_ynh/pull/176) Fix file changed during backup: Why is this not merged? I get 12 emails each day (during each backup one) for this problem, it made the notifications of...
[08:27:34]
<Yunohost Git/Infra notifications> [synapse_ynh] utzer [commented](https://github.com/YunoHost-Apps/synapse_ynh/issues/474#issuecomment-2540859120) on [issue #474](https://github.com/YunoHost-Apps/synapse_ynh/issues/474) Borg backup failed to backup synapse application: Is there no hooks or options that an app can provide for backups? Like omitting some path or so?
[08:39:45]
<Yunohost Git/Infra notifications> [borg_ynh] fflorent [commented](https://github.com/YunoHost-Apps/borg_ynh/pull/176#issuecomment-2540882280) on [issue #176](https://github.com/YunoHost-Apps/borg_ynh/pull/176) Fix file changed during backup: I dont have the merge grant, so I need someone to review this patch.
[13:31:13]
<eric_G> Quésako?
```
16317 WARNING WARNING ./15-nginx: line 39: ynh_render_template: command not found
16432 WARNING ERROR Could not run script: /usr/share/yunohost/hooks/conf_regen/15-nginx
16456 WARNING WARNING ./98-nginx_garage: line 11: ynh_replace_special_string: command not found
```
[13:31:15]
<eric_G> https://ci-apps-dev.yunohost.org/ci/job/4829
[13:31:40]
<Salamandar> missing `source` ?
[13:32:44]
<Salamandar> ah no the script calls `yunohost tools regen-conf nginx` directly
[13:33:00]
<Salamandar> i know
[13:33:12]
<Salamandar> running with helpers v2.1 exports a variable
[13:33:21]
<Salamandar> you need to unexport it when calling `yunohost something`
[13:33:40]
<Salamandar> cuz the internals of yunohost (like the regen-conf) are not yet using the v2.1 helpers
[13:33:54]
<Salamandar> saw something like that a few weeks ago
[13:34:14]
<Salamandar> but it should not be the responsibility of the app… but of the yunohost cmdline
[13:34:27]
<Salamandar> (or even the `regen-conf` code)
[13:35:07]
<Aleks (he/him/il/lui)> heurghaurh
[13:36:08]
<eric_G> grebheu?
[13:38:38]
<quiwy> Hi, could someone review this PR please 🙏 https://github.com/YunoHost-Apps/actual_ynh/pull/50 😀
[13:40:49]
<eric_G> team work 🤝
[13:40:55]
<quiwy> Thx !
[13:42:17]
<Yunohost Git/Infra notifications> [collabora_ynh] ericgaspar merged [pull request #92](https://github.com/YunoHost-Apps/collabora_ynh/pull/92): Testing
[13:43:25]
<eric_G> merging. "Quoi qu'il en coûte" 😅
[14:18:42]
<quiwy> Same for this PR please 🙏 https://github.com/YunoHost-Apps/actual_ynh/pull/51
[14:21:53]
<eric_G> https://github.com/YunoHost-Apps/documize_ynh/issues/29
[14:22:05]
<Aleks (he/him/il/lui)> you probably also want to remove the --keep here : https://github.com/Quiwy/actual_ynh/blob/master/scripts/upgrade#L18
[14:22:06]
<eric_G> this issue is driving me nuts
[14:22:36]
<quiwy> true
[14:23:36]
<quiwy> can one of you fix it on the branch ci-auto-update-24.12.0 or should I do another PR as eric_G already merged it ?
[14:24:40]
<quiwy> Thx
[14:24:48]
<eric_G> will an `.env` file work?
[14:58:39]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2747](https://github.com/YunoHost/apps/pull/2747): Add Ferdium Server to wishlist
[14:58:40]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2747](https://github.com/YunoHost/apps/pull/2747): Add Ferdium Server to wishlist
[15:05:05]
<m606> So, this seems to get settings when yml setting file is defined: https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L2056
But in my current case, i havn't set "bind" properties in config-panel.toml so it is meant to default to /etc/yunohost/apps/$app/settings.yml (as per the doc). So it could rather be inside [`_get_AppConfigPanel`](https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L1887-L2126) and following 2 functions. But what I don't get is where the value is stored ? Apparently not in `/etc/yunohost/apps/$app/settings.yml` as I can't find anything related, but definitely somewhere because if I change the getter to not output the starting `#`, `yunohost app config get fontcompare --full --debug` outputs `value: 78b7de` (instead of None with starting `#`). `grep -rli '78b7de' /etc/yunohost/` returns nothing.
My objective being to find the code triggering the yaml ouput of `yunohost app config get fontcompare --full --debug` to see whether I could have the HTML color code value between quotes.
[16:01:45]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2748](https://github.com/YunoHost/apps/pull/2748): Update app levels according to CI results
[16:08:26]
<eric_G> is the main CI getting slooow?
[16:49:31]
<Salamandar> > <@ericg:matrix.org> is the main CI getting slooow?
oh ?
[16:49:57]
<Salamandar> no it's getting stuck
[16:50:04]
<Salamandar> like "disk full" or something
[16:50:09]
<Salamandar> while it shouldn't
[17:44:27]
<Yunohost Git/Infra notifications> ericgaspar created repository agewasm_ynh https://github.com/YunoHost-Apps/agewasm_ynh
[17:45:52]
<Yunohost Git/Infra notifications> ericgaspar edited repository agewasm_ynh: agewasm package for YunoHost https://github.com/YunoHost-Apps/agewasm_ynh
[18:08:58]
<eric_G> I don't know if its just me but I see quite few apps with authentication not working properly... (etherpad, plainpad, documize...)
[18:15:44]
<Aleks (he/him/il/lui)> yeah could be some apps that we didn't identify right away as needing the basic auth stuff ... does using `yunohost app $app protect_against_basic_auth_spoofing -v false` then `yunohost app ssowatconf` fixes those issues ?
[18:20:31]
<eric_G> `yunohost app plainpad protect_against_basic_auth_spoofing -v false` gives me an error
```
usage: yunohost app
{catalog,search,manifest,list,info,map,install,remove,upgrade,change-url,setting,shell,register-url,makedefault,dismiss-notification,ssowatconf,change-label,action,config}
...
[-h]
yunohost app: error: argument {catalog,search,manifest,list,info,map,install,remove,upgrade,change-url,setting,shell,register-url,makedefault,dismiss-notification,ssowatconf,change-label,action,config}: invalid choice: 'plainpad' (choose from 'catalog', 'search', 'manifest', 'list', 'info', 'map', 'install', 'remove', 'upgrade', 'change-url', 'setting', 'shell', 'register-url', 'makedefault', 'dismiss-notification', 'ssowatconf', 'change-label', 'action', 'config')
```
[18:21:12]
<Aleks (he/him/il/lui)> yeah, "app setting" ...
[18:21:19]
<Aleks (he/him/il/lui)> ` yunohost app setting $app protect_against_basic_auth_spoofing -v false`
[18:28:39]
<Yunohost Git/Infra notifications> [vaultwarden_ynh] yalh76 [commented](https://github.com/YunoHost-Apps/vaultwarden_ynh/pull/280#issuecomment-2542021601) on [issue #280](https://github.com/YunoHost-Apps/vaultwarden_ynh/pull/280) Fix trixie: Own Ci result for Revision: 6624d2774c28ada24c9b562b9d5571c166f6e9af Repository: https://github.com/YunoHost-Apps/vault...
[18:29:03]
<Yunohost Git/Infra notifications> [vaultwarden_ynh] yalh76 merged [pull request #280](https://github.com/YunoHost-Apps/vaultwarden_ynh/pull/280): Fix trixie
[18:29:06]
<Yunohost Git/Infra notifications> [vaultwarden_ynh] yalh76 deleted branch Fix_trixie
[18:29:28]
<Yunohost Git/Infra notifications> [vaultwarden_ynh] yalh76 opened [pull request #281](https://github.com/YunoHost-Apps/vaultwarden_ynh/pull/281): Fix trixie
[18:29:34]
<Yunohost Git/Infra notifications> [vaultwarden_ynh] yalh76 edited [pull request #281](https://github.com/YunoHost-Apps/vaultwarden_ynh/pull/281): Fix trixie
[18:29:36]
<Yunohost Git/Infra notifications> [vaultwarden_ynh] yalh76 edited [pull request #281](https://github.com/YunoHost-Apps/vaultwarden_ynh/pull/281): Fix trixie
[18:35:29]
<eric_G> no success with the `protect_against_basic_auth_spoofing` set to false
[22:32:24]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to agewasm: Update apps.toml ([34615df5](https://github.com/YunoHost/apps/commit/34615df5b80bed909371bb13c858b2bcb08aa8bc))
[22:32:24]
<Yunohost Git/Infra notifications> [apps] ericgaspar created new branch agewasm
[22:32:51]
<Yunohost Git/Infra notifications> [apps] ericgaspar opened [pull request #2749](https://github.com/YunoHost/apps/pull/2749): add agewasm to catalog
[22:34:53]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2749](https://github.com/YunoHost/apps/pull/2749): add agewasm to catalog
[22:34:53]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Update apps.toml (#2749) ([04e5955d](https://github.com/YunoHost/apps/commit/04e5955de035f464abd858bf7209ee72bec663b6))
[22:34:54]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch agewasm