Skip to content

Fix: use a unique temporary file for the error CSV export - #675

Open
stonebuzz wants to merge 1 commit into
mainfrom
fix_injection_export
Open

stonebuzz wants to merge 1 commit into
mainfrom
fix_injection_export

Conversation

@stonebuzz

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes #N/A
  • The "Export the log" action wrote the rejected lines to a file named after the uploaded CSV, so two exports using the same file name could overwrite and remove each other's file.
  • The export now writes to a unique temporary file in the plugin upload directory and is always downloaded as Errors.csv.

Screenshots (if appropriate):

@stonebuzz
stonebuzz force-pushed the fix_injection_export branch 7 times, most recently from 8915739 to 055831f Compare September 30, 2026 13:02
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.

1 participant