Development #18785
ozwillo: finir...
0%
Description
1. ajoute des logs de debug pour qu'on y voit plus clair
2. remplace les simples subprocess.check_call() par une réimplémentation de check_call() qui log les erreurs (j'avais demandé à JB mais c'est parti dans /dev/null je pense)
3. amélioration au README.rst pour qu'on puisse le lire
4. amélioration au packagin pour que le script pour a2 et les templates de site d'example soit dedans
Fichiers
Révisions associées
ozwillo: log subprocess status, stdout and stderr on failures (#18785)
Also do not create the admin if the OIDC provider creation failed.
ozwillo: use NamedTemporaryFile for the recipe file (#18785)
ozwillo: improve README.txt (#18785)
- give a clear installation procedure.
- improve formatting.
ozwillo: include README.rst, scripts and site templates in distribution (#18785)
ozwillo: rename check_call() to run_command() (#18785)
ozwillo: validate destruction requests using OZWILLO_DESTRUCTION_SECRET (#18785)
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
2. remplace les simples subprocess.check_call() par une réimplémentation de check_call() qui log les erreurs (j'avais demandé à JB mais c'est parti dans /dev/null je pense)
Ça loggue mais en cas d'erreur ça continue l'exécution, contrairement aux check_call. (si ce comportement est souhaité, ok, mais que la fonction soit alors nommée différemment). (sur celui-là, aussi, réduire la longueur du message de commit)
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Oui c'est voulu que ça continue mais je vais quand même renvoyer un booléen parce que les deux derniers appels dépendent l'un de l'autre (ce qui n'est pas le cas de l'initialisation de combo, passerelle et w.c.s.) et là je m'en servirai.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Fichier 0003-ozwillo-use-NamedTemporaryFile-for-the-recipe-file-1.patch 0003-ozwillo-use-NamedTemporaryFile-for-the-recipe-file-1.patch ajouté
- Fichier 0002-ozwillo-log-subprocess-status-stdout-and-stderr-on-f.patch 0002-ozwillo-log-subprocess-status-stdout-and-stderr-on-f.patch ajouté
- Fichier 0001-ozwillo-augment-debug-logs-18785.patch 0001-ozwillo-augment-debug-logs-18785.patch ajouté
- Fichier 0005-ozwillo-include-README.rst-scripts-and-site-template.patch 0005-ozwillo-include-README.rst-scripts-and-site-template.patch ajouté
- Fichier 0004-ozwillo-improve-README.txt-18785.patch 0004-ozwillo-improve-README.txt-18785.patch ajouté
- Patch proposed changé de Non à Oui
Voilà au passage j'ai aussi remplacé l'utilisation de mkstemp() par NamedTemporaryFile qui se nettoie tout seul.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Nouveau à Résolu (à déployer)
Mis à jour par Frédéric Péters il y a plus de 6 ans
(si ce comportement est souhaité, ok, mais que la fonction soit alors nommée différemment)
Raté, tant pis.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Ça renvoie un booléen donc nommer la fonction check_call()
ne me parait pas aberrant. Maintenant ta remarque concerne peut-être le fait que le comportement n'est pas complètement identique à subprocess.check_call()
.
Mis à jour par Frédéric Péters il y a plus de 6 ans
... identique à subprocess.check_call().
Oui, quand on se trouve à lire du code, à voir un appel à check_call, c'est reposant d'imaginer que le fonctionnement de cet appel correspond à celui de la bibliothèque standard.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Fichier 0001-ozwillo-rename-check_call-to-run_command-18785.patch 0001-ozwillo-rename-check_call-to-run_command-18785.patch ajouté
Patch pour te permettre un repos mérité :)
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Fichier 0001-ozwillo-validate-destruction-requests-using-OZWILLO_.patch 0001-ozwillo-validate-destruction-requests-using-OZWILLO_.patch ajouté
Encore un bout, JB a pas testé vraiment la validation des signatures (il mettait le même secret pour les deux clés et oh surprise ça marchait).
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Résolu (à déployer) à Nouveau
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Nouveau à Solution déployée
ozwillo: augment debug logs (#18785)