Skip to content

fix: avoid temp-file collisions during concurrent spreadsheet saves#18

Merged
cs50victor merged 1 commit intomainfrom
fix/issue-615-save-temp-race
Mar 24, 2026
Merged

fix: avoid temp-file collisions during concurrent spreadsheet saves#18
cs50victor merged 1 commit intomainfrom
fix/issue-615-save-temp-race

Conversation

@interpreterwork
Copy link
Copy Markdown
Collaborator

@interpreterwork interpreterwork commented Mar 21, 2026

Summary

  • generate unique temp artifact names per /api/save request (temp_changes_<uuid>.bin, params_save_<uuid>.xml)
  • ensure the hash-specific output directory exists before writing temp artifacts
  • keep cleanup scoped to each request's temp files to avoid cross-request deletion

Why

The save endpoint reused shared temp filenames, so overlapping autosaves could delete each other's temp files and return conversion failures.

@cs50victor cs50victor merged commit 31046ce into main Mar 24, 2026
4 checks passed
@cs50victor cs50victor deleted the fix/issue-615-save-temp-race branch March 24, 2026 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants