[01:49:41]
<m606> I am still struggling with this app to set basic authentication on an API endpoint.
Without setting the authentication, a curl request to the endpoint returns HTTP response code 200 and the string `{"status":"ok"}`.
When setting the authentication, here is the log of a curl request to the endpoint: https://paste.yunohost.org/litufuvalo.log
By the error code and the message, I know it gets trapped in one of the two `if` block in this section of the upstream nodejs/express server code: https://github.com/sharevb/puppeteer-htmltopdf/blob/main/server.js#L47-L59. I don't know which one exactly, and more annoying - I don't understand why it does so since the header `authorization` seems to be set properly. So could it be a typo in the header name (like the first space in the value?!), or would YNH mess with the headers, a bit like here https://github.com/YunoHost/issues/issues/2301 ?
[08:39:03]
<orhtej2> > Do you have alkow whatever auth set for app? Otherwise ssowat strips authorization headers mid flight
setting is `protect_against_basic_auth_spoofing`
[08:39:03]
<orhtej2> > <@m606:matrix.org> I am still struggling with this app to set basic authentication on an API endpoint.
> Without setting the authentication, a curl request to the endpoint returns HTTP response and the string `{"status":"ok"}`.
> When setting the authentication, here is the log of a curl request to the endpoint: https://paste.yunohost.org/litufuvalo.log
> By the error code and the message, I know it gets trapped in one of the two `if` block in this section of the upstream nodejs/express server code: https://github.com/sharevb/puppeteer-htmltopdf/blob/main/server.js#L47-L59. I don't know which one exactly, and more annoying - I don't understand why it does so since the header `authorization` seems to be set properly. So could it be a typo in the header name (like the first space in the value?!), or would YNH mess with the headers, a bit like here https://github.com/YunoHost/issues/issues/2301 ?
Do you have alkow whatever auth set for app? Otherwise ssowat strips authorization headers mid flight
[08:39:04]
<Yunohost Git/Infra notifications> [apps] orhtej2 approved [pull request #3554](https://github.com/YunoHost/apps/pull/3554#pullrequestreview-4762275394) Add Continuwuity to wishlist
[08:39:18]
<Yunohost Git/Infra notifications> [apps] orhtej2 deleted branch add-to-wishlist-continuwuity
[08:43:20]
<Yunohost Git/Infra notifications> [apps] orhtej2 approved [pull request #3549](https://github.com/YunoHost/apps/pull/3549#pullrequestreview-4762308876) Add Kima to wishlist: Doable but a lot of AI stuff so IDK
[08:44:04]
<Yunohost Git/Infra notifications> [apps] orhtej2 merged [pull request #3554](https://github.com/YunoHost/apps/pull/3554) : Add Continuwuity to wishlist
[08:44:44]
<Yunohost Git/Infra notifications> [apps] orhtej2 approved [pull request #3394](https://github.com/YunoHost/apps/pull/3394#pullrequestreview-4762318929) Add AptCacher NG to wishlist
[08:44:49]
<Yunohost Git/Infra notifications> [apps] Auto-merge has been enabled by orhtej2 on [pull request #3394](https://github.com/YunoHost/apps/pull/3394) : Add AptCacher NG to wishlist
[08:47:04]
<Yunohost Git/Infra notifications> [apps] orhtej2 pushed 207 commits to add-to-wishlist-minecraft-overviewer ([c164f3796748...c9760e29c6f0](https://github.com/YunoHost/apps/compare/c164f3796748...c9760e29c6f0))
[08:47:48]
<Yunohost Git/Infra notifications> [apps] orhtej2 pushed to main: Add Continuwuity to wishlist (#3554) ([f9c074ba](https://github.com/YunoHost/apps/commit/f9c074ba6ad544812f199f242554dc0d0e6efd93))
[08:47:54]
<Yunohost Git/Infra notifications> [apps] orhtej2 [commented](https://github.com/YunoHost/apps/pull/3406#issuecomment-5056299841) on [issue #3406](https://github.com/YunoHost/apps/pull/3406) Add Minecraft overviewer to wishlist: Static site generator, not sure how much value does it add beyond running locally + upload to my_webapp
[08:48:06]
<Yunohost Git/Infra notifications> [apps] orhtej2 pushed to main: Add AptCacher NG to wishlist (#3394) ([7b4c4c6e](https://github.com/YunoHost/apps/commit/7b4c4c6e7668fa84c54e9bdec7498f013afbd0f6))
[08:49:29]
<Yunohost Git/Infra notifications> [apps] orhtej2 deleted branch add-to-wishlist-aptcacher-ng
[08:50:39]
<Yunohost Git/Infra notifications> [apps] orhtej2 merged [pull request #3394](https://github.com/YunoHost/apps/pull/3394) : Add AptCacher NG to wishlist
[08:55:10]
<Yunohost Git/Infra notifications> [apps] orhtej2 pushed 179 commits to add-to-wishlist-fmhy ([581e44de8208...f8d7f62493a5](https://github.com/YunoHost/apps/compare/581e44de8208...f8d7f62493a5))
[08:58:41]
<Yunohost Git/Infra notifications> [apps] orhtej2 approved [pull request #3427](https://github.com/YunoHost/apps/pull/3427#pullrequestreview-4762407055) Add FMHY to wishlist: Doable but I dont understand what this is 🤷
[10:14:17]
<Yunohost Git/Infra notifications> [borg_ynh] tio-trom [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/141#issuecomment-5057164470) on [issue #141](https://github.com/YunoHost-Apps/borg_ynh/issues/141) Feature request : Prune in config panel: I apologize I am going through some personal problems so I will check as soon as I can. If anyone else can also check th...
[11:04:42]
<Yunohost Git/Infra notifications> [borg_ynh] fflorent [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/141#issuecomment-5057659333) on [issue #141](https://github.com/YunoHost-Apps/borg_ynh/issues/141) Feature request : Prune in config panel: @tio-trom Thank you, take care of yourself
[11:46:34]
<Yunohost Git/Infra notifications> [apps] Thovi98 pushed to seerr: migrate from jellyseerr to seerr ([1aecef58](https://github.com/YunoHost/apps/commit/1aecef589490bc4774a26f21154be695a4d6f31b))
[11:47:01]
<Yunohost Git/Infra notifications> [apps] Thovi98 opened [pull request #3557](https://github.com/YunoHost/apps/pull/3557) : migrate from jellyseerr to seerr
[14:41:10]
<m606> That seems to have worked! Thanks, that was a painful path until then!
Would you be able to explain what setting it to `true` protects from more concretely in general? or what it is vulnerable to when set to `false` ?
I had been through the related doc https://doc.yunohost.org/en/dev/packaging/advanced/sso_ldap_integration#specific-case, but I didn't get that I was concerned by this setting. Actually now I'm lost on how to differentiate the 2 specific cases mentioned?
And if I get it well, SSOWat is still playing a role between nginx and the app even when setting `sso = false` in the manifest which is a bit counterintuitive (e.g.: here stripping the authentication header)?
I will try to improve the doc if getting some answers to those points.
[15:22:27]
<Thomas> If I'm not mistaken, sso and ldap declarations are just information without consequences for now
[17:25:18]
<Yunohost Git/Infra notifications> [borg_ynh] kay0u [commented](https://github.com/YunoHost-Apps/borg_ynh/pull/263#discussion_r3639140109) on pull request #263 Testing: We only need these variables in this scope
suggestion
local hourly daily weekly monthly yearly
args=()
[17:25:19]
<Yunohost Git/Infra notifications> [borg_ynh] kay0u [commented](https://github.com/YunoHost-Apps/borg_ynh/pull/263#issuecomment-5059984812) on [issue #263](https://github.com/YunoHost-Apps/borg_ynh/pull/263) Testing: LGTM otherwise
[17:25:25]
<Yunohost Git/Infra notifications> [borg_ynh] fflorent pushed to testing: make variables in set_pruning_arguments local only Co-authored-by: Kayou <pierre@kayou.io> ([74111778](https://github.com/YunoHost-Apps/borg_ynh/commit/74111778b537b965d649055b3674f3b3bcf9c301))
[17:25:27]
<Yunohost Git/Infra notifications> [borg_ynh] fflorent [commented](https://github.com/YunoHost-Apps/borg_ynh/pull/263#issuecomment-5060065898) on [issue #263](https://github.com/YunoHost-Apps/borg_ynh/pull/263) Testing: Thanks @kay0u
[17:26:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed to testing: Update manifest.toml ([88468824](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/884688242e9062a98651b3000071c5d68f92358c))
[17:26:49]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed to postgresql_reboot: Update manifest.toml ([34bf596a](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/34bf596a8b372d17d0fda3b807aee82515afc913))
[17:26:50]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar opened [pull request #896](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/896) : Testing
[17:26:52]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed to testing: Testing (#889) * Update ADMIN.md: update advice to open shell + add a note on how to promote a user to admin * Update ... ([37d8f703](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/37d8f70302407bcf455fd84e3a2522294dfb1ad9))
[17:26:54]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed to testing: Merge branch master into testing ([8250de03](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/8250de036a7524f73af19333859ba8066d3d9d83))
[17:27:01]
<Yunohost Git/Infra notifications> [borg_ynh] tio-trom [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/141#issuecomment-5060798177) on [issue #141](https://github.com/YunoHost-Apps/borg_ynh/issues/141) Feature request : Prune in config panel: Ok I see it in the panel now
<img width="1530" height="718" alt="Image" src="https://github.com/user-attachments/assets...
[17:27:37]
<Yunohost Git/Infra notifications> [rustdesk-server_ynh] Thovi98 merged [pull request #42](https://github.com/YunoHost-Apps/rustdesk-server_ynh/pull/42) : Upgrade to v1.1.16
[17:27:39]
<Yunohost Git/Infra notifications> [rustdesk-server_ynh] Thovi98 deleted branch ci-auto-update-1.1.16
[17:27:40]
<Yunohost Git/Infra notifications> [rustdesk-server_ynh] Thovi98 opened [pull request #43](https://github.com/YunoHost-Apps/rustdesk-server_ynh/pull/43) : Testing
[17:27:41]
<Yunohost Git/Infra notifications> [rustdesk-server_ynh] Thovi98 merged [pull request #43](https://github.com/YunoHost-Apps/rustdesk-server_ynh/pull/43) : Testing
[18:02:58]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar merged [pull request #896](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/896) : Testing
[18:03:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed to master: Testing (#896) ([bcba5c3e](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/bcba5c3e41f1641c982d0240a72fede6b763bb7a))
[18:06:58]
<m606> i can't find track of it... but I remember having this conversation before, and i kind of remember there was something more than purely declarative. Like something a bit secondary for an other issue I had encountered that wouldn't work the same when sso is false. For my last issue above I can't say whether sso = false does change something or not.
[18:08:47]
<m606> Other topic, would you think of a way to know inside a given helper whether they are other config helpers left to run ?
I need to restart systemd service after changing settings, but if several settings are changed, I'll rather restart the service after saving all the settings.
[22:49:21]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 93 pending update PRs
- 13 new apps PRs: cypht, esphome, forgejo_runner, jackett, ketesa, kresus, librespeed, nodebb, ollama, pluxml, shields, teampass, wekan
- 8 failed apps updates: automad, dmarcguard, loki, nocodb, phpsysinfo, pocket-id, vikunja, youtarr
See the full log here: https://paste.yunohost.org/raw/vepaneyiwa
Autoupdate dashboard: https://apps.yunohost.org/dash?filter=autoupdate
[23:19:22]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #3558](https://github.com/YunoHost/apps/pull/3558) : Add EmDash to wishlist