fix: check session exists before completing inspection
Metadata
Current evaluation
Merged fix for a race condition where inspection completion overlaps with session timeout. Adds a session check before finalizing inspection, ignoring invalid results. Resolves #486. Noble CI failure fixed in PR #490.
Suggested action: —
No scores available.
Issue body
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
A session can be finished before an inspection fully complete. This can happen when a session timeouts at the same moment an inspection completes.
Make sure the session is still active before formally completing the inspection. If the session is not valid, the inspected artifact and the result of the inspection are ignored.
Fixes #486
FETCH-48
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged fix for a race condition where inspection completion overlaps with session timeout. Adds a session check before finalizing inspection, ignoring invalid results. Resolves #486. Noble CI failure fixed in PR #490. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #409 fix: prevent races in cached inspection results | fetch-service | merged | Merged fix preventing race conditions in cached inspection results. Artifacts are now added to the session only after asynchronous inspection completes, ensuring valid reuse during fast download retries. Approved by two reviewers, passed CI, and merged. |