[00:00:21]
<m606> > you want to add a patch to `sources/patches`, not do `sed` wizardy
not exactly sure what you mean by sources/patches, but actually this is for a build script that for now will be run by github actions on the ynh app repo, so I don't have access to YNH core there
[00:03:10]
<m606> it will build the release to be used by the install script
[00:04:58]
<m606> > <@Salamandar:matrix.org> > sed
>
> Or
>
> > for loop
I guess i could indeed do a for loop to mimic the sed behavior, but `sed` is somehow dedicated for that and would be more compact (if I am able to understand how it works)
[03:19:38]
<Solrac> is the CI running?
[03:55:40]
<Émy – OniriCorpe> > <@lsolrac:matrix.org> is the CI running?
Yes, why?
[03:56:54]
<Solrac> I cant test my PR 😅
[03:59:52]
<Émy – OniriCorpe> > <@oniricorpe:im.emelyne.eu> yeah, it's the expected configuration to use 127.0.0.1 for this app
Have you read my messages here?
[04:01:11]
<Émy – OniriCorpe> Your PR is not solving the real issue, I don’t know what it is, but you can’t remove the reverse proxy like that
[04:02:44]
<Solrac> > <@oniricorpe:im.emelyne.eu> Your PR is not solving the real issue, I don’t know what it is, but you can’t remove the reverse proxy like that
The nginx config is left untouched...
https://github.com/YunoHost-Apps/motioneye_ynh/pull/20
[04:03:16]
<Émy – OniriCorpe> Yes, but with your edits, you’re bypassing nginx
[04:04:18]
<Solrac> Well, can you access it as-is?
[04:04:40]
<Solrac> also, how is it bypassing nginx
[04:12:41]
<Solrac> Émy – OniriCorpe:
[04:15:03]
<Émy – OniriCorpe> > <@lsolrac:matrix.org> Émy – OniriCorpe:
Hm sorry I can’t answer while being in the middle of my shower …
[04:15:37]
<Émy – OniriCorpe> > <@lsolrac:matrix.org> Well, can you access it as-is?
What do you mean? Are you asking about the normal use case?
[04:16:08]
<Solrac> > <@oniricorpe:im.emelyne.eu> What do you mean? Are you asking about the normal use case?
if you install motionEye, as it is now, without the PR, can you access it?
[04:16:26]
<Solrac> I was in yunohost 11, when I made the PR, I then migrated to 12
[04:16:55]
<Émy – OniriCorpe> > <@lsolrac:matrix.org> also, how is it bypassing nginx
Your PR is made to open the app port on the internet, while it has to be closed for the internet and be proxied through nginx, who is listening on localhost:app-port
[04:17:36]
<Émy – OniriCorpe> > <@lsolrac:matrix.org> if you install motionEye, as it is now, without the PR, can you access it?
I won’t install it right now as I’m currently going to bed, sorry
[04:18:00]
<Émy – OniriCorpe> Feel free to provide logs, I would be glad to help you fixing your issue properly
[04:18:15]
<Solrac> how can I get the nginx logs then?
[04:18:19]
<Émy – OniriCorpe> And if not me, I’m sure someone else would be glad too
[04:20:07]
<Émy – OniriCorpe> They are in the /var/log/nginx/ directory
[04:20:22]
<Émy – OniriCorpe> In a file named after your domain name
[04:21:04]
<Émy – OniriCorpe> You can also check the logs under /var/log/motioneye
[04:24:25]
<Émy – OniriCorpe> I’m leaving you
Good luck, I’m sure it will be fixed soon, don’t worry ^w^
[06:23:18]
<12b> the motioneye app can be accessed without your PR. When I tested it I had some issues when adding a camera, but the access to the app was fine
[06:30:48]
<12b> Solrac: you access the app using the standard https port, not the 8765 one because it is reverse proxied by nginx. This is for security reason. You do not want to bypass this by making the port 8765 available on all interfaces and accessing the app through there
[06:32:08]
<12b> well if you want to do this on your server, do it. But do not make that change for everybody in the yhunohost package.
[07:52:35]
<Yunohost Git/Infra notifications> [dokuwiki_ynh] ericgaspar merged [pull request #118](https://github.com/YunoHost-Apps/dokuwiki_ynh/pull/118): Testing
[07:52:35]
<Yunohost Git/Infra notifications> [dokuwiki_ynh] ericgaspar pushed 1 commit to master: Testing (#118) * chore(linter) fix linter warning in CI (#114) When using regexp in the nginx location field (locati... ([14e0bb1d](https://github.com/YunoHost-Apps/dokuwiki_ynh/commit/14e0bb1db855c3ef447c7a71dd5222a506ff76d1))
[11:30:12]
<eric_G> ah! I am no longer obeyed by `!testme` https://github.com/YunoHost-Apps/mattermost_ynh/pull/509
[11:31:10]
<eric_G> ha si!... recalcitrant
[11:31:57]
<eric_G> dur de la feuille
[11:47:43]
<mrflos> les droits ont été durcis @eric_G c'est maintenant `sudo !testme`
[11:51:30]
<eric_G> Et chui pas dans les sudoers
[11:53:45]
<mrflos> > <@ericg:matrix.org> Et chui pas dans les sudoers
oui c'est réservé a celleux qui ont packagé plus de 500 apps, tu y es presque!
[13:58:14]
<Aleks (he/him/il/lui)> (smol poke ljf if you have opinion on https://github.com/YunoHost-Apps/borg_ynh/pull/160 )
[14:06:23]
<Solrac> > Solrac: you access the app using the standard https port, not the 8765 one because it is reverse proxied by nginx. This is for security reason. You do not want to bypass this by making the port 8765 available on all interfaces and accessing the app through there
Outside the local network, I cannot access it by any other means. I can only access it as you're describing; as (local IP or localhost):8765 -- I realize that yes, this could be a me-issue since I am in a reverse proxy, as I cannot access it via (domain.tld):8765
[15:17:13]
<oliviermgx> Hey everybody 👋
I'm here to add an app in YNH !
[19:07:39]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar pushed 1 commit to testing: Merge pull request #242 from YunoHost-Apps/testing Update manifest.toml ([8088eef0](https://github.com/YunoHost-Apps/wordpress_ynh/commit/8088eef061863936aac7d83546b5e67cd177f2fe))
[19:09:12]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar pushed 1 commit to testing: Update manifest.toml ([dd02182f](https://github.com/YunoHost-Apps/wordpress_ynh/commit/dd02182fcba6f326230866d837deb89abe1e8182))
[19:09:46]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar opened [pull request #246](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246): Update manifest.toml
[19:09:48]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246#issuecomment-2489351138) on [issue #246](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246) Update manifest.toml: testme
[19:09:52]
<Yunohost Git/Infra notifications> [wordpress_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246#issuecomment-2489351221) on [issue #246](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246) Update manifest.toml: Living in the past, are we? :cow::eye:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bullseye-dev.y...
[19:09:54]
<Yunohost Git/Infra notifications> [wordpress_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246#issuecomment-2489351279) on [issue #246](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246) Update manifest.toml: Test it, book it, worm it, technologic. :robot: :notes:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-ap...
[19:10:06]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar edited [pull request #246](https://github.com/YunoHost-Apps/wordpress_ynh/pull/246): Update manifest.toml
[19:12:19]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/233#issuecomment-2489354814) on [issue #233](https://github.com/YunoHost-Apps/wordpress_ynh/pull/233) add allow_email parameter: the password generated for the mail must be displayed somehow...
[19:12:32]
<Yunohost Git/Infra notifications> [wordpress_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/241#issuecomment-2489355303) on [issue #241](https://github.com/YunoHost-Apps/wordpress_ynh/pull/241) [autopatch] Automatic patch attempt for helpers 2.1: Living in the future, are we?
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api...
[19:12:32]
<Yunohost Git/Infra notifications> [wordpress_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/241#issuecomment-2489355111) on [issue #241](https://github.com/YunoHost-Apps/wordpress_ynh/pull/241) [autopatch] Automatic patch attempt for helpers 2.1: testme
[19:12:32]
<Yunohost Git/Infra notifications> [wordpress_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/wordpress_ynh/pull/241#issuecomment-2489355256) on [issue #241](https://github.com/YunoHost-Apps/wordpress_ynh/pull/241) [autopatch] Automatic patch attempt for helpers 2.1: Living in the past, are we? :cow::eye:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bullseye-dev.y...
[19:13:51]
<eric_G> ^ah ah 😅 touching WordPress adds so much noice to this channel
[19:16:59]
<Aleks (he/him/il/lui)> mhyeah we should rework what we really want to push in terms of notifications i suppose
[19:17:56]
<Aleks (he/him/il/lui)> naively i'd say we could only display like idk the 10% apps with the most votes on the catalog, and maybe not every message
[19:18:19]
<Aleks (he/him/il/lui)> like probably the yunohost-bot and the !testme message are unecessary
[19:24:34]
<eric_G> https://ci-apps.yunohost.org/ci/job/12070 still have an issue. ( 🤔I am thinking of a CI issue)
[19:25:07]
<eric_G> the app installs correctly IRL
[19:26:26]
<eric_G> but appears as broken in the catalog 🫠
[19:27:39]
<eric_G> at some point, if I remember, do.doc_ynh use to pass the CI test
[19:36:26]
<Aleks (he/him/il/lui)> `connect EHOSTUNREACH 0.89.193.208:58823` 🤔
[19:36:29]
<Aleks (he/him/il/lui)> dafuq is dat
[19:37:16]
<Aleks (he/him/il/lui)> that IP is weird af
[19:37:42]
<Aleks (he/him/il/lui)> (or weird ... `AF_INET`, if you know what i mean)
[19:37:56]
<Aleks (he/him/il/lui)> 🥁 🎺
[19:38:58]
<Aleks (he/him/il/lui)> ¯\_(ツ)_/¯ sorry i'm not helpful 😅
[21:23:37]
<orhtej2> I mean [this is the offending code block](https://github.com/l-atelier-des-chercheurs/dodoc/blob/2ac2857ba470715dc9ad230e6e55a7e8b6fd843a/core2/main2.js#L136) and even looking at upstream impl it does not make any sense
[21:23:47]
<orhtej2> (upstream for `portscanner`)
[21:26:05]
<orhtej2> as dodoc does not supply `host` nor `opts` hence `host` should be 127.0.0.1
[21:27:51]
<orhtej2> let's check something.... https://github.com/YunoHost-Apps/dodoc_ynh/pull/21
[22:32:28]
<orhtej2> > let's check something.... https://github.com/YunoHost-Apps/dodoc_ynh/pull/21
eric_G: check succesful, port should be a number not a string
[22:43:23]
<eric_G> thanks! orhtej2, once again you saved me hours...
[23:13:33]
<thomas> Hi all, I did something a bit supid. I updated the Mitra package and now the version is 3.10.0~ynh1. Problem : I just realized that the previous were numbered 3.90 & 3.80 (while they should have been numbered 3.9.0 & 3.8.0). So the problem now is that the new version does not appear in the update section of the YunoHost admin interface, and the only way to upgrade the app is through the CLI using the --force option.
[23:15:14]
<thomas> So question is: is there a trick that could help have the upgrade available in the YunoHost admin web UI?
[23:16:07]
<thomas> (I mean without having to wait for v3.91.x or 4.x.x upstream)
[23:44:38]
<Émy – OniriCorpe> 3.100? 🫠