[15:58:03]
<Yunohost Git/Infra notifications> [yunohost] Axolotle pushed to enh-pydantic: refactor: validation: raise error if base schema is not found ([afe396cd](https://github.com/YunoHost/yunohost/commit/afe396cdb562cd775fb8ec20e77141eadc3d52cc))
[15:58:03]
<Yunohost Git/Infra notifications> [yunohost] Axolotle pushed to enh-pydantic: refactor: validation: assert PasswordConstraints redact is True ([d17aa406](https://github.com/YunoHost/yunohost/commit/d17aa406c0a39b35d6be286d94a6dce329d3d923))
[16:05:20]
<SveDec> Salut, j'ai tenté le `rebase`, mais il y avait beaucoup de conflits, et après mise à jour de mon fork Github, la PR est devenue le foutoir, donc je n'ai pas fait ce qu'il fallait ... je pense que je vais repartir d'une base propre, créer la branche depuis trixie, et repousser sur mon dépôt
[16:06:53]
<Aleks (he/him/il/lui)> j'ai pas eu le temps de me pencher sur la PR mais en vrai si c'est trop relou les conflis etc, jte dirais bien : garde `dev` (bookworm) en target et on gérera ça en fonction de si c'est mergeable rapidement ou pas
[16:09:46]
<SveDec> Ah ok, bah je nettoie le truc (j'avais gardé une copie du dépôt git propre en local) et remets la PR en ordre avec dev en cible, merci 👍️
[16:11:50]
<Yunohost Git/Infra notifications> [yunohost] SveDec edited [pull request #2246](https://github.com/YunoHost/yunohost/pull/2246): [enh] Add a user defined DNS resolvers setting
[16:14:18]
<SveDec> Ok c'est remis, j'ajoute juste un commit pour ajouter la clé manquante dans `locales/en.json` et je passe la PR en "ready for review"
[16:20:36]
<Aleks (he/him/il/lui)> https://aria.im/_bifrost/v1/media/download/Af0SVIVW-6quIeZ2fCIcZEZjJps0o3igVWuYr3icwcoc9z8aoAV5mINAFRwhzX2Dd1skFuuWHquBCx5Y33d_QQtCebdTYTuAAG1hdHJpeC5vcmcvU1ZyS29ObnFWRW9Qc0JYV3NJeU1jQXVE
[16:20:39]
<Aleks (he/him/il/lui)> https://i.imgflip.com/ag4tre.jpg
[16:21:22]
<Aleks (he/him/il/lui)> https://www.youtube.com/watch?v=tHKc-DBb_ME
[16:24:32]
<SveDec> Oui la regexp pour gérer les deux IPv4/IPv6 est un peu ... étendue 😅
[16:26:06]
<Yunohost Git/Infra notifications> [yunohost] SveDec just made [pull request #2246](https://github.com/YunoHost/yunohost/pull/2246) ready for review: [enh] Add a user defined DNS resolvers setting
[16:30:00]
<Salamandar> Ouais je suis d'accord c'est ok
[16:34:09]
<Yunohost Git/Infra notifications> [yunohost] alexAubin [commented](https://github.com/YunoHost/yunohost/pull/2246#discussion_r2653377480) on pull request #2246 [enh] Add a user defined DNS resolvers setting: suggestion "global_settings_setting_dns_custom_resolvers_enabled": "Use custom DNS resolvers", "global_settin...
[16:34:42]
<Yunohost Git/Infra notifications> [yunohost] alexAubin [commented](https://github.com/YunoHost/yunohost/pull/2246#discussion_r2653378457) on pull request #2246 [enh] Add a user defined DNS resolvers setting: suggestion # if no custom resolvers are set. # Custom resolvers can be defined in YunoHost settings.
[16:42:36]
<SveDec> Ça devrait pouvoir être intégré relativement rapidement en fonction de si le code s'intègre bien selon vous (merci pour la review Aleks (he/him/il/lui)), et selon ce que Salamandar avait en tête pour la refacto de la génération du fichier `/etc/resolv.dnsmasq.conf` ?
[16:44:48]
<Aleks (he/him/il/lui)> en vrai oui ça a l'air legit et mergeable rapidement, faudrait que je me pose un peu + et que je teste une fois idéalement, mais à part ça j'ai pas suivi l'histoire de refacto /etc/resolv.dnsmasq.conf
[16:47:14]
<SveDec> Réf à [ce message](https://matrix.to/#/!oUChrIGPjhUkpgjYCW:matrix.org/$qtoDpSgLrpHX4795jH4NifEqjbwhACq_zeI22seTdtQ?via=matrix.org&via=aria-net.org&via=beeper.com)
[16:49:55]
<axolotle> J'peux voir pour rajouter un type IP dans `enh-pydantic`, c'est dispo dans [pydantic](https://docs.pydantic.dev/latest/api/networks/#pydantic.networks.IPvAnyAddress)
[16:50:29]
<axolotle> (entre autres d'ailleurs)
[16:52:19]
<SveDec> Je n'ai fait que tester dans le conteneur Incus lors du dev en effet