Monday, October 09, 2023
apps@conference.yunohost.org
October
Mon Tue Wed Thu Fri Sat Sun
            1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
         

[01:44:17] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-immich: Add Immich to wishlist ([ec6dff27](https://github.com/YunoHost/apps/commit/ec6dff2737090787f2f3ec1e1f6449bf8feec22f))
[01:44:17] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch add-to-wishlist-immich
[01:44:19] <Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1795](https://github.com/YunoHost/apps/pull/1795): Add Immich to wishlist
[03:53:15] <Yunohost Git/Infra notifications> [apps] @tituspijean [commented](https://github.com/YunoHost/apps/pull/1795#issuecomment-1752311376) on [issue #1795](https://github.com/YunoHost/apps/pull/1795) Add Immich to wishlist: A repository has already been created, but not much has been done so far.

https://github.com/YunoHost-Apps/immich_ynh
[07:45:57] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2: Update nginx.conf ([b46b5319](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/b46b53195ef4d2aff11a97931af5b36f234d6039))
[07:46:20] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to v2: Update nginx.conf ([44764988](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/4476498872a89b9006ba53e3ab84f5f32329a593))
[07:46:36] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/596#issuecomment-1752496136) on [issue #596](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/596) V2: testme
[07:46:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/596#issuecomment-1752496177) on [issue #596](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/596) V2: May the CI gods be with you
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/...
[07:47:37] <Yunohost Git/Infra notifications> [apps] @ericgaspar closed [pull request #1795](https://github.com/YunoHost/apps/pull/1795): Add Immich to wishlist
[08:15:08] <Yunohost Git/Infra notifications> [my_webapp_ynh] @stilobique [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/129#discussion_r1349984462) on pull request #129 Testing: True, error from my PR, I dont remember exactly the upgrade mechanism, but its an extra config, my memory say "*above ...
[09:48:22] <limezy> Hi all ! May I ask for some help about that installation failure of the Outline app ? https://forum.yunohost.org/t/outline-an-open-extensible-wiki-for-your-team/17197/197 ? Full log here : https://paste.yunohost.org/raw/kivogibuwe
I have no clue of what happened...
Thanks in advance !
[09:54:48] <orhtej2> > <@limezy:matrix.org> Hi all ! May I ask for some help about that installation failure of the Outline app ? https://forum.yunohost.org/t/outline-an-open-extensible-wiki-for-your-team/17197/197 ? Full log here : https://paste.yunohost.org/raw/kivogibuwe
> I have no clue of what happened...
> Thanks in advance !

> Loaded configuration file "server/config/database.json".
[09:54:50] <orhtej2> but why?
[09:55:12] <orhtej2> anyways given you run leave-broken version can you open that file and make sure the password is OK?
[09:55:33] <eric_G> limezy: you can use `__DB_PWD__` instead of `secret_key` here https://github.com/YunoHost-Apps/outline_ynh/blob/e46c07af6cbe29276eb43017c874765ca9bc7fc4/conf/.env#L13
[09:55:42] <orhtej2> as in same asgiven in `sudo yunohost setting outline db_pwd`
[09:56:01] <limezy> > anyways given you run leave-broken version can you open that file and make sure the password is OK?

It's not me, another user of the forum requesting for help. I'm the packager of that app. But I can indeed ask
[09:56:02] <orhtej2> or what eric_G said
[09:58:28] <limezy> > <@ericg:matrix.org> limezy: you can use `__DB_PWD__` instead of `secret_key` here https://github.com/YunoHost-Apps/outline_ynh/blob/e46c07af6cbe29276eb43017c874765ca9bc7fc4/conf/.env#L13

I guess it will work because of https://github.com/YunoHost-Apps/outline_ynh/blob/e46c07af6cbe29276eb43017c874765ca9bc7fc4/scripts/install#L149 ?
[09:58:33] <limezy> But what is the difference ?
[09:59:01] <limezy> Was there a recent change on how db password is managed by Yunohost ?
[10:00:55] <eric_G> I don't know if this will fix the issue but it is always good to stick to the template
[10:01:02] <limezy> OK
[10:03:12] <Yunohost Git/Infra notifications> [nextcloud_ynh] @supermamie opened [issue #608](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/608): chown -R nextcloud: /var/www/nextcloud /home/yunohost.app/nextcloud/data done twice on upgrade
[10:03:24] <limezy> Is there a command line I could ask the user to run to get the "secret_key" from his (now broken) Outline app ? To compare it with the key on the /var/www/outline/.env file ?
[10:05:33] <limezy> Thanks for your help and time... I'm a bit lost there !
[10:09:36] <orhtej2> after reading some [through upstream](https://github.com/outline/outline/blob/594898affc769829f4db95e29e4bf3ec3e9cc254/server/config/database.json#L10)

I think the solution is changing [this line](https://github.com/YunoHost-Apps/outline_ynh/blob/e46c07af6cbe29276eb43017c874765ca9bc7fc4/scripts/upgrade#L175) to read

```
ynh_exec_warn_less sudo -u $app env $ynh_node_load_PATH NODE_ENV=production yarn db:migrate
```
[10:09:55] <orhtej2> so you know, set `NODE_ENV` to production to use conf from `.env`
[10:11:35] <limezy> Wow that's a very good catch
[10:12:13] <limezy> I don't understand though why the upgrade did work both on my two servers and on the CI
[10:13:15] <orhtej2> checking theory here: https://github.com/YunoHost-Apps/outline_ynh/pull/84
[10:13:28] <orhtej2> > <@limezy:matrix.org> I don't understand though why the upgrade did work both on my two servers and on the CI

that IS concerning
[10:14:09] <orhtej2> ahh that's because dev differs only by accepting borked SSL
[10:14:41] <limezy> > ahh that's because dev differs only by accepting borked SSL

You are speaking chinese for me but I'm glad you seem to have understood
[10:15:12] <orhtej2> https://aria.im/_matrix/media/v1/download/circledsquareroot.ovh/3076232c015e01a0afeb46ab0691468eb9ab53ae85e5735a551ef688db19e7ac
[10:15:43] <orhtej2> > <@limezy:matrix.org> You are speaking chinese for me but I'm glad you seem to have understood

basically my idea is wrong, still no clue why it did not work
[10:17:28] <limezy> Why didn't you just add `--env=production-ssl-disabled` in the db migration line as advised by the upstream dev ?
[10:20:47] <orhtej2> > <@limezy:matrix.org> Why didn't you just add `--env=production-ssl-disabled` in the db migration line as advised by the upstream dev ?

the error says password is wrong, I somehow doubt this has anything to do with SSL certificate. What's more currently these seem to run in dev mode (`Using environment "development".`) so it does not verify certificates anyway
[10:21:07] <orhtej2> I have sinking feeling `.env` file is broken and/or not used but not sure
[10:22:25] <limezy> I think I'll ask the user to DM me his .env file after removing the sensitive info from it
[10:23:07] <limezy> > <@limezy:matrix.org> Is there a command line I could ask the user to run to get the "secret_key" from his (now broken) Outline app ? To compare it with the key on the /var/www/outline/.env file ?

If you could help me on that this could also be very helpful
[10:25:12] <orhtej2> > <@limezy:matrix.org> If you could help me on that this could also be very helpful

`sudo yunohost app setting outline secret_key`
[10:25:44] <orhtej2> and see if it matches the one in `DATABASE_URL` in `.env`
[10:28:12] <limezy> And also DATABASE_URL should match with SECRET_KEY in the .env. I have a feeling this is where the problem lies !
[10:29:07] <limezy> > `sudo yunohost app setting outline secret_key`

Many thanks, do you think this will work even with the app in broken state ?
[10:29:26] <orhtej2> > <@limezy:matrix.org> Many thanks, do you think this will work even with the app in broken state ?

I guess so
[10:36:38] <limezy> https://forum.yunohost.org/t/outline-an-open-extensible-wiki-for-your-team/17197/199 OK let's see where this goes. I'll also change from SECRET_KEY to DB_PWD in the scripts. Many thanks for your help !
[10:38:04] <limezy> > <@ericg:matrix.org> I don't know if this will fix the issue but it is always good to stick to the template

But is the __DB_PWD__ variable always populated ?
[11:38:54] <orhtej2> Is is possible to use custom URL for checking if app installed properly?
[11:39:05] <orhtej2> rather than `$domain$full_path`
[11:40:24] <Aleks (he/him/il/lui)> nope but that would be nice
[11:40:41] <Aleks (he/him/il/lui)> could be integrated in the `tests.toml` and in the test logic
[11:41:08] <Aleks (he/him/il/lui)> could also explicit what page title is expected, or specific keyword expected in the body
[11:41:44] <Aleks (he/him/il/lui)> and if we want to go one step further, even check that some basic feature do work like login on the app etc ...
[11:43:34] <orhtej2> looks like #work :P
[11:44:45] <Aleks (he/him/il/lui)> 😅
[11:55:25] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-peppermint: Add Peppermint to wishlist ([6c79bd6c](https://github.com/YunoHost/apps/commit/6c79bd6c1288066fef8449279d9ef3486bd980a9))
[11:55:26] <Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1796](https://github.com/YunoHost/apps/pull/1796): Add Peppermint to wishlist
[11:55:26] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch add-to-wishlist-peppermint
[12:19:17] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch add-to-wishlist-anthias
[12:19:17] <Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1797](https://github.com/YunoHost/apps/pull/1797): Add Anthias to wishlist
[12:19:17] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-anthias: Add Anthias to wishlist ([fb19364c](https://github.com/YunoHost/apps/commit/fb19364c101572d25f9a3a2927709102a60b73f6))
[12:28:53] <Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to add-to-wishlist-anthias: Update wishlist.toml: we dont care that its "The worlds most popular open source " ([296e7449](https://github.com/YunoHost/apps/commit/296e7449d86d8aeb8e1748872c4f69323682e848))
[12:29:55] <Aleks (he/him/il/lui)> really like ... did they even bother read the note in the form ~_~
[13:09:46] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/596#discussion_r1350288502) on pull request #596 V2: cf a question today on the support chat, we may want to enable email in the future, to properly configure email : [n...
[13:16:47] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar created new branch set_mail
[13:19:50] <Yunohost Git/Infra notifications> [apps] @tituspijean approved [pull request #1790](https://github.com/YunoHost/apps/pull/1790#pullrequestreview-1664438525) Add PairDrop to catalog
[13:20:02] <Yunohost Git/Infra notifications> [apps] @tituspijean merged [pull request #1790](https://github.com/YunoHost/apps/pull/1790): Add PairDrop to catalog
[13:20:03] <Yunohost Git/Infra notifications> [apps] @tituspijean pushed 4 commits to master ([236ffbb29d5e...42af6fed11a5](https://github.com/YunoHost/apps/compare/236ffbb29d5e...42af6fed11a5))
[13:20:08] <Yunohost Git/Infra notifications> [apps/master] Create pairdrop.png - Éric Gaspar
[13:20:16] <Yunohost Git/Infra notifications> [apps/master] Update wishlist.toml - Éric Gaspar
[13:20:21] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1790 from YunoHost/pairdrop Add PairDrop to catalog - tituspijean
[13:22:39] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to set_mail: add mail setup ([ee03b250](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/ee03b250708c92c0e0e3756278be341942270ff4))
[13:24:22] <eric_G> ^ not sure this is the way.
[13:30:01] <Aleks (he/him/il/lui)> to be confirmed but naively sounds legit to me
[13:30:41] <Aleks (he/him/il/lui)> hmmm maybe `@localhost` or `@127.0.0.1` instead of `@__DOMAIN__` ?
[13:31:55] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to set_mail: Update config.json ([f2e491a7](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/f2e491a7a4946b03d2dc315d3ddb783b2397d33a))
[13:37:45] <Tag> > <@Alekswag:matrix.org> hmmm maybe `@localhost` or `@127.0.0.1` instead of `@__DOMAIN__` ?

also `__DOMAIN__` can be better beause the certificate will be valid
[13:38:13] <eric_G> ...and it is not working
[13:38:15] <Aleks (he/him/il/lui)> ah, didnt think about that
[13:38:39] <Aleks (he/him/il/lui)> but uuuh we shouldnt really need a certificate ? It's supposed to be internal communication between programs
[13:39:09] <Aleks (he/him/il/lui)> so naively I would disable TLS
[13:44:43] <eric_G> or we could also use Sendmail, more simple to set 😬
[13:45:58] <orhtej2> > <@ericg:matrix.org> ...and it is not working

Does it show `::1` ss rejected address in log?
[13:47:16] <orhtej2> Cf https://github.com/YunoHost/yunohost/pull/1710
[13:47:49] <Aleks (he/him/il/lui)> ah it's probably not yet released
[13:48:03] <Aleks (he/him/il/lui)> but was planning to hopefully fix the openssl issue today and make a release with all the pending stuff
[13:52:29] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to set_mail: Update config.json ([711050a2](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/711050a22e340652160b1c9120224b4992e52710))
[13:57:48] <Aleks (he/him/il/lui)> > <@yunohostinfra:matrix.org> [nextcloud_ynh] @ericgaspar pushed 1 commit to set_mail: Update config.json ([711050a2](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/711050a22e340652160b1c9120224b4992e52710))

eeer but you kinda need authentication though, with username/password, otherwise mails are not DKIM-signed by postfix/dovecot, and therefore likely to be flagged as spam
[13:58:18] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to set_mail: Revert "Update config.json" This reverts commit 711050a22e340652160b1c9120224b4992e52710. ([cae89b06](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/cae89b069c9266966767910f5e2f805ef03e74ff))
[13:58:51] <orhtej2> > <@yunohostinfra:matrix.org> [nextcloud_ynh] @ericgaspar pushed 1 commit to set_mail: Update config.json ([711050a2](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/711050a22e340652160b1c9120224b4992e52710))

does this work for you? In my testing SMTP on port 25 was expecting secure connection anyway.
[13:59:53] <eric_G> > does this work for you? In my testing SMTP on port 25 was expecting secure connection anyway.

no
[14:03:20] <orhtej2> > <@ericg:matrix.org> no

can you try modifying `/etc/dovecot/app-senders-passwd` by changing user nextcloud so that it has the following entry: `allow_nets=::1,127.0.0.1/24` to see if this is indeed the issue?
[14:04:29] <orhtej2> so basically add `::1` as allowed domain for `nextcloud`
[14:09:07] <eric_G> Still failing
[14:14:20] <orhtej2> > <@ericg:matrix.org> Still failing

any logs in `sudo journalctl -f` ?
[16:01:35] <orhtej2> > any logs in `sudo journalctl -f` ?

Come to think about you did you restart `dovecot` after making the change?
[16:02:08] <eric_G> yes I did
[18:06:47] <orhtej2> > Come to think about you did you restart `dovecot` after making the change?

I configured Nextcloud to use my user and it still fails as if it does not recognise SMTP Configuration of ynh host
[18:09:54] <orhtej2> https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/email_configuration.html#configuring-an-smtp-server
[18:09:54] <orhtej2> ```
Oct 9 18:00:38 komuna postfix/submission/smtpd[41951]: connect from localhost[::1] Oct 9 18:00:38 komuna postfix/submission/smtpd[41951]: lost connection after UNKNOWN from localhost[::1] Oct 9 18:00:38 komuna postfix/submission/smtpd[41951]: disconnect from localhost[::1] unknown=0/2 commands=0/2
Oct 9 18:04:18 komuna postfix/submission/smtpd[42342]: lost connection after STARTTLS from localhost[::1] Oct 9 18:04:18 komuna postfix/submission/smtpd[42342]: disconnect from localhost[::1] ehlo=1 starttls=1 commands=2
```

[19:16:25] <lapineige> > lapineige: yologen creates `conf/php_extra-fpm.conf` even if 'use PHP' is unchecked

Fixed 🙂
[19:19:47] <lapineige> (Oh, I see someone tried to create a `syncstorage` package 🙂)
[19:22:34] <lapineige> Yologen v0.7 is online, I fixed a few bugs 🙂, and added a configuration option for cron.
[19:30:21] <lapineige> (Oh, I see someone tried to create a `syncstorage` package 🙂) (for you info right now any Zip generated is stored on my server)
[19:45:29] <orhtej2> > (Oh, I see someone tried to create a `syncstorage` package 🙂) (for you info right now any Zip generated is stored on my server)

Current status: I'm losing my mind https://github.com/orhtej2/syncserver-rs_ynh/tree/testing
[19:55:10] <lapineige> Yologen v0.7 is online, I fixed a few bugs 🙂 (inclusing a major one), and added a configuration option for cron.
[19:56:48] <lapineige> Oh that's a Firefox sync remplacement ? 😲
[20:00:18] <lapineige> Hopefully with all the fixed bugs a basic app should not be generated without any extra human step to fix things™️ in the packaging
[20:00:23] <lapineige> Hopefully with all the fixed bugs a basic app should now be generated without any extra human step to fix things™️ in the packaging
[20:00:46] <lapineige> So Canada Goose you actually made the second Yologen-based ™️ Yunohost app ? 😄
[20:01:02] <orhtej2> > Oh that's a Firefox sync remplacement ? 😲

Yeah, that's their new impl