[00:24:32]
<Yunohost Git/Infra notifications> App ergo goes down from level 6 to 3 in job [#26745](https://ci-apps.yunohost.org/ci/job/26745)
[00:40:10]
<Yunohost Git/Infra notifications> App drupal goes down from level 8 to 6 in job [#26744](https://ci-apps.yunohost.org/ci/job/26744)
[00:55:49]
<gaut> how to escape the special characters in a password forwarded to curl ? https://github.com/YunoHost-Apps/arn_messager_ynh/blob/init_dev/scripts/install#L109
[01:01:16]
<Aleks (he/him/il/lui)> it's a mess, especially if it's sent as json thingy ... what's `-g` ?
[01:02:52]
<Aleks (he/him/il/lui)> `This option switches off the "URL globbing parser". When you set this option, you can specify URLs that contain the letters {}[] without having curl itself interpret them. Note that these letters are not normal legal URL contents but they should be encoded according to the URI standard.`
wat
[01:04:03]
<Aleks (he/him/il/lui)> not sure to understand how the whole request works, this option doesn't seem to specify how to handle the next arg which seems to be some data payload...
[01:05:33]
<Yunohost Git/Infra notifications> App diaspora goes down from level 8 to 6 in job [#26743](https://ci-apps.yunohost.org/ci/job/26743)
[01:16:29]
<Yunohost Git/Infra notifications> App loki goes down from level 8 to 6 in job [#26747](https://ci-apps.yunohost.org/ci/job/26747)
[01:17:25]
<Aleks (he/him/il/lui)> gaut: supposedly you can json-escape a string with something like
```bash
PASSWORD='abcdef"*+-_[] "'
jq -Rr '@json' <<< "$PASSWORD"
ESCAPED_PASSWORD=$(jq -Rr '@json' <<< "$PASSWORD")
echo $ESCAPED_PASSWORD
```
returns `"abcdef\"*+-_[] \""` (note that the var itself contains the leading/trailing quotes
[01:45:29]
<Yunohost Git/Infra notifications> App pihole goes down from level 7 to 6 in job [#26749](https://ci-apps.yunohost.org/ci/job/26749)
[01:50:21]
<Yunohost Git/Infra notifications> [apps] imreallyliam [commented](https://github.com/YunoHost/apps/pull/3289#issuecomment-3741407277) on [issue #3289](https://github.com/YunoHost/apps/pull/3289) Add Baserow: Given that the system can already provide a redis and postgres database, I suppose it is easier to extract the individua...
[02:06:38]
<Yunohost Git/Infra notifications> App redlib stays broken (level 0) in job [#26759](https://ci-apps.yunohost.org/ci/job/26759)
[07:36:20]
<Yunohost Git/Infra notifications> App localsend rises from level 6 to 7 in job [#26766](https://ci-apps.yunohost.org/ci/job/26766) !
[07:52:48]
<Yunohost Git/Infra notifications> App wger goes down from level 7 to 6 in job [#26765](https://ci-apps.yunohost.org/ci/job/26765)
[08:40:44]
<gaut> I followed this idea, hence the `-g` but it still doesn't work https://stackoverflow.com/questions/8333920/passing-a-url-with-brackets-to-curl
[09:09:02]
<gaut> I need to escape braces, and possibly singles quotes too 😄
[09:14:19]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 merged [pull request #179](https://github.com/YunoHost-Apps/penpot_ynh/pull/179): Upgrade sources
[09:14:25]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 deleted branch ci-auto-update-sources-7c2d102d
[10:06:00]
<gaut> en fait j'ai l'impression que cette API est dépréciée, et il faudrait utiliser https://element-hq.github.io/synapse/latest/admin_api/user_admin_api.html#login-as-a-user ou peut-être https://element-hq.github.io/synapse/latest/jwt.html
[10:31:22]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] Thovi98 closed [pull request #113](https://github.com/YunoHost-Apps/stirling-pdf_ynh/pull/113): Upgrade to v2.2.0
[10:31:43]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] Thovi98 deleted branch ci-auto-update-2.2.0
[11:29:52]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 pushed to userdatas: create userdatas table before the migration ([c97d4a88](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/c97d4a886745bba71601a5e9d5f0390806681576))
[11:30:23]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 opened [pull request #211](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/211): create userdatas table before the migration
[11:56:47]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 pushed to userdatas: add sqlite deps for upgrade script to work ([70d205f4](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/70d205f44f691eac6a388e0a143926a789a3ed71))
[12:27:41]
<Aleks (he/him/il/lui)> but the URL doesn't contain brackets, only the payload ...?
[12:35:43]
<gaut> yes I don't get it but I changed my mind, the access_token will be generated at first service start :
https://github.com/YunoHost-Apps/arn_messager_ynh/blob/init_dev/scripts/install#L119
by https://code.ffdn.org/arn/arn-messager/-/blob/testing/arn-messager.py#L512
[12:38:44]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 pushed to userdatas: fix syntax ([52504183](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/5250418327651b3cdb404c3147136987c59cab6d))
[12:41:34]
<orhtej2> > <@miro5001:matrix.org> I fixed this repo
> https://github.com/YunoHost-Apps/yuno_goti_notify_ynh
> I thought that placeholders were automatically replaced in hooks.
> This sends notifications via a gotify server to any gotify client (I am using android). All you need is one gotify server with a defined token
This stuff is amazing ❤️
[12:55:09]
<miro5001> Glad you like it. I love it too 😁.
All my yunohost servers have it and I need only one gotify server
[13:26:24]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 pushed to userdatas: checks to see if the table has been created ([19ee080f](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/19ee080f35f93919d6f5781b5bd755d94b09d6ff))
[14:07:00]
<miro5001> `/etc/profile.d/gotify_ssh_login_yuno_goti_notify.sh` reports unexpected EOF while looking for matching `"'
I found an extra `" > ` that I has to delete manually
I also removed ynh_setup_source because everything is in the package
[14:19:53]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 pushed to userdatas: oops wrong db file 🙈 ([d0704a6c](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/d0704a6cb1ef5aff9620b04257dba5d2f4e10230))
[14:37:20]
<Yunohost Git/Infra notifications> App grafana rises from level 6 to 8 in job [#26773](https://ci-apps.yunohost.org/ci/job/26773) !
[14:58:15]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 pushed to userdatas: change db to library.db which is the culprit ([8a9d6268](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/8a9d62682e2508d47882f35a4b8de4e836ce96cb))
[15:27:33]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 pushed to ynh-structure: fix no table UserDatas ([7a7a7a58](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/7a7a7a58f103e2fc0f39503daf65fe594af10b21))
[15:27:34]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 pushed to ynh-structure: Merge branch ynh-structure of https://github.com/YunoHost-Apps/jellyfin_ynh into ynh-structure ([3fb44eaa](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/3fb44eaac4801321333493b58f438463e141d674))
[15:28:06]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 [commented](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/211#issuecomment-3744972272) on [issue #211](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/211) create userdatas table before the migration: @Salamandar I’ve yet to test but seems promising 🎉
[15:30:34]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 pushed to ynh-structure: typo ([16795a3a](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/16795a3ad9645c8523dc920bdc3892b4e640e7c7))
[15:43:42]
<orhtej2> > <@miro5001:matrix.org> `/etc/profile.d/gotify_ssh_login_yuno_goti_notify.sh` reports unexpected EOF while looking for matching `"'
> I found an extra `" > ` that I has to delete manually
> I also removed ynh_setup_source because everything is in the package
So did I, but reported the issues on gh just in case someone stumbles upon them in rhe future
[15:51:11]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 pushed to ynh-structure: add sqlite3 deps for migration ([7b486f5f](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/7b486f5fc60b42cdc6acc67a4c313332e7d780e9))
[15:53:37]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 [commented](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/211#issuecomment-3745099962) on [issue #211](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/211) create userdatas table before the migration: Just tested on a test server, works
[15:53:38]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 merged [pull request #211](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/211): create userdatas table before the migration
[15:53:39]
<Yunohost Git/Infra notifications> [jellyfin_ynh] Thovi98 deleted branch userdatas
[16:59:48]
<Yunohost Git/Infra notifications> [element_ynh] ericgaspar pushed to testing: Update manifest.toml ([eaf8d888](https://github.com/YunoHost-Apps/element_ynh/commit/eaf8d8884a8fa6eb8e74ea6fe3ea7d753d772639))
[17:00:14]
<Yunohost Git/Infra notifications> [element_ynh] ericgaspar opened [pull request #251](https://github.com/YunoHost-Apps/element_ynh/pull/251): Testing
[18:07:22]
<rodinux> Est-ce qu'il existe une variable `$YNH_APP_ID` ? je voudrai ajouter une condition dans le cas où une app est installée 2 fois...
[18:08:13]
<tituspijean[m]> > So did I, but reported the issues on gh just in case someone stumbles upon them in rhe future
Nice ! What about generalizing it with apprise (example with SSH https://github.com/YunoHost-Apps/ssh_apprise_ynh) ? 😇
[18:10:01]
<tituspijean[m]> (m606 which message are you answering to? 😅)
[18:11:13]
<m606> oh in Cinny there is the link of the (old) message I answered to as I clicked "Reply".
[18:11:25]
<m606> https://matrix.to/#/#yunohost-apps:matrix.org/$UxXIdgJ3AHm4jZVgpeB1F9SWptmtkq9WVuRxAH5-VmU
[18:11:57]
<m606> @miro5001:matrix.org that's interesting!
Thinking about a few next steps ideas:
1. One thing I would find useful in addition is displaying a nicely formattef changelog (sometimes located in the forge's releases section, on upstream official website, or if nothing is available upstream there should still be most often the forge's commit history (e.g. https://github.com/stan-smith/FossFLOW/compare/6cb649602c10e20ec59431341f774ff2f0264ee0...6b492e3afd0a6b9cac95ad519cb018b7359209f1) as sometimes is notified there breaking revamps (or to be interpreted as such), aside from the config file.
1. it could automatically makes commits in config file and scripts to update the settings (although it not be that easy to cover all specific cases).
1. to make it more convenient to deploy on on more apps, it could be using the automagical parsing of various types of config file as used in YNH core and fill app's specific variables at the top to avoid having to check all the script every time.
1. Maybe at some point it could be merged with this action https://github.com/oleole39/prebuild-app/blob/main/.github/workflows/ynh-build-on-upstream-update.yml which pre-builds (at the moment only NPM-based) packages on upstream update, although this relates to Packaging V3 and that no concrete decision has been taken yet in this regard AFAIK (here the app-specific variables are passed by a bash script located in `scripts/build` folder and maintained by the pacakger to the github workflow triggered by an action file which only acts as an orchestrator and can thus be deployed identically in every apps package requiring this feature (also so far used only in a handful of app packages).
[18:13:04]
<m606> is this new link working?
[18:33:04]
<miro5001> It already sends notifications on ssh connection
https://github.com/YunoHost-Apps/yuno_goti_notify_ynh/blob/testing/conf/gotify_ssh_login.sh
Didn't try apprise, so I don't know what it does more
[18:36:42]
<miro5001> Generalising could be interesting but I don't if giving it the right to commit is good. As you notice, I only gave read access to the repo and propose a PR so in case it requires more changes, it doesn't break the package
[19:21:38]
<miro5001> Issues fixed 😉
[19:21:52]
<tituspijean[m]> > <@miro5001:matrix.org> It already sends notifications on ssh connection
> https://github.com/YunoHost-Apps/yuno_goti_notify_ynh/blob/testing/conf/gotify_ssh_login.sh
> Didn't try apprise, so I don't know what it does more
Apprise would enable you to not force the dependency on Gotify. It allows sending out notifications on Gotify, ntfy, and plenty other notification systems.
[19:55:44]
<Yunohost Git/Infra notifications> [element_ynh] ericgaspar merged [pull request #251](https://github.com/YunoHost-Apps/element_ynh/pull/251): Testing
[20:04:36]
<Yunohost Git/Infra notifications> ewilly edited repository grafana_ynh: Grafana package for YunoHost https://github.com/YunoHost-Apps/grafana_ynh
[21:49:48]
<miro5001> It looks doable to add more notifications to it or probably as a separate package
[23:14:36]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] DeMiro5001 merged [pull request #114](https://github.com/YunoHost-Apps/stirling-pdf_ynh/pull/114): Upgrade to v2.2.1
[23:14:39]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] DeMiro5001 deleted branch ci-auto-update-2.2.1
[23:14:56]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] DeMiro5001 opened [pull request #115](https://github.com/YunoHost-Apps/stirling-pdf_ynh/pull/115): Testing
[23:15:07]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] DeMiro5001 merged [pull request #115](https://github.com/YunoHost-Apps/stirling-pdf_ynh/pull/115): Testing