[10:34:49]
<Yunohost Git/Infra notifications> [cli/main] ruff, isort - Félix Piédallu
[10:34:49]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 2 commits to main ([d281c47bff56...026306cca0b9](https://github.com/YunoHost/cli/compare/d281c47bff56...026306cca0b9))
[10:34:49]
<Yunohost Git/Infra notifications> [cli/main] Use colored instead of custom managed color codes - Félix Piédallu
[13:32:41]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: Add github workflows lint / format ([592cc600](https://github.com/YunoHost/cli/commit/592cc600eb1763c8590638a6764987b611e8b24a))
[13:34:01]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: Run lint on pull requests too ([db95f4fa](https://github.com/YunoHost/cli/commit/db95f4fa62fdb741c5eb74d3d35c599fa4dcc293))
[15:32:59]
<Salamandar> https://aria.im/_bifrost/v1/media/download/AQ76L1R7vSc5XzD25zZybbb_Qj_hQsn-UU3RD1JGhbYnxFV9iqYNT9h99vNNYryUA7VM9tLoJGWWBnxlAjUb-fRCeW1JPRcwAG1hdHJpeC5vcmcvTUp5REZXemhmdmN0SHRWRWFQak5ISXRi
[15:33:02]
<Salamandar> 😍
[15:33:05]
<Salamandar> (yeah running remotely)
[15:35:39]
<Salamandar> Hmmm the debug logs aren't pushed to the SSE ?
[15:36:00]
<Salamandar> This kind of stuff:
[15:36:04]
<Salamandar> https://aria.im/_bifrost/v1/media/download/AaqacSW_h8_R7J_Yn9EBoyllHpcMIFb1rFg4Se2Vi479_2-XKKRzTBujIcAnG5J467C4UlWGMgTv76BZht6aa25CeW1Jal4wAG1hdHJpeC5vcmcvU0l5aEFNTGtOcVlMYUFMREdUTUZPVFVT
[15:37:07]
<Aleks (he/him/il/lui)> not the DEBUG stream yeah
[15:37:25]
<Salamandar> ah, that's too bad :/
[15:40:53]
<Salamandar> although that makes sense that the DEBUG stream is only sent to the one making the request maybe
[15:43:32]
<Aleks (he/him/il/lui)> it's not sent to the API client at all, idk, that was way too much info for the webclient, but yeah i suppose for CLI we'd like to have the `--debug` option
[15:43:57]
<Aleks (he/him/il/lui)> it's relatively simple to tweak the logging configuration but idk i would expect it may overload the SSE buffer
[15:44:19]
<Aleks (he/him/il/lui)> and maybe some small functions should be defined somewhere in the logging / SSE logistic
[15:45:23]
<Salamandar> yeah that could be a secondary stream maybe
[15:47:06]
<Salamandar> maybe an endpoint like `/yunohost/api/logs/<id>/stream` that would be a raw text stream (not SSE)
[15:57:36]
<Aleks (he/him/il/lui)> yeah i've been wondering how doable it is to stream raw text in nginx
[15:57:43]
<Aleks (he/him/il/lui)> but then we need to check authentication
[15:58:49]
<Aleks (he/him/il/lui)> so then we end up having to implement some sort of tail -f in python and detect when the file is not being used in write mode by another process idk
[16:39:22]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: Improve request / sse perf ([6ba5c6a4](https://github.com/YunoHost/cli/commit/6ba5c6a4a613d263160c7cc814611b51b218ed6a))
[16:39:39]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: Handle SSE log end type ([46a57715](https://github.com/YunoHost/cli/commit/46a57715c7548f2de9cf4370c11725adbd59db68))
[16:40:13]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: Linting, reformatting etc ([0a01333c](https://github.com/YunoHost/cli/commit/0a01333cb243acfd81b8e82ce989395594273cf7))
[16:41:21]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: Add ruff to dev deps ([b0c69920](https://github.com/YunoHost/cli/commit/b0c6992042cf7025b5f1d1c889752db6c9f97727))
[16:44:12]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: ruff: change line length ([b7c2518c](https://github.com/YunoHost/cli/commit/b7c2518ccdf129787e26a46240d3db227ec519f9))
[19:01:02]
<Yunohost Git/Infra notifications> [issues] cryoteeth [commented](https://github.com/YunoHost/issues/issues/2544#issuecomment-2881237085) on [issue #2544](https://github.com/YunoHost/issues/issues/2544) Automatic DNS fails to configure Porkbun DNS records due to incorrect API hostname.: I just ran into the same issue - I’m pretty sure it’s because Porkbun’s API hostname is now api.porkbun.com and not p...
[19:01:09]
<Yunohost Git/Infra notifications> [issues] cryoteeth [commented](https://github.com/YunoHost/issues/issues/2544#issuecomment-2881237085) on [issue #2544](https://github.com/YunoHost/issues/issues/2544) Automatic DNS fails to configure Porkbun DNS records due to incorrect API hostname.: I just ran into the same issue - I’d guess it’s because Porkbun’s API hostname is now api.porkbun.com and not porkbun...
[19:45:35]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 4 commits to main ([c208d737cdf5...658f59454704](https://github.com/YunoHost/cli/compare/c208d737cdf5...658f59454704))
[19:45:35]
<Yunohost Git/Infra notifications> [cli/main] Silent non fixable RUF012 - Félix Piédallu
[19:45:35]
<Yunohost Git/Infra notifications> [cli/main] Manage properly the SSE task termination - Félix Piédallu
[19:45:35]
<Yunohost Git/Infra notifications> [cli/main] Handle linter issues - Félix Piédallu
[19:49:01]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to main: Add todos to readme ([036c8025](https://github.com/YunoHost/cli/commit/036c8025274e0aba0bc7a8583a0bf8577dcda88d))
[19:49:02]
<Yunohost Git/Infra notifications> [cli] Salamandar pushed 1 commit to ruff: Add ruff config ([43967cab](https://github.com/YunoHost/cli/commit/43967cab6b486ca3daf4244c1bffabc47b4e9db3))
[19:49:03]
<Yunohost Git/Infra notifications> [cli] Salamandar created new branch ruff
[20:15:59]
<Salamandar> So, there's still an issue on the domain list :
```
░▒▓█ time uv run python3 .venv/bin/ynh domain list
domains:
- salamandar.fr
- cells.salamandar.fr
- cinny.salamandar.fr
- cloud.salamandar.fr
- collabora.salamandar.fr
- dendrite.salamandar.fr
- elouc.salamandar.fr
- feedback.salamandar.fr
- fider.salamandar.fr
- git.salamandar.fr
- hass.salamandar.fr
- olivetin.salamandar.fr
- pad.salamandar.fr
- pad2.salamandar.fr
- papiers.salamandar.fr
- paste.salamandar.fr
- pauline.salamandar.fr
- photos.salamandar.fr
- photoview.salamandar.fr
- recettes.salamandar.fr
- synapse.salamandar.fr
- synapse-admin.salamandar.fr
- trilium.salamandar.fr
- vault.salamandar.fr
- vpn.salamandar.fr
- papier.yunohost.org
main: salamandar.fr
uv run python3 .venv/bin/ynh domain list 0,24s user 0,03s system 50% cpu 0,531 total
░▒▓█ time uv run python3 .venv/bin/ynh domain list --features mail_in
domains:
- salamandar.fr
- cells.salamandar.fr
- cinny.salamandar.fr
- cloud.salamandar.fr
- collabora.salamandar.fr
- dendrite.salamandar.fr
- elouc.salamandar.fr
- feedback.salamandar.fr
- fider.salamandar.fr
- git.salamandar.fr
- hass.salamandar.fr
- olivetin.salamandar.fr
- pad.salamandar.fr
- pad2.salamandar.fr
- papiers.salamandar.fr
- paste.salamandar.fr
- pauline.salamandar.fr
- photos.salamandar.fr
- photoview.salamandar.fr
- recettes.salamandar.fr
- synapse.salamandar.fr
- synapse-admin.salamandar.fr
- trilium.salamandar.fr
- vault.salamandar.fr
- vpn.salamandar.fr
- papier.yunohost.org
main: salamandar.fr
uv run python3 .venv/bin/ynh domain list --features mail_in 0,23s user 0,03s system 0% cpu 1:37,51 total
>>> elapsed time 1m38s
```
Not sure if there's an issue to track this
[20:19:46]
<Aleks (he/him/il/lui)> it's probably related to the fact that it goes through the config panel stuff which loads pydantic etc which has perf issues until pydantic v2 which is in rust
[20:20:42]
<Aleks (he/him/il/lui)> it's designed this way to handle the fact that there's a default value / we only store the diff in the config on disk if i remember correctly
[20:20:52]
<Aleks (he/him/il/lui)> but we could probably implement a function that bypasses this : reads the config, if the key's there, use it, otherwise read the raw config panel and get the default value
[20:28:13]
<Salamandar> hmmm
[20:28:17]
<Salamandar> alright
[20:28:43]
<Salamandar> not sure exactly what you mean here but okay, if it's just fixed by pydantic v2 why not
[20:45:03]
<tituspijean[m]> I dunno if that could be of interest. I've written a small script to remove all unused domains from a server:
```
#!/bin/bash
domains=$(yunohost domain list --output-as json | jq -e '.domains')
domains_used_by_apps=$(yunohost app list -f --output-as json | jq -e '[ .apps[].settings.domain ]')
unused_domains=$(jq -nr --argjson domains_used_by_apps "$domains_used_by_apps" --argjson domains "$domains" '$domains-$domains_used_by_apps | join(" ")')
echo "Here are the unused domains on your server:"
echo $unused_domains
echo "Do you wish to remove them (Yes) or quit (No, Q or Ctrl+C)?"
select strictreply in "Yes" "No"; do
relaxedreply=${strictreply:-$REPLY}
case $relaxedreply in
Yes | yes | y ) for d in $unused_domains; do yunohost domain remove $d; done; break;;
No | no | n | Quit | Q ) exit;;
esac
done
```
[20:45:30]
<tituspijean[m]> (however the remove command is slow af)
[20:50:51]
<Salamandar> well because of the mail_in mail_out issue, deleting a domain on my server takes ~ 10 minutes
[20:51:10]
<Salamandar> (well it's been 7 and still not finished…)