Suppress spurious CRI log warning for jobs with no events
The quire/push synthetic job has a slash in its id so is_safe_path_segment
rejects it, but it never has sh_events so there are no log reads to skip.
Only warn when the unsafe job actually has events that would be silently
dropped.
https://claude.ai/code/session_018vmf1YC15nBnN5fBFppv4G
diff --git a/quire-server/src/quire/web/handlers.rs b/quire-server/src/quire/web/handlers.rs
index da0cc5e..0e16ffa 100644
--- a/quire-server/src/quire/web/handlers.rs
+++ b/quire-server/src/quire/web/handlers.rs
@@ -208,7 +208,7 @@ pub async fn run_detail(
.map(Vec::as_slice)
.unwrap_or(&[]);
let job_dir = db::is_safe_path_segment(&job.job_id).then(|| job_dir_base.join(&job.job_id));
- if job_dir.is_none() {
+ if job_dir.is_none() && !job_events.is_empty() {
tracing::warn!(job_id = %job.job_id, "skipping CRI log reads for unsafe job_id");
}