[08:22:21]
<Yunohost Git/Infra notifications> [apps] @rodinux [commented](https://github.com/YunoHost/apps/pull/1632#issuecomment-1430927122) on [issue #1632](https://github.com/YunoHost/apps/pull/1632) Flag Paheko as working: Yesterday I have updated 2 garradin apps in production with paheko. I have this message todayfrom diagnosis > [ERROR] ...
[08:55:22]
<Yunohost Git/Infra notifications> [apps] @fflorent [commented](https://github.com/YunoHost/apps/pull/1632#issuecomment-1430965005) on [issue #1632](https://github.com/YunoHost/apps/pull/1632) Flag Paheko as working: Adhésions is the name of the Paheko installation? If so, try to run another diagnosis: yunohost diagnosis run
[08:57:17]
<rodinux[m]> > <@yunohostinfra:matrix.org> [apps] @fflorent [commented](https://github.com/YunoHost/apps/pull/1632#issuecomment-1430965005) on [issue #1632](https://github.com/YunoHost/apps/pull/1632) Flag Paheko as working: Adhésions is the name of the Paheko installation? If so, try to run another diagnosis: yunohost diagnosis run
The 2 are Paheko, I just rename them in the webadmin...
[09:00:21]
<rodinux[m]> I have the same diagnosis Error... Perhaps I need rename them Paheko...
[09:05:21]
<rodinux[m]> but it is just the tag I have rename....
[09:05:35]
<rodinux[m]> https://aria.im/_matrix/media/v1/download/matrix.org/tGCINNgqsiIhUNvcJQLwNQtR
[09:13:59]
<rodinux[m]> Ok, it is because we have to wait the catalog app to know this application...
[09:14:13]
<rodinux[m]> https://aria.im/_matrix/media/v1/download/matrix.org/MWtpiezOaLaAzVqtXxqFqFTY
[09:14:30]
<rodinux[m]> as working...
[09:25:51]
<Yunohost Git/Infra notifications> [apps] @rodinux [commented](https://github.com/YunoHost/apps/pull/1632#issuecomment-1431005566) on [issue #1632](https://github.com/YunoHost/apps/pull/1632) Flag Paheko as working: I just rename the tiles but the apps are paheko and paheko__2. If I look for paheko in the install, it is tagged: "ne fo...
[09:39:34]
<Guillaume Bouzige> hello everyone,
quick question. when you remove an app that required nodejs to installed, do you remove nodejs as well ?
[09:40:26]
<Guillaume Bouzige> I mean is it a common practice into the packaging yunohost book of good practice, to un-install all dependencies at removal ?
[09:44:30]
<eric_G> > hello everyone,
>
> quick question. when you remove an app that required nodejs to installed, do you remove nodejs as well ?
yes
[09:45:22]
<Guillaume Bouzige> but then if another app is using the same nodejs version, has it been duplicated ?
[09:49:11]
<eric_G> I think this is managed by the helper https://github.com/YunoHost/yunohost/blob/dev/helpers/nodejs
[09:50:08]
<Guillaume Bouzige> ok I shall keep digging, thanks eric for your answer !
[09:51:29]
<eric_G> What is not managed very well is the cleaning up of old versions
[09:51:44]
<Guillaume Bouzige> (also in term of carbon footprint efficiency...having duplicates of nodejs per application can prove being costly)
[09:52:04]
<Guillaume Bouzige> > <@ericg:matrix.org> What is not managed very well is the cleaning up of old versions
ok that might be it...
[09:52:32]
<eric_G> maybe in here `/usr/local/n/versions/node`
[09:53:31]
<Guillaume Bouzige> > <@ericg:matrix.org> maybe in here `/usr/local/n/versions/node`
yeah I went there last time...I got scared
[09:53:32]
<Guillaume Bouzige> 🫠
[09:54:51]
<Guillaume Bouzige> hence my first question...
[10:02:21]
<eric_G> The problem is that when a node version gets updated, old versions are kept, ie 14.1, 14.2, 14.3 14.x... same for 16, 18 and so one. Ideally we should only have 12.x, 14.x, 16.x, 18.x (.x being the last update available)
[10:03:10]
<eric_G> The Node helper needs to be improved
[10:05:31]
<Guillaume Bouzige> ok we can see through that
[10:13:27]
<Yunohost Git/Infra notifications> [apps] @fflorent [commented](https://github.com/YunoHost/apps/pull/1632#issuecomment-1431071191) on [issue #1632](https://github.com/YunoHost/apps/pull/1632) Flag Paheko as working: Have you tried? yunohost diagnosis run and: yunohost diagnosis show --human-readable To check whether ...
[10:14:55]
<Yunohost Git/Infra notifications> [apps] @rodinux [commented](https://github.com/YunoHost/apps/pull/1632#issuecomment-1431074714) on [issue #1632](https://github.com/YunoHost/apps/pull/1632) Flag Paheko as working: > Have you tried? > > > yunohost diagnosis run > > > and: > > > yunohost diagnosis show --human-readable ...
[10:20:52]
<Yunohost Git/Infra notifications> [apps] @rodinux [commented](https://github.com/YunoHost/apps/pull/1632#issuecomment-1431085507) on [issue #1632](https://github.com/YunoHost/apps/pull/1632) Flag Paheko as working: OOOh now i the webadmin it says "broken"
[10:21:58]
<rodinux[m]> What's happening ?? now in the webadmin I can see "broken"...
[10:21:58]
<rodinux[m]> https://aria.im/_matrix/media/v1/download/matrix.org/YwCjcYhwWYxLhQOeqPZGYJMH
[10:23:07]
<eric_G> https://ci-apps.yunohost.org/ci/ Paheko as not yet been CIed
[10:23:24]
<Yunohost Git/Infra notifications> [apps] @rodinux [commented](https://github.com/YunoHost/apps/pull/1632#issuecomment-1431090194) on [issue #1632](https://github.com/YunoHost/apps/pull/1632) Flag Paheko as working: I send screenshots also on matrix...
[10:25:42]
<rodinux[m]> > <@ericg:matrix.org> https://ci-apps.yunohost.org/ci/ Paheko as not yet been CIed
OK, it is the next step to wait... Thanks eric_G
[10:26:31]
<florent> (or is there any way to trigger a run? right now there is room for a job run)
[10:31:08]
<eric_G> on the [ci-dev](https://ci-apps-dev.yunohost.org/ci/) maybe but you need to wait for the [ci](https://ci-apps.yunohost.org/ci/) to do his job
[10:35:27]
<rodinux[m]> Ok, so when the task will be done the issue I have will be resolved ? exact ?
[10:36:32]
<eric_G> yes
[10:37:30]
<florent> > <@ericg:matrix.org> on the [ci-dev](https://ci-apps-dev.yunohost.org/ci/) maybe but you need to wait for the [ci](https://ci-apps.yunohost.org/ci/) to do his job
I see, thanks!
[12:30:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/407#issuecomment-1431295340) on [issue #407](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/407) add support for custom data directory: testme
[12:30:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/407#issuecomment-1431295374) on [issue #407](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/407) add support for custom data directory: Fingers crossed
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5663/bad...
[14:03:35]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/504#issuecomment-1431420021) on [issue #504](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/504) update failed from 22.2.10~ynh1 to 24.0.4~ynh1: Should be fixed with #555
[14:03:36]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #504](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/504): update failed from 22.2.10~ynh1 to 24.0.4~ynh1
[14:04:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #524](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/524): Upgrade from 19.0 to 24.0 fails, due to php version
[14:04:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/524#issuecomment-1431421078) on [issue #524](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/524) Upgrade from 19.0 to 24.0 fails, due to php version: Should be fixed with #555
[14:04:37]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/520#issuecomment-1431421608) on [issue #520](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/520) Upgrade error from (21? / 22) to 24: should be fixed with #555
[14:04:37]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #520](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/520): Upgrade error from (21? / 22) to 24
[14:05:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/474#issuecomment-1431422702) on [issue #474](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/474) /etc/php/7.3/fpm/pool.d/nextcloud.conf does not exist, Could not upgrade nextcloud: An error occurred inside the app upgrade script: should be fixed with #555
[14:05:17]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #474](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/474): /etc/php/7.3/fpm/pool.d/nextcloud.conf does not exist, Could not upgrade nextcloud: An error occurred inside the app upgrade script
[14:06:00]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/452#issuecomment-1431423393) on [issue #452](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/452) Upgrade nextcloud from 21.0.2~ynh1 to 22.2.0~ynh1 Failure: should be fixed with #555
[14:07:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed [issue #387](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/387): Upgrade fro 19 to 20 is impossible
[14:53:31]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to Built-in-CODE-server: Add code server ([29eece06](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/29eece06752d88b7e3d3fdaccbd975e374a22cf2))
[14:53:31]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar created new branch Built-in-CODE-server
[15:10:12]
<kayou> does someone add Ddataa as a maintainer of AdguardHome?
[15:10:12]
<kayou> https://github.com/YunoHost-Apps/adguardhome_ynh/pull/90
[15:10:19]
<kayou> eric_G maybe?
[15:12:28]
<tituspijean> huh I thought we could go back up to the message when clicking on the quote, but no
[15:13:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @Thatoo [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/317#issuecomment-1424073413) on [issue #317](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/317) Keeweb doesnt work: I wonder if something like that could work location ~ ^/(?apps\/keeweb) { include conf.d/yunohost_panel....
[15:13:10]
<kayou> we can, but it's kind of buggy
[15:13:22]
<kayou> ok ok, i was not sure and didn't saw the ping
[15:15:52]
<kayou> Guillaume Bouzige: Yunohost will not upgrade the package after this merge: https://github.com/YunoHost-Apps/adguardhome_ynh/pull/90
you must change the version https://github.com/YunoHost-Apps/adguardhome_ynh/blob/d0d22d21736909582f4457700a2732833c9baf8d/manifest.json#L9
[15:16:33]
<kayou> if you want YunoHost detect the upgrade
[15:23:07]
<florent> Salamandar: ElasticSearch 7 has been updated due to fix a bug which made the Ci fail (also the service could not start after a reboot of the server)
[18:01:52]
<Guillaume Bouzige> > <@kayou:matrix.org> Guillaume Bouzige: Yunohost will not upgrade the package after this merge: https://github.com/YunoHost-Apps/adguardhome_ynh/pull/90
>
> you must change the version https://github.com/YunoHost-Apps/adguardhome_ynh/blob/d0d22d21736909582f4457700a2732833c9baf8d/manifest.json#L9
ok, thanks for letting me know
[18:15:05]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 6 commits to v2 ([d72cb6228992...76a1324f7eee](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/d72cb6228992...76a1324f7eee))
[18:15:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to v2: Auto-update README ([7cce0307](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/7cce030768ee8c35d9aeaf00e4bc6a857dbdeea0))
[18:15:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Fix - Éric Gaspar
[18:15:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Auto-update README - yunohost-bot
[18:15:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh/v2] Merge branch testing into v2 - Éric Gaspar
[18:15:31]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1431806816) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: testme
[18:15:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1431806859) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5667/badge)](...
[18:16:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1427951101) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: :rocket:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5618/badge)](htt...
[18:16:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1426667262) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: testme
[18:16:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1426667268) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: Fingers crossed
[[Test Badge](https://img.shields.io/endpoint?url=https://ci.pijean.ovh/ci/api/job/502/badge)](https:/...
[18:16:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1427951043) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: testme
[18:16:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1426667269) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5596/badge)](...
[18:16:31]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1428415552) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: testme
[18:16:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#issuecomment-1428415625) on [issue #559](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559) Packaging version 2: :v:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/5620/badge)](https://...
[18:20:06]
<Yunohost Git/Infra notifications> [package_linter] @alexAubin pushed 1 commit to master: Bump use of --package option in ynh_add_fpm_config as an error, we really gotta get rid of this legacy stuff ([5d75cc2a](https://github.com/YunoHost/package_linter/commit/5d75cc2a73c3257e1f34d40a17e67f06a8a262a0))
[18:21:13]
<Yunohost Git/Infra notifications> [package_linter] @alexAubin pushed 1 commit to master: Remove unecessary warning about apps.json not existing ([c90526db](https://github.com/YunoHost/package_linter/commit/c90526dbd6c95f50d7f47afd7420fb2a0b7bbbb5))
[18:48:21]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/559#discussion_r1107562914) on pull request #559 Packaging version 2: suggestion api.url = "re:__DOMAIN__\\/.well-known\\/.*" im pushing a change in the core to support this syn...
[20:00:20]
<tituspijean> Guillaume Bouzige: https://forum.yunohost.org/t/cryptpad-v5-2-1-blocage-via-sandbox-cryptpad/23645
J'ai bon? Si oui il faudrait créer un fichier post-upgrade.md (j'ai plus le nom en tête)pour notifier les utilisateurs qu'il faut faire une étape manuelle.
[20:02:29]
<Guillaume Bouzige> oui, ya pas d'emails envoyer a l'admin pour l'upgrade surement
[20:22:30]
<Guillaume Bouzige> > <@titus:pijean.ovh> Guillaume Bouzige: https://forum.yunohost.org/t/cryptpad-v5-2-1-blocage-via-sandbox-cryptpad/23645
>
> J'ai bon? Si oui il faudrait créer un fichier post-upgrade.md (j'ai plus le nom en tête)pour notifier les utilisateurs qu'il faut faire une étape manuelle.
https://github.com/Ddataa/cryptpad_ynh/blob/2e17de7978e1e4f976f597a64fdfb6d27a46ed7e/scripts/upgrade#L209 si pourtant
[20:24:46]
<tituspijean> Ah oui en effet. 👍
[20:31:12]
<eric_G> I haven't received any email after install :/
[20:32:00]
<Aleks (he/him/il/lui)> can you confirm the address you're checking receives the mails sent to admin@ ?
[20:33:37]
<eric_G> eric@
[20:33:46]
<eric_G> 🫠
[20:34:32]
<eric_G> BecauseIamTheAdmin 😅
[20:34:55]
<Aleks (he/him/il/lui)> hmyeah so should work i guess
[20:35:08]
<Aleks (he/him/il/lui)> if you want to debug it you can try looking at `tail /var/log/mail.info -n 100`
[20:35:18]
<Aleks (he/him/il/lui)> or `tail /var/log/mail.err -n 50` maybe
[20:35:25]
<Aleks (he/him/il/lui)> but the logs are kinda horrendous
[20:45:18]
<eric_G> `dovecot: auth: Error: net_connect_unix(/run/dovecot/stats-writer) failed: Permission denied`
[20:45:45]
<eric_G> but that was at 15:25
[20:46:16]
<eric_G> february 13
[20:48:25]
<Aleks (he/him/il/lui)> yeah it's unrelated
[21:12:47]
<eric_G> after install, CryptPad is "Loading" indefinitely
[21:21:48]
<eric_G> and on upgrading the app:
```
Now upgrading cryptpad...
Info: [....................] > Loading installation settings...
Info: [....................] > Checking version...
Info: [....................] > Backing up the app before upgrading (may take a while)...
Info: [....................] > Stopping a systemd service...
Info: [+...................] > Ensuring downward compatibility...
Info: [#...................] > Making sure dedicated system user exists...
Info: [#...................] > Setting up sandobx domain : sandbox-cryptpad.ericandmarie.fr
Warning: The domain already exists
Warning: [Error] Upgrade failed.
Warning: 42372 Could not restore cryptpad: Something unexpected went wrong:
Warning: Traceback (most recent call last):
Warning: File "/usr/lib/python3/dist-packages/yunohost/backup.py", line 1458, in _restore_app
Warning: permission_create(
Warning: File "/usr/lib/python3/dist-packages/yunohost/log.py", line 410, in func_wrapper
Warning: result = func(*args, **kwargs)
Warning: File "/usr/lib/python3/dist-packages/yunohost/permission.py", line 450, in permission_create
Warning: permission_url(
Warning: File "/usr/lib/python3/dist-packages/yunohost/log.py", line 410, in func_wrapper
Warning: result = func(*args, **kwargs)
Warning: File "/usr/lib/python3/dist-packages/yunohost/permission.py", line 546, in permission_url
Warning: ur = _validate_and_sanitize_permission_url(ur, app_main_path, app)
Warning: File "/usr/lib/python3/dist-packages/yunohost/permission.py", line 926, in _validate_and_sanitize_permission_url
Warning: _assert_domain_exists(domain)
Warning: File "/usr/lib/python3/dist-packages/yunohost/domain.py", line 186, in _assert_domain_exists
Warning: raise YunohostValidationError("domain_unknown", domain=domain)
Warning: yunohost.utils.error.YunohostValidationError: Domain 'sandbox-cryptpad.ericandmarie.fr' unknown
Warning: 45196 Nothing was restored
Warning: Uhoh ... Yunohost failed to restore the app to the way it was before the failed upgrade :|
Error: Could not upgrade cryptpad: An error occurred inside the app upgrade scriptInfo: The operation 'Upgrade the 'cryptpad' app' could not be completed. Please share the full log of this operation using the command 'yunohost log share 20230215-211745-app_upgrade-cryptpad' to get help
Warning: Here's an extract of the logs before the crash. It might help debugging the error:
Info: DEBUG - + return 1
Info: INFO - [#...................] > Making sure dedicated system user exists...
Info: DEBUG - + ynh_system_user_create --username=cryptpad --home_dir=/var/www/cryptpad
Info: DEBUG - + use_shell=0
Info: DEBUG - + home_dir=/var/www/cryptpad
Info: DEBUG - + groups=
Info: DEBUG - + ynh_system_user_exists cryptpad
Info: DEBUG - + getent passwd cryptpad
Info: DEBUG - + '[' UPGRADE_PACKAGE == UPGRADE_APP ']'
Info: DEBUG - + chgrp -R www-data /var/www/cryptpad
Info: DEBUG - + [[ cryptpad.ericandmarie.fr == *\.* ]]
Info: DEBUG - + sandboxdomain=sandbox.cryptpad.ericandmarie.fr
Info: DEBUG - + [[ cryptpad.ericandmarie.fr == *\.*\.* ]]
Info: DEBUG - + sandboxdomain=sandbox-cryptpad.ericandmarie.fr
Info: DEBUG - + [[ cryptpad.ericandmarie.fr == *\.\l\o\c\a\l ]]
Info: INFO - [#...................] > Setting up sandobx domain : sandbox-cryptpad.ericandmarie.fr
Info: DEBUG - + yunohost domain add sandbox-cryptpad.ericandmarie.fr
Info: DEBUG - attribute 'virtualdomain' with value 'sandbox-cryptpad.ericandmarie.fr' is not unique
Info: WARNING - The domain already exists
Info: DEBUG - + ynh_exit_properly
```
[21:22:47]
<eric_G> ```
ynh_script_progression --message="Setting up sandobx domain : $sandboxdomain" --weight=1
# We don't test that in CI
if ! [ ${PACKAGE_CHECK_EXEC:-0} -eq 1 ]; then
yunohost domain add $sandboxdomain
yunohost domain config set $sandboxdomain -a "mail_in=0&mail_out=0"
fi
```
[21:22:55]
<eric_G> it seems to fail here
[21:23:04]
<Guillaume Bouzige> wot
[21:24:03]
<Guillaume Bouzige> ` WARNING - The domain already exists`
[21:24:17]
<Guillaume Bouzige> you shall not have a sandbox domain already...
[21:26:09]
<Guillaume Bouzige> the very very latest version now is also better
[21:27:43]
<eric_G> ah... CryptPad has disappeared from my installed apps...
[21:28:14]
<eric_G> a well as the sandbox domain
[21:29:23]
<eric_G> ...reinstalling
[21:30:09]
<Guillaume Bouzige> try to reinstall with the link to https://github.com/YunoHost-Apps/cryptpad_ynh so you get the latest latest version
[21:35:59]
<eric_G> Installed from testing and it work!
[21:36:55]
<eric_G> I am trying to force upgrade to check if `The domain already exists` warning persists
[21:37:41]
<Guillaume Bouzige> yeah the work is just starting, there some more edge cases I havent cover. like change_url for example :)
[21:40:42]
<eric_G> still fails...
[21:41:00]
<eric_G> https://paste.yunohost.org/raw/kedujuvenu
[21:42:13]
<Guillaume Bouzige> hum not sure I follow here...
[21:42:55]
<Guillaume Bouzige> you trying to upgrade with the cli from 5.2.1~ynh2 to 5.2.1~ynh2
[21:43:05]
<Guillaume Bouzige> yeah it fails, normal
[21:43:28]
<eric_G> not normal 😄
[21:44:01]
<eric_G> I add the same error from 5.2.1~ynh2 to 5.2.1~ynh2
[21:44:11]
<eric_G> 5.2.1~ynh1 to 5.2.1~ynh2
[21:44:17]
<Guillaume Bouzige> aahh
[21:44:34]
<Guillaume Bouzige> ok
[21:45:55]
<eric_G> Also, upgrading from the same version should not fail
[21:46:19]
<Guillaume Bouzige> yes. I shall check if sandbox domain already exist and skip creation then
[21:48:53]
<eric_G> because at the moment the user will, on upgrading, just see they CryptPad disappear 😱
[21:49:24]
<Guillaume Bouzige> noice
[21:49:42]
<eric_G> happy news
[21:49:55]
<eric_G> there is a backup
[21:50:29]
<Guillaume Bouzige> this is the way `sudo yunohost app checkurl $domain$path -a $app` ?
[21:52:11]
<Guillaume Bouzige> ou plutot `yunohost domain list --output-as plain | grep $sandboxdomain`
[21:54:19]
<eric_G> or
```
# We don't test that in CI
if ! [ ${PACKAGE_CHECK_EXEC:-0} -eq 1 ]; then
# If sandboxdomain doesn't exist, create it
if [ -z "$sandboxdomain" ]; then
yunohost domain add $sandboxdomain
yunohost domain config set $sandboxdomain -a "mail_in=0&mail_out=0"
fi
fi
```
[21:55:27]
<eric_G> but there is certainly a smarter way to do it
[21:56:09]
<Guillaume Bouzige> I do not know this -z blackmagic at the moment
[21:57:15]
<Guillaume Bouzige> I summit to your script superior knowledge
[21:57:41]
<Aleks (he/him/il/lui)> `-z` is short for "is empty" of course
[21:59:39]
<Guillaume Bouzige> so is it correct ?
[22:00:28]
<Guillaume Bouzige> I am confused, it doesnt mean the domain does already exist tho if the variable is empty or not
[22:01:42]
<eric_G> and I got an error trying to restore the backup https://paste.yunohost.org/raw/upikewajel
[22:06:36]
<Guillaume Bouzige> the restore script hasnt been changed since
[22:06:58]
<Aleks (he/him/il/lui)> hmf yeah that's because the whole thing about permission-with-a-different-domain is not really supported and it's not trivial
[22:07:17]
<Aleks (he/him/il/lui)> basically yunohost wants to restore the permission before restoring the backup
[22:07:20]
<Aleks (he/him/il/lui)> but the permission includes the other domain
[22:07:22]
<Aleks (he/him/il/lui)> which must exists
[22:07:45]
<Guillaume Bouzige> ohoah so you know
[22:07:58]
<Aleks (he/him/il/lui)> yeah i don't remember who already tried this kind of stuff
[22:08:05]
<Aleks (he/him/il/lui)> i though this was already with that same app
[22:08:40]
<Guillaume Bouzige> it is not trivial I concede
[22:11:05]
<Guillaume Bouzige> so we need to create the sandbox domain in the restore script before yunhost try to restore the backup permissions
[22:11:48]
<Aleks (he/him/il/lui)> yes but the restore script is called *after* yunohost restores the permission ...
[22:12:08]
<Guillaume Bouzige> hook we need a hook
[22:12:11]
<Guillaume Bouzige> no ?
[22:13:03]
<Aleks (he/him/il/lui)> ¯\_(ツ)_/¯
[22:13:04]
<Aleks (he/him/il/lui)> how are you gonna install the hook even before the app is restored at all
[22:13:05]
<Guillaume Bouzige> hook doesnt do pre and post stuffs
[22:13:06]
<Aleks (he/him/il/lui)> or like a custom thing like "if script X exists in the app package, run it before" yeah >_>
[22:13:12]
<Guillaume Bouzige> lolilol
[22:14:05]
<Guillaume Bouzige> who call the restore script then
[22:14:33]
<Aleks (he/him/il/lui)> what was it called already ? `package-is-a-pain-in-the-ass` antifeature ? 😬
[22:15:02]
<Guillaume Bouzige> this one yes, that sounds like the one
[22:15:13]
<Aleks (he/him/il/lui)> can't we just ask the user to configure the sandbox thingy or idk ? x_x
[22:15:28]
<Aleks (he/him/il/lui)> (i have no idea what this sandbox thingy exists)
[22:15:28]
<Guillaume Bouzige> yeah for the restore that would be fine
[22:16:08]
<Guillaume Bouzige> so wait you need to create your sandbox domain, that would suffice
[22:16:49]
<Aleks (he/him/il/lui)> well maybe a permission is needed too such that the domain is public or something
[22:17:01]
<Aleks (he/him/il/lui)> or like isn't there an nginx config thing
[22:17:52]
<Guillaume Bouzige> yeah that famous one `ynh_permission_url --permission="main" --add_url=$sandboxdomain --auth_header=true`
[22:18:22]
<Guillaume Bouzige> but the user cannot do that himself
[22:18:26]
<Aleks (he/him/il/lui)> could also imagine having another "companion"(?) app like `cryptpad-sandbox` or idk ...
[22:18:51]
<Guillaume Bouzige> yeah i explored that way too but it didnt help and was more heavy
[22:19:28]
<Aleks (he/him/il/lui)> 😐️
[22:19:34]
<Guillaume Bouzige> there still a repo or two hanging on that idea it somewhere in github
[22:19:59]
<Guillaume Bouzige> but i found it also even more confusing than having to do manual admin thingy
[22:56:19]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @Thatoo [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/317#issuecomment-1424073413) on [issue #317](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/317) Keeweb doesnt work: I wonder if something like that could work location ~ ^/(?apps\/keeweb) { include conf.d/yunohost_panel....
[22:57:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @Thatoo [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/317#issuecomment-1432190669) on [issue #317](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/317) Keeweb doesnt work: We could add a location like location ^~ /apps/keeweb/ { xxxxxxx } and rewrite all the first location inside thi...