Friday, January 10, 2025
apps@conference.yunohost.org
January
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
   
             

[11:39:01] <Salamandar> gniiiii
[11:39:01] <Salamandar> > Errors :
> - Got return code 200, but was expecting 202
[11:39:30] *Salamandar writing a pull request to add support for `expect_return_code` being a list
[14:15:58] <rodinux> hello, on cryptpad last release we still have a old permissions we want to remove on the script upgrade
```
permissions:
cryptpad.main:
additional_urls: sandbox-domain.tld
```
how can we remove it ?? something like this ??
```
ynh_permission_url --permission="main" --remove_url="sandbox-$domain" || true
ynh_permission_url --permission="main" --remove_url="sandbox.$domain" || true
```
[14:32:26] <rodinux> this permission configuration come from the old config when we needed to use the explicit permission for the subdomain (out of the manifest)
[14:40:35] <Aleks (he/him/il/lui)> if this is v2 packaging the permission automatically gets removed if it's not in the manifest anymore
[14:41:02] <Aleks (he/him/il/lui)> same for the additional url
[14:41:05] <Aleks (he/him/il/lui)> in theory at least
[14:43:08] <rodinux> In the last releases, there is only

```
[resources.permissions]
main.url = "/"
```

and this permissions seems here from oldest as we remove such domains...

[15:42:47] <rodinux> it coud be this ?
```
ynh_permission_url --permission="cryptpad.main" --remove_url="sandbox-$domain" || true
ynh_permission_url --permission="cryptpad.main" --remove_url="sandbox.$domain" || true
```
[15:45:00] <Aleks (he/him/il/lui)> >if this is v2 packaging the permission automatically gets removed if it's not in the manifest anymore
[15:50:29] <Yunohost Git/Infra notifications> [borg_ynh] t​io-trom [commented](https://github.com/YunoHost-Apps/borg_ynh/pull/188#issuecomment-2583019415) on [issue #188](https://github.com/YunoHost-Apps/borg_ynh/pull/188) Strip work_dir from paths inside the backup archive: I made the same changes manually to my borg setup. Restarted the server. No change. The .tar archives have the same stru...
[15:51:20] <rodinux> So why is this permission is still here after an upgrade ? perhaps it doesn't matter....
[15:57:32] <Aleks (he/him/il/lui)> i don't get how it's possible, here it precisely reset the additional url stuff of the perm to what's declared in the manifest :| https://github.com/YunoHost/yunohost/blob/dev/src/utils/resources.py#L746
[15:58:03] <Yunohost Git/Infra notifications> [borg_ynh] o​rhtej2 [commented](https://github.com/YunoHost-Apps/borg_ynh/pull/188#issuecomment-2583049273) on [issue #188](https://github.com/YunoHost-Apps/borg_ynh/pull/188) Strip work_dir from paths inside the backup archive: @tio-trom have you modified the deployed file in hooks.d dir?
[15:58:30] <Aleks (he/him/il/lui)> if you're able to reproduce the issue it would be nice to debug what exactly in that part of the code during the upgrade
[16:02:21] <Yunohost Git/Infra notifications> [borg_ynh] t​io-trom [commented](https://github.com/YunoHost-Apps/borg_ynh/pull/188#issuecomment-2583068813) on [issue #188](https://github.com/YunoHost-Apps/borg_ynh/pull/188) Strip work_dir from paths inside the backup archive: > @tio-trom have you modified the deployed file in hooks.d dir? No just the one in /etc/yunohost/apps/borg/conf - now...
[16:06:05] <Yunohost Git/Infra notifications> [apps] y​unohost-bot opened [pull request #2767](https://github.com/YunoHost/apps/pull/2767): Update app levels according to CI results
[16:11:29] <rodinux> well, here the log 20250108-084332-permission_url-cryptpad.log https://paste.yunohost.org/raw/mexuxoyimu there is nothing about this url...
[16:12:34] <rodinux> the only old conf could be from a old checksum in the settings.yml ? checksum__etc_nginx_conf.d_sandbox-cryptpad.linux07.fr.d_cryptpad.conf: 6390b68f85cdd0273d9e99be28ad21ea
[16:20:14] <rodinux> here the lo of the upgrade https://rodinux.fr/zerobin/?2cfaa428bc00a533#2HMJqcg2wqxuTuMvWt4oK6ASQVVmsV5jLha5w3QZuHK1
[16:29:26] <rodinux> perhaps from it comes from `2025-01-08 09:43:57,364: DEBUG - + '[' -f /var/cache/yunohost/app_tmp_work_dirs/app_zc0qrx86/conf/nginx-sandbox.conf ']'` ??
[16:39:50] <rodinux> nothing to do, this a file from the config
[18:35:25] <Yunohost Git/Infra notifications> e​ricgaspar created repository webdav_ynh https://github.com/YunoHost-Apps/webdav_ynh
[18:37:55] <Yunohost Git/Infra notifications> e​ricgaspar edited repository webdav_ynh: WebDAV package for YunoHost https://github.com/YunoHost-Apps/webdav_ynh
[18:37:56] <Yunohost Git/Infra notifications> e​ricgaspar edited repository webdav_ynh: WebDAV package for YunoHost https://github.com/YunoHost-Apps/webdav_ynh
[20:37:43] <Yunohost Git/Infra notifications> [apps] y​unohost-bot labeled Wishlist on [pull request #2768](https://github.com/YunoHost/apps/pull/2768): Add KindleEar to wishlist
[20:37:43] <Yunohost Git/Infra notifications> [apps] y​unohost-bot opened [pull request #2768](https://github.com/YunoHost/apps/pull/2768): Add KindleEar to wishlist
[21:20:53] <Yunohost Git/Infra notifications> [nextcloud_ynh] z​iima opened [issue #776](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/776): Logs in incorrect place
[21:21:15] <Yunohost Git/Infra notifications> [nextcloud_ynh] z​iima edited [issue #776](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/776): Logs in incorrect place
[22:27:00] <orhtej2> https://forum.yunohost.org/t/hedgedoc-a-collaborative-editor/13726/9 should I retract HedgeDoc 1.10.0~ynh2 so that people don't brick their installs?
[23:07:10] <Yunohost Git/Infra notifications> [joplin_ynh] y​unohost-bot opened [pull request #73](https://github.com/YunoHost-Apps/joplin_ynh/pull/73): Upgrade to v3.2.10
[23:11:33] <Yunohost Git/Infra notifications> [penpot_ynh] y​unohost-bot opened [pull request #82](https://github.com/YunoHost-Apps/penpot_ynh/pull/82): Upgrade to v2.4.1
[23:13:39] <Paprika> Created a [YunoRunner package](https://github.com/botagas/yunorunner_ynh) that allows one to use it within your own github repositories. Very useful when just starting to build a brand new package that may fail a lot of times (and installing within ynh might as well break it sometimes too), and manually testing it just when starting out may take just too much time. Still testing, but for the time being, loving it, especially since I plan on building another app that might prove difficult to do.
[23:17:41] <Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:

- 21 pending update PRs
- 7 new apps PRs
- 2 failed apps updates: flood, stremio

See the full log here: https://paste.yunohost.org/raw/yenenaqute