[05:53:38]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 60 pending update PRs
- 12 new apps PRs
- 3 failed apps updates: fluffychat, mlmmj, mobilizon
See the full log here: http://paste.yunohost.org/raw/hiletobana
[06:03:14]
<Yunohost Git/Infra notifications> App strut stays at level 1 in job [#30794](https://ci-apps.yunohost.org/ci/job/30794)
[06:34:21]
<Yunohost Git/Infra notifications> App teampass goes down from level 8 to 6 in job [#30795](https://ci-apps.yunohost.org/ci/job/30795)
[07:48:37]
<Yunohost Git/Infra notifications> App torrelay failed all tests in job [#30796](https://ci-apps.yunohost.org/ci/job/30796) :(
[07:49:58]
<Yunohost Git/Infra notifications> App microblogpub goes down from level 7 to 6 in job [#30799](https://ci-apps.yunohost.org/ci/job/30799)
[07:50:02]
<Yunohost Git/Infra notifications> App scrutiny rises from level 6 to 8 in job [#30759](https://ci-apps.yunohost.org/ci/job/30759) !
[07:50:13]
<Yunohost Git/Infra notifications> Job [#30789](https://ci-apps.yunohost.org/ci/job/30789) for icecoder failed miserably :(
[07:51:53]
<@chrichri:ween.de> > <@chrichri:ween.de> Are there other http(s) apps (beside flohmarkt) that do not work correctly when nginx adds the yunohost code to their (javascript) pages?
https://codeberg.org/flohmarkt/flohmarkt_ynh/issues/71#issuecomment-2334712
[07:51:53]
<@chrichri:ween.de> Are there other http(s) apps (beside flohmarkt) that do not work correctly when nginx adds the yunohost code to their (javascript) pages?
[07:52:53]
<Thomas> Like here https://github.com/YunoHost-Apps/fittrackee_ynh/blob/373f17aa7537c10d4b443e3f18ccd5d47ab80714/manifest.toml#L96
[07:52:54]
<Thomas> You have to add auth_header = false
[08:17:50]
<Aleks (he/him/il/lui)> (Not sure the javascript inject is related to auth_header, but the good news is that there's no js inject in bookworm anymore)
[08:27:40]
<@chrichri:ween.de> > <@Alekswag:matrix.org> (Not sure the javascript inject is related to auth_header, but the good news is that there's no js inject in bookworm anymore)
That is really helpful information! I'll not put much thought into it and just disable the feature for now if I can't get around it. Thanks!
[08:31:02]
<nicofrand> Hi
[08:34:48]
<nicofrand> Where can I get details about the CI? What is really installed (os, container?)?
[08:36:20]
<nicofrand> Is there a way to detect the CI inside the apps packages?
[09:45:39]
<tituspijean> Hi, the CI is managed by https://github.com/YunoHost/yunorunner, OS is... YunoHost 🙃
[09:55:01]
<tituspijean> > Is there a way to detect the CI inside the apps packages?
yup! https://github.com/YunoHost-Apps/cryptpad_ynh/blob/ba4c2ca62877e4750b3f947830008d4985679494/scripts/remove#L44
[09:55:01]
<orhtej2> > Is there a way to detect the CI inside the apps packages?
Time for https://github.com/auchenberg/volkswagen ?
[09:55:01]
<tituspijean> > <@titus:pijean.ovh> Hi, the CI is managed by https://github.com/YunoHost/yunorunner, OS is... YunoHost 🙃
yunorunner handles lxd or incus, IIRC right now we are using incus.
[09:55:01]
<tituspijean> but do not abuse it :p
`if CI; then pass`
[09:55:02]
<nicofrand> > <@titus:pijean.ovh> but do not abuse it :p
> `if CI; then pass`
I might try catching errors due to the CI only, and not make the script fail if `update-locale` does not work
[09:55:49]
<nicofrand> That's either the VW way either fixing the CI 🤷
[10:37:04]
<@chrichri:ween.de> > <@Alekswag:matrix.org> (Not sure the javascript inject is related to auth_header, but the good news is that there's no js inject in bookworm anymore)
I put the yunohost code at the end of pages like this:
```
[...]
# Don't include SSOWAT user panel - breaks flohmarkt pages
# include conf.d/yunohost_panel.conf.inc;
# Instead try to put yunohost button + portal overlay at the end of body
sub_filter </body> '</body><script type="text/javascript" src="/ynh_portal.js"></script><link type="text/css" rel="stylesheet" href="/ynh_overlay.css"><script type="text/javascript" src="/ynhtheme/custom_portal.js"></script><link type="text/css" rel="stylesheet" href="/ynhtheme/custom_overlay.css">';
sub_filter_once on;
# Apply to other mime types than text/html
sub_filter_types application/xhtml+xml;
# Prevent YunoHost panel files from being blocked by specific app rules
location ~ (ynh_portal.js|ynh_overlay.css|ynh_userinfo.json|ynhtheme/custom_portal.js|ynhtheme/custom_overlay.css) {
```
That works. Is that o.k. to do for the package until 11.x is not used anymore?
[10:56:21]
<orhtej2> eric_G: can you have a look at https://github.com/YunoHost-Apps/nextcloud_ynh/pull/732 ?
[10:57:01]
<orhtej2> > <@Salamandar:matrix.org> https://github.com/yunohost/apps_tools or something
as in why are they not triggering ;P
[10:57:02]
<orhtej2> readme generating hooks where
[10:57:02]
<orhtej2> (also it should have spammed here that I created a PR in nextcloud_ynh, n'est-ce-pas?)
[10:57:03]
<Salamandar> ah lol
[10:57:03]
<Salamandar> https://github.com/yunohost/apps_tools or something
[11:05:27]
<orhtej2> hm, no dash between 'ce' and 'pas'
[11:56:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar approved [pull request #732](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/732#pullrequestreview-2327952866) Fixed backup
[11:56:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: Fixed backup (#732) * Update backup * Update manifest.toml ([e7c5a94e](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/e7c5a94e05774f4bfbc7aea6b2c80ad3cfc70317))
[11:56:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar merged [pull request #732](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/732): Fixed backup
[11:57:08]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot pushed 1 commit to testing: Auto-update READMEs ([77e3ed59](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/77e3ed59a46cd3c6d77c4842bc8831808fac81ed))
[12:01:31]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: cleaning ([28ac86d3](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/28ac86d30a0336a3e3d228e17347880420795ad1))
[12:01:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to testing: Update backup ([ab2df754](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/ab2df754a28c9ec7e8d8aec099c8665f6cd52859))
[12:01:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] orhtej2 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/733#issuecomment-2373812572) on [issue #733](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/733) Testing: testme
[12:01:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] orhtej2 approved [pull request #733](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/733#pullrequestreview-2327964377) Testing
[12:01:35]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u approved [pull request #733](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/733#pullrequestreview-2327959651) Testing
[12:02:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/733#issuecomment-2373812685) on [issue #733](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/733) Testing: Meow :cat2:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/3099...
[12:02:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/733#issuecomment-2373812739) on [issue #733](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/733) Testing: :rocket:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/19687/badge)](ht...
[12:49:40]
<Émy - OniriCorpe> https://aria.im/_matrix/media/v1/download/im.emelyne.eu/OEUycUtMNVLMJVnJWxpIjRQA
[12:50:57]
<Émy - OniriCorpe> https://aria.im/_matrix/media/v1/download/im.emelyne.eu/ZlgMjGALnekZwdIlQnNQVTuH
[12:55:01]
<Émy - OniriCorpe> ????? go home ynh ur drunk
[13:03:03]
<Émy - OniriCorpe> how can i fix this shit? 🙃
[14:22:11]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to master: Testing (#733) * Update manifest.toml * Auto-update READMEs * add pre upgrade warning for NC 29 * update to PHP8.3 ... ([bf8c0f95](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/bf8c0f95c7ed7cb7fa93069b60988b366b3f55f6))
[14:22:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar merged [pull request #733](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/733): Testing
[14:23:48]
<kayou> Salamandar: can I have your opinion on https://github.com/YunoHost-Apps/nextcloud_ynh/pull/722 and https://github.com/YunoHost-Apps/borgserver_ynh/pull/74# pls?
[14:24:34]
<kayou> when you have time to review it ofc
[14:25:33]
<Salamandar> let me see
[14:27:01]
<Salamandar> hmmm
[14:29:23]
<Salamandar> in post_user_create, the setfacl recursive wouldn't be even worse than the current situation ?
[14:30:01]
<Salamandar> also for borg, idk, if you tested it yeah alright, i don't have much opinion about acl
[14:30:47]
<Salamandar> here i'm not sure your'e applying them on restore though
[14:31:12]
<tituspijean> > <@oniricorpe:im.emelyne.eu> how can i fix this shit? 🙃
I think it's the same way you declared other variables? Like in https://github.com/YunoHost-Apps/gotosocial_ynh/blob/cb64af866832d3e290e03df052e2acd8ff74e622/scripts/install#L41 ?
[14:45:47]
<Émy - OniriCorpe> > <@titus:pijean.ovh> I think it's the same way you declared other variables? Like in https://github.com/YunoHost-Apps/gotosocial_ynh/blob/cb64af866832d3e290e03df052e2acd8ff74e622/scripts/install#L41 ?
afaik it’s done properly…
On this branch btw https://github.com/YunoHost-Apps/gotosocial_ynh/tree/v0.17.0-rc1
[15:04:43]
*tituspijean discovers that github searches only on the main branch :/
[15:06:00]
<tituspijean> erf yeah indeed, it's in there, so yeah, wtf
[19:13:23]
<Yunohost Git/Infra notifications> [apps] ericgaspar opened [pull request #2597](https://github.com/YunoHost/apps/pull/2597): Add dodoc to catalog
[19:14:10]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to Add-dodoc: Update wishlist.toml ([d40a77c2](https://github.com/YunoHost/apps/commit/d40a77c25e1764a59da25208eb79e9ff9737060c))
[19:14:12]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to Add-dodoc: Create dodoc.png ([5fb2fc63](https://github.com/YunoHost/apps/commit/5fb2fc6336ca7a3f753554dee321756d90b3b2b4))
[19:23:30]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to add-convos-catalog: add convos to the catalog ([16b89c81](https://github.com/YunoHost/apps/commit/16b89c81d8725c0b1b2772c8591fbb2fd44679d6))
[19:23:30]
<Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch add-convos-catalog
[19:24:34]
<Yunohost Git/Infra notifications> [apps] OniriCorpe labeled ready to merge on [pull request #2598](https://github.com/YunoHost/apps/pull/2598): add convos to the catalog
[19:24:35]
<Yunohost Git/Infra notifications> [apps] OniriCorpe labeled catalog on [pull request #2598](https://github.com/YunoHost/apps/pull/2598): add convos to the catalog
[19:24:35]
<Yunohost Git/Infra notifications> [apps] OniriCorpe opened [pull request #2598](https://github.com/YunoHost/apps/pull/2598): add convos to the catalog
[19:35:49]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2597](https://github.com/YunoHost/apps/pull/2597): Add dodoc to catalog
[19:35:49]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch Add-dodoc
[19:35:50]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Add dodoc to catalog (#2597) * Update apps.toml * Create dodoc.png * Update wishlist.toml ([9ce47a57](https://github.com/YunoHost/apps/commit/9ce47a574594d4da1fe3b4ddd06577824c253e08))
[19:35:55]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to master: Automatically add dates to the catalog files ([8c025490](https://github.com/YunoHost/apps/commit/8c0254902edb01012b6b4e98870b19cc19b5cb97))
[19:36:35]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2598](https://github.com/YunoHost/apps/pull/2598#pullrequestreview-2329377492) add convos to the catalog
[19:36:59]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Add ospreadsheet to wishlist (#2596) ([1e6edc39](https://github.com/YunoHost/apps/commit/1e6edc39f89909698558634570eceff7ac7971e9))
[19:37:03]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch add-to-wishlist-ospreadsheet
[19:37:03]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2596](https://github.com/YunoHost/apps/pull/2596): Add ospreadsheet to wishlist
[19:37:25]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2595](https://github.com/YunoHost/apps/pull/2595#pullrequestreview-2329379269) Add logo for fail2ban-web
[19:37:25]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to master: Automatically add dates to the catalog files ([39b52f62](https://github.com/YunoHost/apps/commit/39b52f627694c0c0d5ff41b4b0dac6387c347ba3))
[19:37:42]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2595](https://github.com/YunoHost/apps/pull/2595): Add logo for fail2ban-web
[19:37:42]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Add logo for fail2ban-web (#2595) ([3c0c4b01](https://github.com/YunoHost/apps/commit/3c0c4b01158fbe33af7a22e10389186cc4198935))
[19:39:11]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to update_app_levels: Update apps.toml ([23293ccb](https://github.com/YunoHost/apps/commit/23293ccb48f512e8aab02f760ced1c74a1536b31))
[19:40:40]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2590](https://github.com/YunoHost/apps/pull/2590#pullrequestreview-2329385287) Update app levels according to CI results
[19:41:07]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2590](https://github.com/YunoHost/apps/pull/2590): Update app levels according to CI results
[19:41:08]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Update app levels according to CI results (#2590) * Update app levels according to CI results * :art: Format TOML with... ([b1c34206](https://github.com/YunoHost/apps/commit/b1c342063673b3f71683f46bf432eff8d2cde0d1))
[19:41:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch update_app_levels
[20:01:20]
<eric_G> I am intrigued by this test https://ci-apps-dev.yunohost.org/ci/job/19699
[20:02:44]
<eric_G> is this the IP of the our server?
[20:02:53]
<eric_G> especially by this line: `Sep 25 19:16:26 node\[3369\]: address: '0.89.170.196',`
[20:04:56]
<tituspijean> Nope. Very weird indeed. Especially starting with 0?
[20:07:55]
<tituspijean> Maybe that's the address of the VM in the pool, but still, I don't think that's a valid IP.
[20:22:34]
<gredin67> Salamandar: I tested your PR https://github.com/YunoHost-Apps/fab-manager_ynh/pull/126
but the app is not able to send email
https://github.com/YunoHost-Apps/fab-manager_ynh/issues/131#issuecomment-2307207454
[20:22:41]
<gredin67> do someone have a clue ?