Sunday, November 13, 2022
apps@conference.yunohost.org
November
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
       
             

[08:22:38] <Yunohost Git/Infra notifications> [apps] @ericgaspar opened pull request #1539: Update apps level https://github.com/YunoHost/apps/pull/1539
[08:22:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar commented on issue #417 Add notify push option: testme https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-1312159773
[08:22:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to add-notify_push-option: fix json https://github.com/YunoHost-Apps/nextcloud_ynh/commit/0d05a4f43b393afd88fe2d4597cca8c561f4e815
[08:22:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot commented on issue #417 Add notify push option: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/4186/badge)](... https://github.com/YunoHost-Apps/nextcloud_ynh/pull/417#issuecomment-1312159792
[08:22:40] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to modify_level: Update apps level https://github.com/YunoHost/apps/commit/69a2d4590e0f54f3c3bedb02fd3a835893212389
[08:22:48] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch modify_level
[08:22:54] <Yunohost Git/Infra notifications> [nextcloud_ynh] @LAfricain commented on issue #380 how to access nextcloud locally: 192.168.x.x myurl.yunohost.me https://github.com/YunoHost-Apps/nextcloud_ynh/issues/380#issuecomment-1312191559
[08:23:34] <Tio> Friendica's stable release has been released a month or so ago https://github.com/YunoHost-Apps/friendica\_ynh/issues/79 - would you guys mind if you have the time to bump the ynh package to the latest please 😄
[08:23:34] <Yunohost Git/Infra notifications> [nextcloud_ynh] @LAfricain commented on issue #380 how to access nextcloud locally: 192.168.x.x myurl.yunohost.me But the problem is when Im outside the local network I need to comment the line in the f... https://github.com/YunoHost-Apps/nextcloud_ynh/issues/380#issuecomment-1312191559
[08:23:39] <Tio> Friendica's stable release has been released a month or so ago https://github.com/YunoHost-Apps/friendica_ynh/issues/79 - would you guy mind if you have the time to bump the ynh package to the latest please 😄
[08:26:21] <tituspijean> Idea for app packaging v2/v3: optional build steps if built sources are not available for an arch
[08:26:22] <Yunohost Git/Infra notifications> [apps] @tituspijean commented on issue #1539 Update apps level: - [zerotier](https://ci-apps.yunohost.org/ci/job/11502) - [ztncui](https://ci-apps.yunohost.org/ci/job/11374) https://github.com/YunoHost/apps/pull/1539#issuecomment-1312479828
[08:26:22] <Yunohost Git/Infra notifications> [apps] @tituspijean commented on issue #1539 Update apps level: - [ ] [zerotier](https://ci-apps.yunohost.org/ci/job/11502) - [ ] [ztncui](https://ci-apps.yunohost.org/ci/job/11374) https://github.com/YunoHost/apps/pull/1539#issuecomment-1312479828
[08:27:03] <Yunohost Git/Infra notifications> [apps] @tituspijean edited review pull request #1539: Update apps level https://github.com/YunoHost/apps/pull/1539#pullrequestreview-1178129103
[08:27:03] <Yunohost Git/Infra notifications> [apps] @tituspijean commented on pull request #1539 Update apps level: suggestion "level": 6, https://github.com/YunoHost/apps/pull/1539#discussion_r1020766442
[08:27:03] <Yunohost Git/Infra notifications> [apps] @tituspijean commented on pull request #1539 Update apps level: suggestion "level": 7, https://github.com/YunoHost/apps/pull/1539#discussion_r1020766471
[08:27:04] <Yunohost Git/Infra notifications> [apps] @tituspijean commented on issue #1539 Update apps level: - [x] [zerotier](https://ci-apps.yunohost.org/ci/job/11502) - [ ] [ztncui](https://ci-apps.yunohost.org/ci/job/11374) https://github.com/YunoHost/apps/pull/1539#issuecomment-1312479828
[08:27:04] <Yunohost Git/Infra notifications> [apps] @tituspijean approved pull request #1539 Update apps level https://github.com/YunoHost/apps/pull/1539#pullrequestreview-1178129290
[08:27:04] <Yunohost Git/Infra notifications> [apps] @tituspijean changes_requested pull request #1539 Update apps level https://github.com/YunoHost/apps/pull/1539#pullrequestreview-1178129103
[08:27:04] <Yunohost Git/Infra notifications> [apps] @tituspijean pushed 1 commit to modify_level: zerotier and ztncui are fixed https://github.com/YunoHost/apps/commit/4962c4bb7794bffb54795a4e639f29edb4c48bdb
[08:27:14] <Yunohost Git/Infra notifications> [apps/master] Update apps level - yunohost-bot
[08:27:14] <Yunohost Git/Infra notifications> [apps] @ericgaspar merged pull request #1539: Update apps level https://github.com/YunoHost/apps/pull/1539
[08:27:14] <Yunohost Git/Infra notifications> [apps/master] Merge pull request #1539 from YunoHost/modify_level Update apps level - Éric Gaspar
[08:27:14] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 3 commits to master: https://github.com/YunoHost/apps/compare/bbe61663240c...c4119de0f1e7
[08:27:14] <Yunohost Git/Infra notifications> [apps] @ericgaspar deleted branch modify_level
[08:27:14] <Yunohost Git/Infra notifications> [apps/master] zerotier and ztncui are fixed - tituspijean
[08:27:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @tituspijean closed issue #380: how to access nextcloud locally https://github.com/YunoHost-Apps/nextcloud_ynh/issues/380
[08:27:38] <Yunohost Git/Infra notifications> [nextcloud_ynh] @tituspijean commented on issue #380 how to access nextcloud locally: This is most likely a hairpinning issue as per YunoHosts documentation. Your solution with /etc/hosts is the correct... https://github.com/YunoHost-Apps/nextcloud_ynh/issues/380#issuecomment-1312521109
[08:27:50] <Yunohost Git/Infra notifications> @tituspijean edited repository hextris_ynh: A fast paced puzzle game package for YunoHost. https://github.com/YunoHost-Apps/hextris_ynh
[08:27:50] <Yunohost Git/Infra notifications> @tituspijean edited repository dokuwiki_ynh: Dokuwiki package for YunoHost https://github.com/YunoHost-Apps/dokuwiki_ynh
[08:27:50] <Yunohost Git/Infra notifications> @tituspijean edited repository wordpress_ynh: WordPress package for YunoHost https://github.com/YunoHost-Apps/wordpress_ynh
[08:27:51] <Yunohost Git/Infra notifications> @tituspijean edited repository my_webapp_ynh: Custom Web app with SFTP access https://github.com/YunoHost-Apps/my_webapp_ynh
[08:27:51] <Yunohost Git/Infra notifications> @tituspijean edited repository nextcloud_ynh: Nextcloud package for YunoHost https://github.com/YunoHost-Apps/nextcloud_ynh
[08:27:51] <Yunohost Git/Infra notifications> @tituspijean edited repository gitlab_ynh: GitLab package for YunoHost https://github.com/YunoHost-Apps/gitlab_ynh
[08:27:54] <Yunohost Git/Infra notifications> @tituspijean edited repository example_ynh: Example package for YunoHost applications. https://github.com/YunoHost/example_ynh
[08:27:55] <Yunohost Git/Infra notifications> @tituspijean edited repository package_check: Shell script which check package actions: install, remove, upgrade, backup, restore… https://github.com/YunoHost/package_check
[08:27:55] <Yunohost Git/Infra notifications> @tituspijean edited repository apps: YunoHost apps catalog https://github.com/YunoHost/apps
[08:27:55] <Yunohost Git/Infra notifications> @tituspijean edited repository test_apps: Dummy apps designed for unit tests of Yunohost https://github.com/YunoHost/test_apps
[08:27:55] <Yunohost Git/Infra notifications> @tituspijean edited repository change_url_app_ynh: app to test change_url script https://github.com/YunoHost/change_url_app_ynh
[08:28:02] <Yunohost Git/Infra notifications> [nextcloud_ynh] @nin0-0 opened issue #525: custom_overlay.css 302 not found https://github.com/YunoHost-Apps/nextcloud_ynh/issues/525
[08:28:02] <Yunohost Git/Infra notifications> [nextcloud_ynh] @nin0-0 edited issue #525: custom_overlay.css 302 not found https://github.com/YunoHost-Apps/nextcloud_ynh/issues/525
[08:28:21] <Yunohost Git/Infra notifications> [wordpress_ynh/testing] Merge branch testing into ci-auto-update-v6.0.3 - Éric Gaspar
[08:28:21] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean pushed 5 commits to testing: https://github.com/YunoHost-Apps/wordpress_ynh/compare/1fbb8f5a36b9...af117419e2b0
[08:28:21] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean merged pull request #196: Upgrade to v6.0.3 and fix auto-updater https://github.com/YunoHost-Apps/wordpress_ynh/pull/196
[08:28:21] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean deleted branch ci-auto-update-v6.0.3
[08:28:21] <Yunohost Git/Infra notifications> [wordpress_ynh/testing] Fix auto-updater - tituspijean
[08:28:22] <Yunohost Git/Infra notifications> [wordpress_ynh/ci-auto-update-v6.1] Fix auto-updater - tituspijean
[08:28:22] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean pushed 6 commits to ci-auto-update-v6.1: https://github.com/YunoHost-Apps/wordpress_ynh/compare/0b1a84bb6eb5...79b878ea090e
[08:28:22] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean commented on issue #200 Upgrade to version 6.1: testme https://github.com/YunoHost-Apps/wordpress_ynh/pull/200#issuecomment-1312543135
[08:28:22] <Yunohost Git/Infra notifications> [wordpress_ynh/testing] Merge pull request #196 from YunoHost-Apps/ci-auto-update-v6.0.3 - tituspijean
[08:28:22] <Yunohost Git/Infra notifications> [wordpress_ynh/ci-auto-update-v6.1] Merge pull request #196 from YunoHost-Apps/ci-auto-update-v6.0.3 - tituspijean
[08:28:22] <Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot commented on issue #200 Upgrade to version 6.1: Fingers crossed
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/4200/bad... https://github.com/YunoHost-Apps/wordpress_ynh/pull/200#issuecomment-1312543141
[08:28:23] <Yunohost Git/Infra notifications> [wordpress_ynh/ci-auto-update-v6.1] Merge branch testing into ci-auto-update-v6.1 - tituspijean
[08:28:23] <Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot pushed 1 commit to ci-auto-update-v6.1: Auto-update README https://github.com/YunoHost-Apps/wordpress_ynh/commit/a2edb3e4df146b674e748847a048c4dce4999080
[08:28:47] <Yunohost Git/Infra notifications> @SquirrelMe forked nextcloud_ynh to SquirrelMe/nextcloud_ynh: https://github.com/SquirrelMe/nextcloud_ynh
[08:30:39] <Yunohost Git/Infra notifications> [wordpress_ynh] @github-actions[bot] pushed 1 commit to ci-auto-update-v6.1: Upgrade to version 6.1 https://github.com/YunoHost-Apps/wordpress_ynh/commit/d5b78b6db86b48cc490d3b5a09606ef5d1b4cbe4
[10:30:04] <limezy> Hi ! I was just wondering how an app goes to level 8 from level 7 ? I guess that the process isn’t automatic ? If I’m not wrong it’s now more than 6 months that a few apps I maintain (Dex, Minio, Outline) are stabilized at level 7 and regularly updated 😊. May I “apply” for a nice and clean level 8 badge ? 🎖
[11:02:55] <tituspijean> limezy: it's automated :) https://github.com/YunoHost/package_linter/blob/76e8d962e189d635cb904bc2bd1c97ad6b2ba02e/package_linter.py#L460
[16:53:21] <tituspijean> @all I'll temporarily put the testing branch as default for Wordpress, there's a bug with the auto-updater and prefer not to directly push onto master branch ;)
[16:53:47] <Yunohost Git/Infra notifications> @tituspijean edited repository wordpress_ynh: WordPress package for YunoHost https://github.com/YunoHost-Apps/wordpress_ynh
[16:54:31] <Yunohost Git/Infra notifications> [wordpress_ynh] @github-actions[bot] opened pull request #202: Upgrade to version 6.1 https://github.com/YunoHost-Apps/wordpress_ynh/pull/202
[16:54:31] <Yunohost Git/Infra notifications> [wordpress_ynh] @github-actions[bot] created new branch ci-auto-update-v6.1
[16:54:31] <Yunohost Git/Infra notifications> [wordpress_ynh] @github-actions[bot] pushed 1 commit to ci-auto-update-v6.1: Upgrade to version 6.1 https://github.com/YunoHost-Apps/wordpress_ynh/commit/2e162a9b0d45b176a7c2fbc1b60124d005438bd1
[16:54:38] <Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot pushed 1 commit to ci-auto-update-v6.1: Auto-update README https://github.com/YunoHost-Apps/wordpress_ynh/commit/e471044dc6d32049e29048dc122e7a6e27e265fa
[16:54:38] <Yunohost Git/Infra notifications> [wordpress_ynh] @yunohost-bot commented on issue #202 Upgrade to version 6.1: :v:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/4209/badge)](https://... https://github.com/YunoHost-Apps/wordpress_ynh/pull/202#issuecomment-1312774818
[16:55:13] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean commented on issue #200 Upgrade to version 6.1: Closing, due to bugged auto-updater. https://github.com/YunoHost-Apps/wordpress_ynh/pull/200#issuecomment-1312774937
[16:55:13] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean closed pull request #200: Upgrade to version 6.1 https://github.com/YunoHost-Apps/wordpress_ynh/pull/200
[16:55:16] <Yunohost Git/Infra notifications> [wordpress_ynh] @tituspijean deleted branch ci-auto-update-v6.1
[17:13:49] <Krakinou> Hello, I got a pb while packaging a new app : I get redirected to login page when I try to access the app via a path (mydomain.org/mypath) but not when I install it on its own domain, and I can't figure why
[17:14:04] <Krakinou> this is my nginx :
[17:14:05] <Krakinou> rewrite '^/zwavejsui(/.*)$' $1 break;
proxy_pass http://localhost:8091;
proxy_set_header X-External-Path /zwavejsui;
proxy_http_version 1.1;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection $connection_upgrade;

[17:14:47] <Krakinou> and this is the relevant part of /etc/ssowat/conf.json:
[17:15:03] <Krakinou> "zwave-js-ui.main": {
"auth_header": true,
"label": "Zwave-JS-UI",
"public": false,
"show_tile": true,
"uris": [
"yuno.dev/zwavejsui"
],
"users": [
"nicolas"
]

[17:15:14] <tituspijean> Krakinou: you need to follow the redirection flow from your browser to understand where NGINX/SSOwat is not doing what you want them to do.
[17:15:55] <Krakinou> yep, but I just have a 302 on https://yuno.dev/zwavejsui/
[17:15:58] <tituspijean> you might try adding a trailing slash after the port, who knows it may work that way.
[17:16:20] <tituspijean> and then it does not tell you where the 302 goes?
[17:16:56] <Krakinou> to https://yuno.dev/yunohost/sso/
[17:17:31] <Krakinou> I tried with a trailing slash in the nignx config... no luck...
[17:18:05] <tituspijean> and are you sure the rewrite is needed?
[17:19:56] <Krakinou> well, I"m not very clear on it, but it's in the application documentation, and when I removed it, it does not work anymore, even on the root domain
[17:20:53] <Krakinou> if my understanding is correct, it remove the path from what is sent to the server
[17:21:52] <Krakinou> if i modify conf.json this way:
[17:21:55] <Krakinou> "uris": [
"yuno.dev/zwavejsui" ,
],
[17:22:19] <Krakinou> sorry, bad typing
[17:22:42] <Krakinou> "uris": [
"yuno.dev/zwavejsui",
"yuno.dev"
],

[17:23:06] <Krakinou> then in this case the app is working, but only with the rewrite, so I assume the rewrite is required
[17:23:29] <Krakinou> of course, this is not perfect from a security standpoint 😀
[17:43:54] <Krakinou> I assume that as the rewrite remove the path, ssowat considers that I'm trying to access the root domain and forbid it,
[17:43:55] <Krakinou> what is not clear is why it's not in the log of the browser
[17:53:34] <tituspijean> @room I am re-enabling the auto-updaters that were disabled due to inactivity:
```
dotclear2_ynh
searx_ynh
cesium_ynh
spip_ynh
humhub_ynh
thelounge_ynh
mytinytodo_ynh
mastodon_ynh
rss-bridge_ynh
concrete5_ynh
monica_ynh
plume_ynh
writefreely_ynh
codimd_ynh
mineweb_ynh
phpboost_ynh
phpipam_ynh
yeswiki_ynh
navidrome_ynh
simple-torrent_ynh
privatebin_ynh
backdrop_ynh
django_ynh
django_ynh
tiddlywiki_ynh
question2answer_ynh
photoview_ynh
dispatch_ynh
omeka-s_ynh
owncast_ynh
plainpad_ynh
send_ynh
digiwords_ynh
libreddit_ynh
2FAuth_ynh
tracim_ynh
getsimple_ynh
```
[17:55:47] <tituspijean> I'm astounished... there are 163 apps using an auto-updater :o
[17:56:59] <tituspijean> (I'm proud of you, y'all lazy packagers)
[18:06:15] <Krakinou> i'm in the process of giving it a look :), because updating is a pain in the a**!
[18:11:31] <Krakinou> ok, my issue is a crash between nginx rewrite and ssowat:
[18:11:40] <Krakinou> if I edit conf.json this way:
[18:11:54] <Krakinou> "uris": [
"yuno.dev/zwavejsui",
"yuno.dev/control-panel",
"yuno.dev/static",
"yuno.dev/api",
"yuno.dev/socket.io",
],

[18:12:24] <Krakinou> ie : all paths required by the app while removing the "zwavejsui" part that is rewrited by nginx
[18:12:32] <Krakinou> it works
[18:18:25] <tituspijean> Can you push your work so far on YunoHost-Apps? I'll try to hack into it. Does it need a physical device for zwave, or can it run alone?
[18:20:21] <Krakinou> yep, I opened a repo this afternoon : https://github.com/YunoHost-Apps/zwave-js-ui_ynh
[18:20:56] <tituspijean> oh sorry I saw it, but did not notice the testing branch
[18:23:59] <Krakinou> so it should be "testable" now
[18:24:03] <Krakinou> only install and remove
[18:24:10] <Krakinou> no need for a device
[18:24:40] <Krakinou> and this is reaaaaally a work in progress, so don't expect too much 🙂
[18:26:52] <Krakinou> I use this as a documentation : https://github.com/zwave-js/zwave-js-ui/discussions/1396
[18:27:21] <Krakinou> but in the end, changing the /config/app.ts file to enter the path does not change anything
[18:39:19] <tituspijean> Fixed
[18:39:28] <tituspijean> give me two minutes to send a PR :)
[18:40:30] <tituspijean> Krakinou: or can I directly push to testing?
[18:40:41] <Krakinou> yes you can
[18:41:17] <Krakinou> I don't thing any one apart you and me are looking at this repo 🙂
[18:41:23] <Krakinou> think*
[18:45:06] <Krakinou> and if you can explain me what you did, it could help for the next apps 🙂
[18:48:04] <tituspijean> Errr, it boils down to removing the rewrite instruction and adding a trailing slash after the port :/
[18:48:27] <tituspijean> I also changed `localhost` to `127.0.0.1` but it looks like it was not necessary
[18:49:41] <Krakinou>
[18:49:49] <Krakinou> i tried that...
[18:50:43] <tituspijean> as to *why* it works like that, it's years of ~~experience~~ mindlessly fighting against NGINX without actually understanding what it does. 😅
And I have nothing to explain this kind of horrors https://github.com/YunoHost-Apps/flarum_ynh/blob/8c883b8366d28f09de78b1f6beb83bbb84ea8d93/conf/nginx.conf#L10
[18:50:54] <tituspijean> > <Krakinou> i tried that...

(╯°□°)╯︵ ┻━┻
[18:51:01] <Krakinou> mwokey, I take note for next time : always triple down on the tests I'm doing as I'm not to be trusted...
[18:51:41] <tituspijean> Be careful with JS apps too, there might be some caching occurring, try also after clearing the browser cache
[18:52:16] <Krakinou> ah, maybe that's why
[18:52:30] <tituspijean> let me know if it works on your end :)
[18:52:31] <Krakinou> or maybe not...
[18:52:47] <Krakinou> i have to go to eat, but i'll try later tonight
[18:52:54] <Krakinou> thansk a lot anyway
[18:53:13] <tituspijean> you're welcome :)
[18:53:22] <tituspijean> and for proof that it might work:
[18:53:23] <tituspijean> https://aria.im/_matrix/media/v1/download/pijean.ovh/431b1450fb9f0c38d9c6b3f0aa3b89db43c59baed5dfafcb921e3ea86c75bde7
[19:16:44] <Krakinou> it's working! thanks!
[19:17:10] <Krakinou> i think you saved me like two weeks of screaming on nginx 🙂
[19:25:28] <tituspijean> 🧡
[19:49:31] <Yunohost Git/Infra notifications> [nextcloud_ynh] @SquirrelMe opened pull request #526: Fix upgrade of old versions <=v21 and for v22 to v23 https://github.com/YunoHost-Apps/nextcloud_ynh/pull/526
[19:52:05] <Yunohost Git/Infra notifications> [nextcloud_ynh] @SquirrelMe edited pull request #526: Fix upgrade of old versions <=v21 and for v22 to v23 https://github.com/YunoHost-Apps/nextcloud_ynh/pull/526
[20:02:31] <Yunohost Git/Infra notifications> [nextcloud_ynh] @SquirrelMe edited pull request #526: Fix upgrade of old versions <=v21 and for v22 to v23 https://github.com/YunoHost-Apps/nextcloud_ynh/pull/526
[21:12:12] <Yunohost Git/Infra notifications> @alexAubin deleted repository change_url_app_ynh: app to test change_url script https://github.com/YunoHost/change_url_app_ynh
[21:48:33] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed issue #177: NC Colabora office app fails to open documents https://github.com/YunoHost-Apps/nextcloud_ynh/issues/177
[21:50:04] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar closed issue #73: Nextcloud optimizations https://github.com/YunoHost-Apps/nextcloud_ynh/issues/73