[00:38:44]
<Yunohost Git/Infra notifications> [my_webapp_ynh] OniriCorpe approved [pull request #143](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/143#pullrequestreview-2274499072) [Doc] Add PHP 8.3 in available versions
[00:41:07]
<Yunohost Git/Infra notifications> [my_webapp_ynh] OniriCorpe merged [pull request #143](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/143): [Doc] Add PHP 8.3 in available versions
[00:41:53]
<Yunohost Git/Infra notifications> [my_webapp_ynh] OniriCorpe pushed 1 commit to testing: [Doc] Add PHP 8.3 in available versions (#143) ([0819fc4b](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/0819fc4b17dd257653a5fce79f0fba86ed955094))
[00:43:25]
<Yunohost Git/Infra notifications> [my_webapp_ynh] yunohost-bot pushed 1 commit to testing: Auto-update READMEs ([d068a861](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/d068a8612e80a0c44ecd2fe9bd17f6f1875fd29d))
[02:00:15]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 25 pending update PRs
- 24 new apps PRs
- 6 failed apps updates: anarchism, epicyon, fluffychat, mlmmj, mobilizon, pleroma
See the full log here: http://paste.yunohost.org/raw/wuqesuyowe
[02:02:19]
<Yunohost Git/Infra notifications> App grr goes down from level 8 to 1 in job [#30158](https://ci-apps.yunohost.org/ci/job/30158)
[03:30:51]
<Yunohost Git/Infra notifications> App rspamdui goes down from level 3 to 1 in job [#30162](https://ci-apps.yunohost.org/ci/job/30162)
[03:51:57]
<Yunohost Git/Infra notifications> App tvheadend failed all tests in job [#30166](https://ci-apps.yunohost.org/ci/job/30166) :(
[03:57:47]
<Yunohost Git/Infra notifications> App wireguard failed all tests in job [#30167](https://ci-apps.yunohost.org/ci/job/30167) :(
[09:22:08]
<Yunohost Git/Infra notifications> [apps] Salamandar pushed 1 commit to logos_check: Add logos_check.py to validate logos filesize/resolution ([db8f844d](https://github.com/YunoHost/apps/commit/db8f844d57f25434e5475f9a9141eaf9845aaee7))
[09:22:09]
<Yunohost Git/Infra notifications> [apps] Salamandar created new branch logos_check
[09:23:14]
<Yunohost Git/Infra notifications> [apps] Salamandar opened [pull request #2553](https://github.com/YunoHost/apps/pull/2553): Add logos_check.py to validate logos filesize/resolution
[09:25:40]
<Yunohost Git/Infra notifications> [apps] selfhoster1312 [commented](https://github.com/YunoHost/apps/pull/2553#discussion_r1740602527) on pull request #2553 Add logos_check.py to validate logos filesize/resolution: Explicitly listing the steps is clearer :)
[09:27:01]
<Yunohost Git/Infra notifications> [apps] selfhoster1312 [commented](https://github.com/YunoHost/apps/pull/2553#discussion_r1740604468) on pull request #2553 Add logos_check.py to validate logos filesize/resolution: Should we accept all those formats? Many software dont support web yet. And JPG doesnt allow background transparency w...
[09:28:04]
<Yunohost Git/Infra notifications> [apps] selfhoster1312 [commented](https://github.com/YunoHost/apps/pull/2553#discussion_r1740605968) on pull request #2553 Add logos_check.py to validate logos filesize/resolution: Could we maybe hardcode those constants at the top of the file for easier changes?
[09:28:15]
<Yunohost Git/Infra notifications> [apps] selfhoster1312 [commented](https://github.com/YunoHost/apps/pull/2553#discussion_r1740606174) on pull request #2553 Add logos_check.py to validate logos filesize/resolution: Could we maybe hardcode this constant at the top of the file for easier changes?
[09:41:50]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to fix-logo-margins: Update immich.png ([2c0d65e5](https://github.com/YunoHost/apps/commit/2c0d65e511ffea348709d59734908740e74300df))
[09:41:54]
<Yunohost Git/Infra notifications> [apps] ericgaspar created new branch fix-logo-margins
[09:42:15]
<Yunohost Git/Infra notifications> [apps] ericgaspar opened [pull request #2554](https://github.com/YunoHost/apps/pull/2554): Fix immich.png margins
[09:43:43]
<selfhoster1312> also maybe that's stupid but is github the best place to upload images?
[09:43:57]
<selfhoster1312> i mean since the repo contains the entire history it will just grow forever (it's already like 35MiB u_u)
[10:48:24]
<Salamandar> > <selfhoster1312> also maybe that's stupid but is github the best place to upload images?
IMHO logos should be in the packages repos, but appart from that, yeah, git is fine
[11:49:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to Add-screego-to-catalog: Update wishlist.toml ([2fa3a0fe](https://github.com/YunoHost/apps/commit/2fa3a0fe2420ac678e7c516ce2a7a64dd010a838))
[11:49:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar created new branch Add-screego-to-catalog
[11:49:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to Add-screego-to-catalog: Update apps.toml ([74fd1d33](https://github.com/YunoHost/apps/commit/74fd1d33b70bd4df1263e9155940ddaccb21764a))
[11:53:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to Add-screego-to-catalog: Update apps.toml ([16e63df5](https://github.com/YunoHost/apps/commit/16e63df53bb5208af410645daf87b6b50164e93d))
[11:53:33]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Update immich.png (#2554) ([9cae1620](https://github.com/YunoHost/apps/commit/9cae162014feba1cf696fe958997d203bfde9f64))
[11:53:34]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2554](https://github.com/YunoHost/apps/pull/2554): Fix immich.png margins
[11:53:46]
<Yunohost Git/Infra notifications> [apps] ericgaspar closed [pull request #2552](https://github.com/YunoHost/apps/pull/2552): Add Redlib to wishlist
[11:53:51]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch fix-logo-margins
[12:33:27]
<Salamandar> @ericg:matrix.org yo so fast
[12:58:37]
<eric_G> `We regret to inform you that an unexpected cooling issue at our Nuremberg Data Center is affecting several services:` oh my god! what have I done 😱
[12:59:02]
<selfhoster1312> you're not cool :(
[12:59:27]
<Aleks (he/him/il/lui)> wut ? xD
[13:00:01]
<Yunohost Git/Infra notifications> [my_webapp_ynh] utkarshsethi [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/100#issuecomment-2324705879) on [issue #100](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/100) uploading hugo static website doesnt work: @lucyferu can you confirm that you only copied the content inside public or www folder generated by hugo and not the...
[13:00:08]
<selfhoster1312> ^^"
[13:00:11]
<Aleks (he/him/il/lui)> uhoh
[13:00:18]
<eric_G> I have a VPS in that Data center
[13:00:23]
<Aleks (he/him/il/lui)> was this right after you tried intalling jellyfin ?
[13:01:02]
<Aleks (he/him/il/lui)> or did you ... divide by empty string ?
[13:01:12]
<eric_G> cal.com 😅
[13:03:00]
<eric_G> I wonder if `unexpected cooling issue` is not an alternative way to describe a data center being on fire...
[13:04:41]
<selfhoster1312> euphemism of the day
[13:07:17]
<Yunohost Git/Infra notifications> [my_webapp_ynh] utkarshsethi [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/100#issuecomment-2324705879) on [issue #100](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/100) uploading hugo static website doesnt work: @lucyferu can you confirm that you only copied the content inside public or www folder generated by hugo and not the...
[13:08:35]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> I wonder if `unexpected cooling issue` is not an alternative way to describe a data center being on fire...
naaah, this is fine 🍺 ┬─┬
[13:35:04]
<Yunohost Git/Infra notifications> [my_webapp_ynh] utkarshsethi [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/89#issuecomment-2324778382) on [issue #89](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/89) paths like /post/example-post should serve example-post.html: Right now Im achieving this on my own my_webapp by: Creating the folder posts/hello-world and adding index.html ...
[14:04:26]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch Add-screego-to-catalog
[14:04:31]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2555](https://github.com/YunoHost/apps/pull/2555): Add screego to catalog
[14:04:37]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Add screego to catalog (#2555) * Update apps.toml * Update wishlist.toml * Create screego.png * Update apps.toml ([c00696f0](https://github.com/YunoHost/apps/commit/c00696f04cbab90e940a6450956d895f4d09bcba))
[14:04:52]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to master: Automatically add dates to the catalog files ([35828298](https://github.com/YunoHost/apps/commit/35828298788d979ea6e99793891c07cb0f50e5c3))
[15:31:36]
<thatoo> Hello,
In the manifest.toml, I first indicated "wkhtmltopdf" in packages = [ ] but now I discover that I should not install from debian repo but directly from github repo because it is "(with patched qt".
How can I tell the manifest to install https://github.com/wkhtmltopdf/packaging/releases/download/0.12.6.1-3/wkhtmltox_0.12.6.1-3.bookworm_amd64.deb ?
[15:32:22]
<Aleks (he/him/il/lui)> i guess by adding a `[resource.sources]`
[15:32:46]
<Aleks (he/him/il/lui)> (but then you have to `dpkg -i` install it, it won't be magically extracted / installed or whatevr
[15:33:12]
<thatoo> ok, I get it
[15:33:15]
<thatoo> thanks
[15:54:29]
<anubister> Hello, I provided a custumized version of Prosody (to extend functionality by default), but with the same app name it leads to upgrade the "official" app. What would be your advice to keep this 2 apps in parallel?
1/ Should I always a *big* version number (no risk of unexpected upgrade from "official" to "customized" ?)
2/ Rename my app (it might break other XMPP app if they request prosody or metronome?) ? Any recommandations? prosody_plus?
[15:54:53]
<selfhoster1312> anubister, why not make a PR to the prosody_ynh package?
[15:54:56]
<selfhoster1312> what did you change? :)
[15:56:58]
<anubister> this it quite heavy modifications, not sure it's wanted : brings coturn and default configuration that might not be wanted by other users? https://github.com/anubister/prosody_ynh
[15:59:14]
<anubister> but maybe a branch on the official app would be an option ?
[16:00:13]
<anubister> (does Yunohost check for updates only in the master branch?)
[16:01:23]
<Aleks (he/him/il/lui)> why would other users not want this, are there specific overhead about coturn or controversy about the configuration you want to change ?
[16:09:29]
<selfhoster1312> sooooo there's a lot to see there but it's not heavy modifications
[16:09:56]
<selfhoster1312> first you should definitely use lua 5.4 it's much better than lua5.2
[16:10:29]
<selfhoster1312> then there's a few things about coturn like if it's a separate app, why would prosody_ynh declare the ports in its own manifest?
[16:11:50]
<selfhoster1312> then there's the actual XMPP settings, i recommend taking a look at snikket.org it's based on prosody (actually developed by prosody devs) and has the best supports for everything, so you can just copy modules/settings from there ^^
[16:12:04]
<selfhoster1312> but overall it should be upstreamed in prosody_ynh for sure !!
[17:01:17]
<anubister> thanks for your feedbacks; the problem is maybe more for the coturn app then; I add to force the ports, which might not be OK for other uses? But if it is fine for you then i'll prepare the PR (first coturn, then prosody)
[17:03:05]
<anubister> ah, and probably my app *does not* work on Yunohost 11, so it is probably too early ?
[17:14:53]
<Aleks (he/him/il/lui)> YunoHost 12 is ugh "around the corner", for some definitions of "corner" 😬
[17:15:32]
<Aleks (he/him/il/lui)> and anyway prosody on Yunohost 11 is probably a mess because you metronome is too much integrate in the core, but it's "removed" in Yunohost so makes things simpler
[17:33:53]
<anubister> 😅️ OK I'll prepare that (also realistically speaking let say in ~2 weeks)
[18:02:46]
<Salamandar> > <@thatoo:defis.info> Hello,
> In the `` manifest.toml ``, I first indicated "wkhtmltopdf" in `` packages = [ ] `` but now I discover that I should not install from debian repo but directly from github repo because it is "(with patched qt)".
> How can I tell the manifest to install https://github.com/wkhtmltopdf/packaging/releases/download/0.12.6.1-3/wkhtmltox_0.12.6.1-3.bookworm_amd64.deb ?
waitwaitwait
[18:03:02]
<Salamandar> that reminds something
[18:03:08]
<Salamandar> isn't this qt patch already provided in the deb packaqe ?
[18:04:25]
<Salamandar> libreerp_ynh used this "patched qt" trick but we went back to the repository one
[18:04:49]
<thatoo> the qt patch is provided in the deb package I pointed, yes, but not in the debian repository package
[18:05:49]
<thatoo> maybe librerp doesn't need this qt patch
[18:06:43]
<Salamandar> i can't find the source of the patch :D
[18:07:08]
<Salamandar> ok
[18:07:08]
<Salamandar> ah no it's a full repo
[18:07:16]
<Aleks (he/him/il/lui)> (i think it does, I had to deal with it a few months ago, I was sort of optimist i could get around with the .deb and had to revert to the damn specific version they need)
[18:07:44]
<Aleks (he/him/il/lui)> like there's a bug when generating docs with header/footer, something along these lines
[18:07:55]
<Salamandar> erf
[18:11:24]
<thatoo> > <@Alekswag:matrix.org> like there's a bug when generating docs with header/footer, something along these lines
exactly
[18:11:40]
<Aleks (he/him/il/lui)> #JustOdooThings
[18:12:21]
<thatoo> Well Dokos isn't nicer on that matter then :-)
[18:12:33]
<Aleks (he/him/il/lui)> all computers are broken
[18:13:01]
<Salamandar> complete different subject but wkhtml2pdf isn't supported anymore :D
[18:13:35]
<thatoo> the repo has been marked as archived last month yes
[18:14:30]
<Aleks (he/him/il/lui)> bold of you to assume anybody developing or using odoo cares 🙃
[20:55:04]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2556](https://github.com/YunoHost/apps/pull/2556): Translations update from Weblate for readme-generator
[20:57:05]
<Yunohost Git/Infra notifications> [apps] OniriCorpe approved [pull request #2556](https://github.com/YunoHost/apps/pull/2556#pullrequestreview-2276147646) Translations update from Weblate for readme-generator
[20:57:12]
<Yunohost Git/Infra notifications> [apps] OniriCorpe pushed 1 commit to master: Translated readme-generator using Weblate (Spanish) Currently translated at 100.0% (39 of 39 strings) Translation: Yun... ([71de100e](https://github.com/YunoHost/apps/commit/71de100e93641e8fa728d96e968b2b56443ee9d8))
[20:57:17]
<Yunohost Git/Infra notifications> [apps] OniriCorpe merged [pull request #2556](https://github.com/YunoHost/apps/pull/2556): Translations update from Weblate for readme-generator
[21:27:38]
<PapaDragon> Humpf. Getting a 403 error after merging the automatic patch for helpers 2.1... Well I mean while trying to fix the code. When running the package check (locally) I get this for the first install step:
URL : https://sub.domain.tld/
Code : 403
Title : 403 Forbidden
Content extract:
403 Forbidden
nginx
Errors :
- Got return code 403, but was expecting 200
[21:28:07]
<PapaDragon> Here's a more detailed log : https://paste.yunohost.org/raw/xufayizife
[21:30:01]
<PapaDragon> As the error i get is
/var/www/streams/index.php" is forbidden (13: Permission denied), client: 10.110.62.1, server: sub.domain.tld, request: "GET / HTTP/2.0", host: "sub.domain.tld
I added an ls to the install script which gives me this: rwxr-x--- 1 streams www-data 1389 Aug 26 21:00 /var/www/streams/index.php
[21:30:54]
<PapaDragon> i don't know what I'm missing here, so if anyone has an idea...
[21:37:15]
<Aleks (he/him/il/lui)> can you check `ls -ld /var/www/streams` ?
[21:37:31]
<Aleks (he/him/il/lui)> or `namei -l /var/www/streams/index.php`
[21:38:55]
<Aleks (he/him/il/lui)> it feels like it should have `groups = "www-data"` for the install dir : https://github.com/YunoHost-Apps/streams_ynh/blob/master/manifest.toml#L53
[21:39:23]
<Aleks (he/him/il/lui)> i'm guessing `www-data` ain't the group-owner on /var/www/streams and therefore doesn't have the "x" perm
[21:41:28]
<PapaDragon> Well, the autopatch added this : group = "www-data:r-x"
[21:44:25]
<PapaDragon> Well, ls -ld /var/www/streams => drwxr-x--- 27 streams streams 4096 Sep 2 21:41 /var/www/streams
[21:44:46]
<PapaDragon> So yeah, there's the problem.
[21:45:19]
<Aleks (he/him/il/lui)> ah yes i wasnt looking at the testing branch
[21:45:36]
<Aleks (he/him/il/lui)> are you sure you're testing the testing branch ?
[21:45:54]
<Aleks (he/him/il/lui)> (well uh yeah otherwise you would get the 403)
[21:46:04]
<Aleks (he/him/il/lui)> hmmm
[21:46:39]
<PapaDragon> Actually I'm working locally on another branch, because the autopatch broke a few things (remaining brackets of "if").
[21:47:01]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/streams_ynh/blob/testing/scripts/install#L47
[21:47:13]
<Aleks (he/him/il/lui)> probably git clone resets the install dir permissions ?
[21:47:42]
<Aleks (he/him/il/lui)> (and git clone shouldnt be used because it has no guarantee to fetch the same commit every time ...
[21:48:31]
<PapaDragon> Yeah, I'm removing this also. The install script I'm testing is using a more valid ynh_setup_source --dest_dir="$install_dir"
[21:49:01]
<PapaDragon> Should I push the branch I'm working on on GitHub?
[21:49:15]
<PapaDragon> It's reaaaaaaaally a WIP
[21:50:03]
<Aleks (he/him/il/lui)> yes the testing branch or PR are made for this
[22:02:53]
<PapaDragon> Ok, I updated the testing branch
[22:03:38]
<PapaDragon> So right now, the step 2/7 install works but ends up with the 403 error...
[22:32:29]
<PapaDragon> Ok found it, ynh_config_add_logrotate does a chown $app:$app on the folder where the logfile is. And here it was in /var/www/$app. Moved it to /var/log/$app and the 403 error is gone
[22:33:03]
<PapaDragon> Now there are some other errors, but I should be fine. Baby steps.
[22:37:21]
<PapaDragon> Last question though. Since the atopatch added stuff like
#REMOVEME? Everything about fpm_footprint is removed in helpers2.1... | fpm_footprint="medium"
Should I remove stuff like ynh_config_add_phpfpm from the scripts?
[23:20:30]
<PapaDragon> > Last question though. Since the atopatch added stuff like
> #REMOVEME? Everything about fpm_footprint is removed in helpers2.1... | fpm_footprint="medium"
>
> Should I remove stuff like ynh_config_add_phpfpm from the scripts?
OK, so definitely not 🙂