• Joined on 2026-05-18
Herald commented on pull request qpismont/herald#2 2026-06-06 18:53:34 +02:00
started gitea api impl

La PR introduit une architecture plus complète avec un canal asynchrone entre l'API et le bot, une gestion robuste des erreurs et le passage à rustls pour le TLS, ce qui améliore la sécurité en éliminant les dépendances OpenSSL. Plusieurs améliorations de sécurité sont bienvenues : limitation de taille du diff, timeouts configurés, et utilisation d'en-têtes Authorization par défaut. Toutefois, quelques points pourraient être renforcés : validation de l'origine des URLs récupérées via le webhook, protection contre les valeurs extrêmes de timeout, et gestion correcte des noms de fichiers sans extension pour la coloration syntaxique. Dans l'ensemble, la qualité du code est bonne pour un travail en cours, avec une attention particulière à la sécurité via le choix des bibliothèques et la limitation des ressources.