Friday, October 21, 2022
apps@conference.yunohost.org
October
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
           

[00:09:06] <Yunohost Git/Infra notifications> [nextcloud_ynh] @itslino commented on issue #442 Changing of main yunohost domain doesnt seem to have been considered by nextcloud_ynh: yea, Im over here trying to figure out how to move to a domain I purchased but cant seem to find any guides. Should I ... https://github.com/YunoHost-Apps/nextcloud_ynh/issues/442#issuecomment-1286295735
[08:16:47] <Yunohost Git/Infra notifications> [wordpress_ynh] @github-actions[bot] pushed 1 commit to ci-auto-update-v6.0.3: Upgrade to version 6.0.3 https://github.com/YunoHost-Apps/wordpress_ynh/commit/7e44ae48ebcafa3eef704b38105d4880d845c2ff
[09:07:25] <Memo> Info: https://yunohost.org/ certificate has expired ❗️
[09:24:18] <Yunohost Git/Infra notifications> @ajoly-perso forked my_webapp_ynh to ajoly-perso/portfolio: https://github.com/ajoly-perso/portfolio
[11:49:13] <tituspijean> Memo: fixed
[14:47:07] <eric_G[m]> Are we still using `/opt/yunohost/$app` for node.js apps? -> https://github.com/YunoHost/example_ynh/blob/b41917736cbce3db3963f6467f97a9f524d3913b/scripts/install#L60
[15:04:56] <Guillaume Bouzige> Can helper `ynh_webpath_register` receive multiple `-d domain` parameter ?
[15:06:44] <Guillaume Bouzige> Does `ynh_webpath_register` can receive multiple domains parameter ? https://yunohost.org/en/packaging_apps_helpers#ynh-webpath-register
[15:09:05] <Guillaume Bouzige> the code doesnt look like it does tho https://github.com/YunoHost/yunohost/blob/933603ae1ecdcb2880c3e80dde2487f4175c5c1e/helpers/setting#L145
[16:20:06] <cmoi> Guillaume Bouzige, everything is behind nginx
[16:20:12] <cmoi> (i think)
[16:21:31] <Guillaume Bouzige> (that would make sense)
[16:59:55] <tituspijean> > Does `ynh_webpath_register` can receive multiple domains parameter ? https://yunohost.org/en/packaging_apps_helpers#ynh-webpath-register

IIRC no, right now one app = one domain. That would make sense to allow more, but it has to be changed in YunoHost core
[17:16:34] <Guillaume Bouzige> > Hello !
>
> still no clue about the error HTTP 403 from CI
> it fails upgrade and restore on that error
> ```
> > Validating that the app cryptpad can/can't be accessed with its URL...
>
> Error: The HTTP code shows an error.
> Warning: Test URL: sub.domain.tld
> Real URL: https://sub.domain.tld/
>
> HTTP code: 403
> Page title:
> Page extract:
> 403 Forbidden
> __________________________________________________________________
>
> nginx
> Error: The HTTP code shows an error.
> Warning: Test URL: sub.domain.tld/
> Real URL: https://sub.domain.tld/
>
> HTTP code: 403
> Page title:
> Page extract:
> 403 Forbidden
> __________________________________________________________________
>
> nginx
>
> --- FAIL ---
> ```
>
> anyone ?

