[00:22:24]
<Yunohost Git/Infra notifications> App jellyfin goes down from level 6 to 1 in job [#30577](https://ci-apps.yunohost.org/ci/job/30577)
[06:06:38]
<Yunohost Git/Infra notifications> [my_webapp_ynh] jcn50 opened [issue #144](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/144): Cannot install: error & operation failed
[06:06:47]
<Yunohost Git/Infra notifications> [my_webapp_ynh] jcn50 edited [issue #144](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/144): Cannot install: error & operation failed
[06:12:32]
<Yunohost Git/Infra notifications> [my_webapp_ynh] jcn50 edited [issue #144](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/144): Cannot install: error & operation failed
[08:10:24]
<orhtej2> Ci-dev is dead?
[11:04:08]
<m606> Hello, is the URL of an installed application available as a variable in the install script?
I can recreate the default with `$domain/$app`, but it would not work if the admin specified a custom URL
[11:16:45]
<m606> > <@m606:matrix.org> Hello, is the URL of an installed application available as a variable in the install script?
> I can recreate the default with `$domain/$app`, but it would not work if the admin specified a custom URL
`$domain/$path` I guess!
[11:17:04]
<m606> I will add default vars to the doc
[11:28:59]
<Aleks (he/him/il/lui)> `$domain` and `$path` are what you are looking for (these are install questions in the manifest)
[11:36:00]
<tituspijean> Technically it should be `$domain$path` otherwise you would get double slashes
[11:56:45]
<Yunohost Git/Infra notifications> App redirect stays at level 1 in job [#30437](https://ci-apps.yunohost.org/ci/job/30437)
[13:00:08]
<Aleks (he/him/il/lui)> Salamandar: wait what, we can do this to run a bash function as a user ?!?! https://github.com/YunoHost/appstore/blob/main/maintenance.sh#L66
[13:00:31]
<Aleks (he/him/il/lui)> ah yes i see
[13:00:41]
<Aleks (he/him/il/lui)> didn't see the wrapping `$()`
[13:05:00]
<Salamandar> uh yes but that's an incredibly ugly trick
[13:05:08]
<Salamandar> also doesn't capture environment (i mean local variables)
[13:05:38]
<Salamandar> TBH the only reason this script needs to be run as root is because the credentials for sendxmpppy are owned by root so we should change that and remove tricks
[13:05:59]
<Aleks (he/him/il/lui)> ah yeah
[13:06:11]
<Aleks (he/him/il/lui)> we could create some group like `infra-notifications` or whatever
[13:06:19]
<Salamandar> ah yes maybe
[13:55:53]
<Salamandar> > <@Salamandar:matrix.org> TBH the only reason this script needs to be run as root is because the credentials for sendxmpppy are owned by root so we should change that and remove tricks
ah no
[13:55:54]
<Salamandar> crap
[13:55:58]
<Salamandar> there's the systemctl restart too
[15:54:53]
<m606> I have a kind of dilemna
[15:59:44]
<m606> I need to import a JSON file in a JS file. the module's documentation (old) is telling to `import emojiData from 'emojibase-data/en/data.json';`
On recent FF & Chrome version, the ressource won't get loaded but will display a console error about the fact it was expecting a JS module and not JSON so it blocked it due to strict MIME type policy.
With `import emojiData from 'emojibase-data/en/data.json with { type: "json" }` it works as expected. But this relates to recent update in JS, and is [not yet widely browser-compatible](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/import#browser_compatibility) (basically it works on all recent version of browsers except FF due to a [bug not yet solved](https://bugzilla.mozilla.org/show_bug.cgi?id=1736059).
[16:00:51]
<m606> can I consider the YNH package using this as final (thinking that FF will eventually cope with that) ?
[16:00:52]
<Salamandar> not sure what you're asking from us, we aren't web devs ;)
[16:01:10]
<m606> see last line
[16:02:20]
<Salamandar> hmmm then tl;dr no, because generally yunohost users are privacy aware and don't want to use anything coming from Google
[16:02:44]
<Salamandar> and you don't want to handle more issues from users that don't understand why your app doesnt work
[16:07:25]
<m606> and what is the most striking issue here ? that FF is not compatible yet, that only recent browsers are compatible or both ?
[16:09:07]
<m606> the issue is that I have not found alternative solution so far (or it would take to modify a library used by the app, but that goes a bit too far in my opinion).
[16:10:08]
<m606> I can still wait that FF becomes compatible to release the YNH package )
[16:11:13]
<Salamandar> what's the app in question ?
[16:11:16]
<Aleks (he/him/il/lui)> not sure to understand how that's a yunohost-specific issue and not an upstream-app issue
[16:26:19]
<m606> > <@Alekswag:matrix.org> not sure to understand how that's a yunohost-specific issue and not an upstream-app issue
Technically it is an upstream issue (app developers or browser developers). However it deals with my efforts as YNH packager to remove all calls to CDN from the app and make external assets available locally. There wouldn't be such issue otherwise.
[16:28:20]
<Yunohost Git/Infra notifications> App cinny stays at level 1 in job [#30595](https://ci-apps.yunohost.org/ci/job/30595)
[16:28:41]
<m606> > <@Salamandar:matrix.org> what's the app in question ?
https://github.com/YunoHost-Apps/pinkarrows_ynh/tree/testing
[17:01:49]
<Yunohost Git/Infra notifications> App vaultwarden failed all tests in job [#30598](https://ci-apps.yunohost.org/ci/job/30598) :(
[17:08:01]
<Yunohost Git/Infra notifications> App ttrss rises from level 0 to 8 in job [#30597](https://ci-apps.yunohost.org/ci/job/30597) !
[19:35:59]
<m606> how does the install question of `type = "file"` work?
[19:36:49]
<m606> is this example relevant? https://github.com/YunoHost-Apps/eleventy_ynh/blob/f04f5afab811d3f9a4f0c5b141de82465f7da0d2/config_panel.toml.example#L244C1-L249C46
[19:38:37]
<tituspijean> yes
[19:46:57]
<m606> > <@titus:pijean.ovh> yes
actually i have a unique question to allow admins to set their own watermark picture. Ideally it would be asked at install (as a non-mandatory field) and could be changed anytime by the admin.
Should I create a config panel file ? add it under `[install]` in manifest.toml ? both ?
[19:57:20]
<m606> in manifest.toml
```
[install.watermark]
ask = "Watermark"
type = "file"
accept = ".png"
help = "Fill with your own watermark (when the box \"watermark\" is checked, it will be displayed on the bottom right of each picture saved in Pink Arrows)."
bind = "__INSTALL_DIR__/assets/watermark.png"
```
it outputs on YNH 12.0.3
```
While parsing manifest: 1 validation error for OptionsModel
options -> 3 -> FileOption -> accept
value is not a valid list (type=type_error.list)
```
[19:58:31]
<tituspijean> I think that would be a first to have an app to have a "file" question type in the installation form. I don't even known if that's even been tried, if that would work with the CI or the command line installations.
[19:59:09]
<tituspijean> well I think I've got my answer with your test
[19:59:39]
<tituspijean> I'd put it in the config panel.
[20:00:55]
<Aleks (he/him/il/lui)> `alue is not a valid list (type=type_error.list)` i.e. `accept` should be a list such as `['.png']` ?
[20:01:35]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/vpnclient_ynh/blob/master/config_panel.toml#L29
[20:01:58]
<m606> > <@Alekswag:matrix.org> `alue is not a valid list (type=type_error.list)` i.e. `accept` should be a list such as `['.png']` ?
let me test
[20:02:13]
<tituspijean> > <@Alekswag:matrix.org> https://github.com/YunoHost-Apps/vpnclient_ynh/blob/master/config_panel.toml#L29
(both are acceptable? https://github.com/search?q=org%3AYunoHost-Apps+accept++path%3Aconfig_panel.toml&type=code )
[20:02:43]
<tituspijean> (or maybe it has changed "recently"?)
[20:02:44]
<Aleks (he/him/il/lui)> those are all '.example' so not actually used
[20:02:53]
<Aleks (he/him/il/lui)> maybe it was changed with pydantinc in bookworm
[20:04:37]
<Aleks (he/him/il/lui)> on the other hand, having file arg during install is tricky as you said, in particular because argument value need to be serialized using `foo=foovalue&bar=barvalue` syntax so it's not clear what happens when it's a file here
[20:04:53]
<Aleks (he/him/il/lui)> theoretically, maybe it's converted to base64 but i'm not sure
[20:05:16]
<Aleks (he/him/il/lui)> actually maybe args are serialized in the same way for "config set" ?
[20:06:04]
<tituspijean> If it's converted in base64, wouldn't we hit the url length limit easily?
[20:06:15]
<Aleks (he/him/il/lui)> ah yea : https://github.com/YunoHost/yunohost/blob/dev/share/actionsmap.yml#L749
[20:06:18]
<tituspijean> (depends on the image size though)
[20:06:19]
<Aleks (he/him/il/lui)> hmmm
[20:06:22]
<Aleks (he/him/il/lui)> possibly
[20:06:43]
<Aleks (he/him/il/lui)> maybe it's only converted internally in moulinette ? i don't remember
[20:07:06]
<Aleks (he/him/il/lui)> `The maximum length of a URL in the address bar is 2048 characters.`
[20:07:15]
<Aleks (he/him/il/lui)> wut ? i thought it would be much bigger
[20:08:27]
<m606> > <@m606:matrix.org> let me test
```
[install.watermark]
ask = "Watermark"
type = "file"
accept = [".png"]
help = "Fill with your own watermark (when the box \"watermark\" is checked, it will be displayed on the bottom right of each picture saved in Pink Arrows)."
bind = "__INSTALL_DIR__/assets/watermark.png"
```
this in the manifest.toml does not show the previous error anymore and displays the install questions successfully. However,
1. watermark is a mandatory field
2. I tried several png files, but after clicking install, I get the error `Unsupported image type : image/png`
[20:08:58]
<m606> note that source includes a default watermark.png (but i don't think it can conflict at this point as the source has not been downloaded)
[20:09:10]
<tituspijean> `optional=true` would help for #1
[20:10:38]
<Aleks (he/him/il/lui)> anyway to me that sounds like an advanced question and should go into a regular config panel
[20:11:20]
<m606> > <@Alekswag:matrix.org> anyway to me that sounds like an advanced question and should go into a regular config panel
yes ok
[20:12:27]
<Aleks (he/him/il/lui)> also yes supposedly in bookworm you need to put mimetypes instead of extensions ? https://github.com/YunoHost-Apps/vpnclient_ynh/pull/121/files
[20:13:08]
<Nadine> Hi,
I'm attempting to package protonmail bridge and just transferred the repository from my private GitHub account to YunoHost-Apps org. However, now I cannot access settings anymore to make the repo publicly visible (for running tests etc.). Seems I'm locked out of my own repo now. 🙈😀 Anyone can grant me repo admin rights (GitHub user: CodeShakingSheep)? https://github.com/YunoHost-Apps/protonmail-bridge_ynh
[20:14:23]
<Aleks (he/him/il/lui)> switched you to admin on the repo
[20:17:39]
<m606> > <@titus:pijean.ovh> `optional=true` would help for #1
is this property meaningful in the config panel or everything there is optional ?
[20:19:43]
<tituspijean> you need to specify it if you want to allow an empty field
[20:34:30]
<m606> I removed the entry from manifest.toml and created a config_panel.toml.
```
version = "1.0"
[main]
name = "Pink Arrows settings"
[main.customization]
name = "Customization"
[main.customization.watermark]
ask = "Watermark"
type = "file"
accept = ".png"
help = "Fill with your own watermark (when the box \"watermark\" is checked, it will be displayed on the bottom right of each picture saved in Pink Arrows)."
bind = "__INSTALL_DIR__/assets/watermark.png"
```
I can install the app, and when checking the app settings (Applications => Pink Arrows), i see the following error :
```
Une erreur empêche l'affichage du panneau de configuration :
Erreur serveur inattendue
Merci de signaler cette erreur à l'équipe de YunoHost pour qu'elle soit corrigée !
```
Not sure if this is related, but the console shows 500 internal error when trying to reach https://{instance.tld}/yunohost/api/apps/pinkarrows/config?full&locale=fr
[20:34:54]
<m606> oh... same mistake [".png"]
[20:43:21]
<m606> So below is the new config_panel.toml.
```
version = "1.0"
[main]
name = "Pink Arrows settings"
[main.customization]
name = "Customization"
[main.customization.watermark]
ask = "Watermark"
type = "file"
accept = [".png"]
help = "Fill with your own watermark (when the box \"watermark\" is checked, it will be displayed on the bottom right of each picture saved in Pink Arrows)."
bind = "__INSTALL_DIR__/assets/watermark.png"
```
It displays without error. However, I can't upload any image - when clicking on "Save", i've got this error `Unsupported image type : image/png`.
Would `accept = ["image/png"]` help ?
[20:50:13]
<m606> well `accept = ["image/png"]` does not trigger any error when clicking on "Save", but it does seem to save anything:
- console shows a (successful) PUT request with `Status Code: 204 No Content`
- it did not overwrite the default watermark.png as expected
[21:01:27]
<m606> hmm i havn't changed anything, but for some reasons it now works! cache maybe!
[21:01:56]
<m606> so `accept` seems to take MIME types
[21:03:24]
<tituspijean> thanks for investigating :)
[21:04:03]
<m606> > <@m606:matrix.org> the issue is that I have not found alternative solution so far (or it would take to modify a library used by the app, but that goes a bit too far in my opinion).
btw I eventually found a working alternative there
[21:53:27]
<Yunohost Git/Infra notifications> [apps] oleole39 opened [pull request #2592](https://github.com/YunoHost/apps/pull/2592): Add app Pink Arrows
[21:54:26]
<Yunohost Git/Infra notifications> [apps] oleole39 edited [pull request #2592](https://github.com/YunoHost/apps/pull/2592): Add app Pink Arrows to catalog
[23:59:30]
<Yunohost Git/Infra notifications> App nomad stays at level 1 in job [#30605](https://ci-apps.yunohost.org/ci/job/30605)