vinzenz/hyperhive
0
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

forge_notify: check mark-read HTTP status, warn on non-2xx

Browse source
This commit is contained in:
damocles 2026-05-20 21:22:53 +02:00
parent ddd0248619
commit a13f63f30c
1 changed files with 10 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

12
hive-ag3nt/src/forge_notify.rs
View file

@ -267,13 +267,21 @@ async fn poll_once(client: &reqwest::Client, forge_url: &str, token: &str, socke
// Mark as read only after successful delivery so a failed-delivery // Mark as read only after successful delivery so a failed-delivery
// notification resurfaces on the next poll tick. // notification resurfaces on the next poll tick.
let mark_url = format!("{forge_url}/api/v1/notifications/threads/{id}"); let mark_url = format!("{forge_url}/api/v1/notifications/threads/{id}");
if let Err(e) = client match client
.patch(&mark_url) .patch(&mark_url)
.header("Authorization", format!("token {token}")) .header("Authorization", format!("token {token}"))
.send() .send()
.await .await
{ {
debug!(%id, error = ?e, "forge_notify: mark-read failed (non-fatal)"); Err(e) => {
warn!(%id, error = ?e, "forge_notify: mark-read request failed — notification will resurface");
}
Ok(r) if !r.status().is_success() => {
warn!(%id, status = %r.status(), "forge_notify: mark-read returned non-2xx — notification will resurface");
}
Ok(_) => {
debug!(%id, "forge_notify: marked read");
}
} }
} }
} }