GKaszewski/movies-diary
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: idempotency guard on wrapup handler to prevent duplicate processing
Some checks failed
CI / Check / Test (push) Failing after 42s
Details

Browse Source
This commit is contained in:
Gabriel Kaszewski
2026-06-03 01:01:50 +02:00
parent 83103e5eb5
commit e905842b62
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
crates/application/src/wrapup/handle_requested.rs
View File

@@ -13,6 +13,13 @@ pub async fn execute(
start_date: chrono::NaiveDate, start_date: chrono::NaiveDate,
end_date: chrono::NaiveDate, end_date: chrono::NaiveDate,
) -> Result<(), DomainError> { ) -> Result<(), DomainError> {
if let Ok(Some(rec)) = ctx.repos.wrapup_repo.get_by_id(&wrapup_id).await {
if rec.status == WrapUpStatus::Ready || rec.status == WrapUpStatus::Generating {
tracing::debug!("wrapup {} already {}, skipping", wrapup_id.value(), if rec.status == WrapUpStatus::Ready { "ready" } else { "generating" });
return Ok(());
}
}
ctx.repos ctx.repos
.wrapup_repo .wrapup_repo
.update_status(&wrapup_id, &WrapUpStatus::Generating, None) .update_status(&wrapup_id, &WrapUpStatus::Generating, None)