[00:08:56]
<Yunohost Git/Infra notifications> App aeneria goes down from level 7 to 6 in job [#12378](https://ci-apps.yunohost.org/ci/job/12378)
[00:11:49]
<Yunohost Git/Infra notifications> App digiwords goes down from level 8 to 6 in job [#12380](https://ci-apps.yunohost.org/ci/job/12380)
[00:12:05]
<Yunohost Git/Infra notifications> App bicbucstriim goes down from level 7 to 6 in job [#12379](https://ci-apps.yunohost.org/ci/job/12379)
[00:24:04]
<Yunohost Git/Infra notifications> App slingcode goes down from level 7 to 1 in job [#12383](https://ci-apps.yunohost.org/ci/job/12383)
[00:33:26]
<Yunohost Git/Infra notifications> App tyto goes down from level 7 to 6 in job [#12384](https://ci-apps.yunohost.org/ci/job/12384)
[00:33:27]
<Yunohost Git/Infra notifications> App peertube goes down from level 8 to 6 in job [#12381](https://ci-apps.yunohost.org/ci/job/12381)
[00:43:25]
<Yunohost Git/Infra notifications> App nullboard goes down from level 8 to 6 in job [#12387](https://ci-apps.yunohost.org/ci/job/12387)
[00:48:15]
<Yunohost Git/Infra notifications> App watchdog goes down from level 7 to 6 in job [#12389](https://ci-apps.yunohost.org/ci/job/12389)
[00:48:18]
<Yunohost Git/Infra notifications> App kanboard goes down from level 8 to 6 in job [#12386](https://ci-apps.yunohost.org/ci/job/12386)
[01:09:40]
<Yunohost Git/Infra notifications> App mitra goes down from level 7 to 6 in job [#12390](https://ci-apps.yunohost.org/ci/job/12390)
[05:52:35]
<Yunohost Git/Infra notifications> [nextcloud_ynh] utzer [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2505313378) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: For me the update failed but Nextcloud was still working. Otherwise I would have lost some hours to restore (in worst ca...
[07:18:57]
<Yunohost Git/Infra notifications> App mobilizon rises from level 6 to 8 in job [#12392](https://ci-apps.yunohost.org/ci/job/12392) !
[08:13:14]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to helpers_2.1-3.0.1: Update upgrade ([a70607db](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/a70607dbb5c20cf1b001bef2475ca96927ba9e3c))
[08:36:02]
<Yunohost Git/Infra notifications> App slingcode goes down from level 7 to 6 in job [#12383](https://ci-apps.yunohost.org/ci/job/12383)
[10:05:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] stemy2 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2505722343) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: Im unable to even install nextcloud because of this.
[10:39:30]
<orhtej2> > that change to linter now downgrades apps that were at lvl8 and required no maintenance (hence readme was not regenerated). should we autopatch all 500+ apps?
Aleks (he/him/il/lui): I was thinking about it some more and autopatch is actually required as the linter rule that fails actually asserts the lvl badge is displayed correctly and there was a change to URLs backing these up, cf https://github.com/YunoHost/package_linter/commit/5dfd1c6758fe80915474b0609b9db1e455ae794b
[10:40:25]
<Aleks (he/him/il/lui)> we can just adjust the and/or condition to check wether the old url or the new url is there x_x
[10:41:44]
<orhtej2> > <@Alekswag:matrix.org> we can just adjust the and/or condition to check wether the old url or the new url is there x_x
but the old URL is gone, points to 404
[10:42:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2505804425) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: try the 30.0.2 version sudo yunohost app install https://github.com/YunoHost-Apps/nextcloud_ynh/tree/testing
[10:42:33]
<Aleks (he/him/il/lui)> meh idk it's less a huge deal to have broken READMEs badges for a while than suddenly downgrading every apps to level 6 just for this
[10:43:09]
<orhtej2> Salamandar @Salamandar:matrix.org: wdyt?
[10:43:41]
<Salamandar> hmmm what ?
[10:45:06]
<Salamandar> ah
[10:45:19]
<Salamandar> maybe we can add back the test in the linter that accepts legacy readmes
[11:08:16]
<orhtej2> regarding helpers 2.1: there's [this issue](https://github.com/YunoHost-Apps/kavita_ynh/issues/54) reported that indicates (rightfully so?) that `ynh_setup_sources` runs as `root` and tries to do WTF stuff. Perhaps it should run as `$app` by default? Or, alternatively, it should pass this `--no-whatever-something-about-user` to avoid restoring random ownership?
[11:11:34]
<Aleks (he/him/il/lui)> hmmmyeah i suppose we could try running every command in ynh_setup_source as the app user, most of the time we'll do a chmod/chown right after anyway
[11:41:18]
<orhtej2> > pycurl.error: (6, 'Could not resolve host: assets')
the f?
[11:42:23]
<Aleks (he/him/il/lui)> jesus christ it's again the stuff about curl test "base" ...
[11:42:25]
<Aleks (he/him/il/lui)> zzzzz
[12:11:23]
<Yunohost Git/Infra notifications> [nextcloud_ynh] stemy2 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2505974869) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: > try the 30.0.2 version sudo yunohost app install https://github.com/YunoHost-Apps/nextcloud_ynh/tree/testing It fai...
[16:51:29]
<m606> Hello, I referred to https://github.com/YunoHost/issues/issues/2348 but CI is having a yellow complain about my license code:
> ! The license id 'LicenseRef-json-crack-license' is not registered in https://spdx.org/licenses/.
Upstream license is a custom one which basically says "free for non-commercial use": https://github.com/AykutSarac/jsoncrack.com/?tab=License-1-ov-file#readme
[16:51:52]
<m606> Should i ignore the warning or change the license code to something else ?
[16:53:47]
<Aleks (he/him/il/lui)> th license code should correspond to the real license of the software, imho we should change the code of the linter because the current way we do this check (if that code didnt change since then) is just a search in the entire HTML (or XML) content and it's bullshit (or there's some stuff like "some license listed are actually not free as approved by GNU or the OSI) and there are code not listed which should be "OK" for the linter
[16:53:49]
<Aleks (he/him/il/lui)> so imho we should hardcode a list of code in the linter and voila
[16:59:20]
<m606> by HTML/XML you mean it searches the code of that page https://spdx.org/licenses/ ? Do you suggest hardcoding a shorter list ? And if so, how would custom license such as the one of my current case be treated (would we have to add it to the linter code, would it be refused, or would it be hardcoded as a regex like `LicenseRef-.*`) ?
so for now I can ignore the warning and publish the app like this, right ?
[16:59:52]
<Aleks (he/him/il/lui)> imho it would be reviewed and accepted by the team on a case-by-case basis
[17:00:20]
<Aleks (he/him/il/lui)> > I can ignore the warning and publish the app like this, right ?
i guess but isnt this reported as a major error and not just a warning?
[17:01:08]
<m606> > <@Alekswag:matrix.org> imho it would be reviewed and accepted by the team on a case-by-case basis
Yellow font means warning I guess ?
[17:01:17]
<Aleks (he/him/il/lui)> indeed
[17:08:04]
<m606> it makes sense to me.
Although of course these kinds of "one developer's license" are a bit annoying because to be rigorously on the safe side it would require making sure periodically that license was not edited since it was approved by the team. It could require a diff watch tool, i.e. for each new license to be approved, we may need name, TL;DR points, code and source link, with regular comparison of source link content which would ping the team when it changes.
[17:08:39]
<m606> it makes sense to me.
Although of course these kinds of "one developer's license" are a bit annoying because to be rigorously on the safe side it would require making sure periodically that license was not edited since it was approved by the team. It could require a diff watch tool, i.e. for each new license to be approved, we may need name, TL;DR points, codename, and source link, with regular comparison of source link content which would ping the team when it changes.
[17:09:09]
<m606> it makes sense to me.
Although of course these kinds of "one developer's license" are a bit annoying because to be rigorously on the safe side it would require making sure periodically that license was not edited since it was approved by the team. It could require a diff watch tool, i.e. for each new license to be approved, we may need full name, TL;DR points, codename, and source link, with regular comparison of source link content which would ping the team when it changes.
[17:12:33]
<Yunohost Git/Infra notifications> [nextcloud_ynh] drewwalton19216801 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2506536315) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: > try the 30.0.2 version sudo yunohost app install https://github.com/YunoHost-Apps/nextcloud_ynh/tree/testing Does n...
[17:12:36]
<Yunohost Git/Infra notifications> [nextcloud_ynh] drewwalton19216801 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2506536315) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: > try the 30.0.2 version sudo yunohost app install https://github.com/YunoHost-Apps/nextcloud_ynh/tree/testing Does n...
[17:21:45]
<Émy – OniriCorpe> > <@m606:matrix.org> Hello, I referred to https://github.com/YunoHost/issues/issues/2348 but CI is having a yellow complain about my license code:
> > ! The license id 'LicenseRef-json-crack-license' is not registered in https://spdx.org/licenses/.
>
> Upstream license is a custom one which basically says "free for non-commercial use": https://github.com/AykutSarac/jsoncrack.com/?tab=License-1-ov-file#readme
Make sure to tag the app with the “not-totally-free-upstream” in the ynh catalog
[17:22:05]
<m606> I have also a linter error regarding the nginx file vulnerable to path traversal issues
```
location __PATH__/widget {
# Path to source
alias __INSTALL_DIR__/;
index widget.html;
# Include SSOWAT user panel.
include conf.d/yunohost_panel.conf.inc;
}
```
So I understand I should change to `location __PATH__/widget/` (with trailing slash).
But then as I need the absence of trailing slash to be redirected to the same point, I could add `#sub_path_only rewrite ^__PATH__/widget$ __PATH__/widget/ permanent;` But what if YNH admin decides to install it on root ? and if remove `#sub_path_only` would would `__PATH__` be looking to when installed on root ?
[17:24:24]
<Aleks (he/him/il/lui)> if you mean `__PATH__` is going to be replaced with `/` and that's going to end up in `//` then don't think about this, helpers handle this for you
[17:27:28]
<m606> ok so this should be fine in all cases (root/subpath) ?
```
rewrite ^__PATH__/widget$ __PATH__/widget/ permanent;
location __PATH__/widget/ {
# Path to source
alias __INSTALL_DIR__/;
index widget.html;
# Include SSOWAT user panel.
include conf.d/yunohost_panel.conf.inc;
}
```
[17:29:08]
<Aleks (he/him/il/lui)> yes
[18:03:48]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2725](https://github.com/YunoHost/apps/pull/2725): Add InvoiceShelf to wishlist
[18:03:49]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2725](https://github.com/YunoHost/apps/pull/2725): Add InvoiceShelf to wishlist
[18:04:13]
<Yunohost Git/Infra notifications> [apps] OniriCorpe [commented](https://github.com/YunoHost/apps/pull/2725#issuecomment-2506603372) on [issue #2725](https://github.com/YunoHost/apps/pull/2725) Add InvoiceShelf to wishlist: It’s simpler than invoice ninja and all
[18:05:03]
<Yunohost Git/Infra notifications> [apps] OniriCorpe edited [pull request #2725](https://github.com/YunoHost/apps/pull/2725): Add InvoiceShelf to wishlist
[18:07:57]
<Yunohost Git/Infra notifications> [apps] OniriCorpe [commented](https://github.com/YunoHost/apps/pull/2725#issuecomment-2506607441) on [issue #2725](https://github.com/YunoHost/apps/pull/2725) Add InvoiceShelf to wishlist: Installation docs: https://docs.invoiceshelf.com/install/manual.html
[18:35:53]
<orhtej2> > > pycurl.error: (6, 'Could not resolve host: assets')
>
> the f?
```
$ python3 -c "from urllib.parse import urljoin; print(urljoin(\"http://example.com\", \"//assets\"))"
http://assets
```
python plz
[18:37:58]
<Émy – OniriCorpe> > ```
> $ python3 -c "from urllib.parse import urljoin; print(urljoin(\"http://example.com\", \"//assets\"))"
> http://assets
> ```
>
> python plz
Why are you escaping quotes? 🤔
[18:41:28]
<orhtej2> > <@oniricorpe:im.emelyne.eu> Why are you escaping quotes? 🤔
Because I'm bad at it and also that's not the point 😜
[18:48:09]
<Yunohost Git/Infra notifications> ericgaspar created repository invoiceshelf_ynh https://github.com/YunoHost-Apps/invoiceshelf_ynh
[18:50:48]
<Yunohost Git/Infra notifications> ericgaspar edited repository invoiceshelf_ynh: InvoiceShelf package for YunoHost https://github.com/YunoHost-Apps/invoiceshelf_ynh
[18:50:49]
<Yunohost Git/Infra notifications> ericgaspar edited repository invoiceshelf_ynh: InvoiceShelf package for YunoHost https://github.com/YunoHost-Apps/invoiceshelf_ynh
[18:50:49]
<Yunohost Git/Infra notifications> ericgaspar edited repository invoiceshelf_ynh: InvoiceShelf package for YunoHost https://github.com/YunoHost-Apps/invoiceshelf_ynh
[18:53:04]
<Émy – OniriCorpe> :’3
I was planning to do it by myself but thanks
[18:53:07]
<Aleks (he/him/il/lui)> > <@yunohostinfra:matrix.org> ericgaspar created repository invoiceshelf_ynh https://github.com/YunoHost-Apps/invoiceshelf_ynh
Dude xD
[19:02:41]
<Émy – OniriCorpe> [@ericg:matrix.org](https://matrix.to/#/@ericg:matrix.org) Doesn't postgresql take priority over mysql if the app supports both?
[19:23:07]
<eric_G> of course... when the app works
[19:35:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2506686382) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: @drewwalton19216801 you should complain to Nextcloud forum about it. The version 29.0.9 has been removed from they repo ...
[19:35:41]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2506686382) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: @drewwalton19216801 if you want to complain, you should complain to Nextcloud forum about it. The version 29.0.9 has bee...
[20:01:42]
<Yunohost Git/Infra notifications> App jenkins rises from level 6 to 8 in job [#12397](https://ci-apps.yunohost.org/ci/job/12397) !
[20:43:07]
<Yunohost Git/Infra notifications> [package_check] orhtej2 opened [pull request #185](https://github.com/YunoHost/package_check/pull/185): Use proper URI parsing for asset tests.
[20:47:05]
<Yunohost Git/Infra notifications> [package_check] orhtej2 edited [pull request #185](https://github.com/YunoHost/package_check/pull/185): Use proper URI parsing for asset tests.
[20:47:27]
<Yunohost Git/Infra notifications> [package_check] orhtej2 edited [pull request #185](https://github.com/YunoHost/package_check/pull/185): Use proper URI parsing for asset tests.
[21:05:36]
<Yunohost Git/Infra notifications> [package_check] alexAubin merged [pull request #185](https://github.com/YunoHost/package_check/pull/185): Use proper URI parsing for asset tests.
[21:05:57]
<Yunohost Git/Infra notifications> [package_check] github-actions[bot] opened [pull request #186](https://github.com/YunoHost/package_check/pull/186): Format Python code with Black
[21:06:48]
<Yunohost Git/Infra notifications> [package_check] alexAubin merged [pull request #186](https://github.com/YunoHost/package_check/pull/186): Format Python code with Black
[21:06:50]
<Yunohost Git/Infra notifications> [package_check] alexAubin deleted branch actions/black
[21:27:16]
<orhtej2> aaaand I broke Grist again x_x
[21:53:19]
<Yunohost Git/Infra notifications> [package_check] orhtej2 opened [pull request #187](https://github.com/YunoHost/package_check/pull/187): Fixed Grist again as it uses non-/ base tag.
[21:54:22]
<Yunohost Git/Infra notifications> [package_check] orhtej2 edited [pull request #187](https://github.com/YunoHost/package_check/pull/187): Fixed Grist again as it uses non-/ base tag.
[21:55:51]
<Yunohost Git/Infra notifications> [nextcloud_ynh] stemy2 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2506795860) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: Do you mean that the YNH team is totally powerless about this problem and all we have to do is hoping that nextcloud doe...
[22:09:46]
<Yunohost Git/Infra notifications> [nextcloud_ynh] drewwalton19216801 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756#issuecomment-2506804019) on [issue #756](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/756) Download of upload file fails for nextcloud-29.0.9.tar.bz2: > @drewwalton19216801 if you want to complain, you should complain to Nextcloud forum about it. The version 29.0.9 has b...
[23:22:55]
<Yunohost Git/Infra notifications> [apps] oleole39 opened [pull request #2726](https://github.com/YunoHost/apps/pull/2726): add jsoncrack to catalog