the issue seems to be that `/var/www/$app` doesn't have group owner as www-data...I did add it in install script then forgot about it and so forgot to add it in restore / upgrade script. I thought that was quite standard and didnt think of having to do it manually...
[17:26:19] <tituspijean> One day™
[17:30:41] <Guillaume Bouzige> Tomorrow (cc)
[17:37:51] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 commented on issue #193 Upgrade to version 6.0.3: @tituspijean autoupgrader is broken https://github.com/YunoHost-Apps/wordpress_ynh/pull/193#issuecomment-1287255987
[17:40:34] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 created new branch Fix
[17:40:34] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 pushed 1 commit to Fix: Update check_process https://github.com/YunoHost-Apps/wordpress_ynh/commit/c40ab637da488a202235f73115aa76cb56a7f9da
[17:41:03] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 opened pull request #195: Update check_process https://github.com/YunoHost-Apps/wordpress_ynh/pull/195
[17:41:16] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 merged pull request #195: Update check_process https://github.com/YunoHost-Apps/wordpress_ynh/pull/195
[17:41:17] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 pushed 2 commits to master: https://github.com/YunoHost-Apps/wordpress_ynh/compare/7bec7d770026...1fbb8f5a36b9
[17:41:18] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 deleted branch Fix
[17:41:19] <Yunohost Git/Infra notifications> [wordpress_ynh/master] Update check_process - yalh76
[17:41:23] <Yunohost Git/Infra notifications> [wordpress_ynh/master] Merge pull request #195 from YunoHost-Apps/Fix Update check_process - yalh76
[17:41:50] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 pushed 5 commits to testing: https://github.com/YunoHost-Apps/wordpress_ynh/compare/fdb73af0642f...1fbb8f5a36b9
[17:41:54] <Yunohost Git/Infra notifications> [wordpress_ynh/testing] Auto-update README - yunohost-bot
[17:41:57] <Yunohost Git/Infra notifications> [wordpress_ynh/testing] Update check_process - yalh76
[17:42:05] <Yunohost Git/Infra notifications> [wordpress_ynh/testing] Merge pull request #195 from YunoHost-Apps/Fix Update check_process - yalh76
[18:01:26] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch modify_level
[18:01:26] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to modify_level: Update apps level https://github.com/YunoHost/apps/commit/7b2cfbd06c6ccc244edfb6d31917537bacab899c
[18:03:19] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean commented on issue #193 Upgrade to version 6.0.3: In what sense? https://github.com/YunoHost-Apps/wordpress_ynh/pull/193#issuecomment-1287280007
[18:05:30] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 commented on issue #193 Upgrade to version 6.0.3: [image](https://user-images.githubusercontent.com/30271971/197260563-93cd01cb-49bd-4098-8ad0-e00af7dae3f5.png) try to ... https://github.com/YunoHost-Apps/wordpress_ynh/pull/193#issuecomment-1287282014
[18:06:53] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 commented on issue #193 Upgrade to version 6.0.3: [image](https://user-images.githubusercontent.com/30271971/197260563-93cd01cb-49bd-4098-8ad0-e00af7dae3f5.png) try to ... https://github.com/YunoHost-Apps/wordpress_ynh/pull/193#issuecomment-1287282014
[18:07:09] <Yunohost Git/Infra notifications> [wordpress_ynh] @yalh76 commented on issue #193 Upgrade to version 6.0.3: [image](https://user-images.githubusercontent.com/30271971/197260563-93cd01cb-49bd-4098-8ad0-e00af7dae3f5.png) try to ... https://github.com/YunoHost-Apps/wordpress_ynh/pull/193#issuecomment-1287282014
[18:25:53] <Yunohost Git/Infra notifications> [apps] @ericgaspar opened pull request #1530: Update apps level https://github.com/YunoHost/apps/pull/1530
[18:27:43] <Yunohost Git/Infra notifications> [apps] @ericgaspar edited pull request #1530: Update apps level https://github.com/YunoHost/apps/pull/1530
[18:27:51] <Yunohost Git/Infra notifications> [apps] @ericgaspar edited pull request #1530: Update apps level https://github.com/YunoHost/apps/pull/1530
[19:57:59] <Yunohost Git/Infra notifications> [wordpress_ynh] @ericgaspar commented on issue #193 Upgrade to version 6.0.3: testme https://github.com/YunoHost-Apps/wordpress_ynh/pull/193#issuecomment-1287379743
[19:58:14] <Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot commented on issue #193 Upgrade to version 6.0.3: :carousel_horse:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/3925/bad... https://github.com/YunoHost-Apps/wordpress_ynh/pull/193#issuecomment-1287379774
[20:19:52] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean commented on issue #193 Upgrade to version 6.0.3: Branch name somehow ended up with a dollar sign, which gets interpreted as 6, which is empty.

I cannot look into it ... https://github.com/YunoHost-Apps/wordpress_ynh/pull/193#issuecomment-1287387303
[20:20:06] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean pushed 1 commit to ci-auto-update-v6.0.3: Upgrade to version 6.0.3 https://github.com/YunoHost-Apps/wordpress_ynh/commit/7e44ae48ebcafa3eef704b38105d4880d845c2ff
[20:20:09] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean deleted branch ci-auto-update-v6.0.3
[20:20:13] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean closed pull request #193: Upgrade to version 6.0.3 https://github.com/YunoHost-Apps/wordpress_ynh/pull/193
[20:20:14] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean created new branch ci-auto-update-v6.0.3
[20:20:20] <Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot pushed 1 commit to ci-auto-update-v6.0.3: Auto-update README https://github.com/YunoHost-Apps/wordpress_ynh/commit/6235b63248c765742a8c663fe8946082a549d0bf
[20:20:26] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean opened pull request #196: Upgrade to v6.0.3 https://github.com/YunoHost-Apps/wordpress_ynh/pull/196
[20:20:27] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean commented on issue #196 Upgrade to v6.0.3: testme https://github.com/YunoHost-Apps/wordpress_ynh/pull/196#issuecomment-1287393284
[20:20:28] <Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot commented on issue #196 Upgrade to v6.0.3: Meow :cat2:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/3926/badge)](... https://github.com/YunoHost-Apps/wordpress_ynh/pull/196#issuecomment-1287393309
[20:20:29] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean edited pull request #196: Upgrade to v6.0.3 https://github.com/YunoHost-Apps/wordpress_ynh/pull/196
[20:26:37] <Erik Kemp> I see the test is passed. So what is the next step in the process? I might get to know a bit about the Yunohost update process here :D
[20:31:25] <Yunohost Git/Infra notifications> [package_linter] @alexAubin pushed 1 commit to master: Update official helper list + ignore config panel helpers that may legitimately be overwritten https://github.com/YunoHost/package_linter/commit/76e8d962e189d635cb904bc2bd1c97ad6b2ba02e
[20:33:02] <Yunohost Git/Infra notifications> [wordpress_ynh] @ericgaspar pushed 6 commits to ci-auto-update-v6.0.3: https://github.com/YunoHost-Apps/wordpress_ynh/compare/6235b63248c7...61f089118ec7
[20:33:06] <Yunohost Git/Infra notifications> [wordpress_ynh/ci-auto-update-v6.0.3] Update check_process - yalh76
[20:33:08] <Yunohost Git/Infra notifications> [wordpress_ynh/ci-auto-update-v6.0.3] Merge pull request #195 from YunoHost-Apps/Fix Update check_process - yalh76
[20:33:11] <Yunohost Git/Infra notifications> [wordpress_ynh/ci-auto-update-v6.0.3] Merge branch testing into ci-auto-update-v6.0.3 - Éric Gaspar