Starting impl Sentry and tracing #3

Merged
qpismont merged 6 commits from tracing into main 2026-06-10 20:27:26 +02:00
Showing only changes of commit 3d751ae6c6 - Show all commits
+10 -2
View File
@@ -61,6 +61,14 @@ impl Bot {
let mut tasks = tokio::task::JoinSet::new(); let mut tasks = tokio::task::JoinSet::new();
while let Some(wb) = rx.recv().await { while let Some(wb) = rx.recv().await {
// Drain completed tasks to avoid the JoinSet growing unbounded
while let Some(res) = tasks.try_join_next() {
if let Err(e) = res {
error!("Task panicked: {e}");
}
}
info!(queued = rx.len(), active = tasks.len(), "Webhook received");
let permit = sem.clone().acquire_owned().await?; let permit = sem.clone().acquire_owned().await?;
let self_clone = self.clone(); let self_clone = self.clone();
qpismont marked this conversation as resolved
Review

Les commentaires sont en français, alors que le reste du code est en anglais. Pour la cohérence du projet, il est conseillé de rédiger tous les commentaires en anglais (ou dans une seule langue).

Les commentaires sont en français, alors que le reste du code est en anglais. Pour la cohérence du projet, il est conseillé de rédiger tous les commentaires en anglais (ou dans une seule langue).
@@ -70,8 +78,8 @@ impl Bot {
}); });
} }
// Le channel est fermé : on attend que les tâches en cours se terminent // When all webhook tasks have completed, we can safely exit
// proprement avant de rendre la main // properly before returning
while let Some(res) = tasks.join_next().await { while let Some(res) = tasks.join_next().await {
if let Err(e) = res { if let Err(e) = res {
error!("Task panicked: {e}"); error!("Task panicked: {e}");