Skip to content

chore: cleanup uploaded tmp files#1927

Merged
usingtechnology merged 2 commits into
bcgov:mainfrom
usingtechnology:chore/CCP-5189_uploadfilescleanup
Jul 7, 2026
Merged

chore: cleanup uploaded tmp files#1927
usingtechnology merged 2 commits into
bcgov:mainfrom
usingtechnology:chore/CCP-5189_uploadfilescleanup

Conversation

@usingtechnology

Copy link
Copy Markdown
Contributor

CCP-5189

Description

We have not been deleting files in our tmp directory after they have been uploaded and moved to object storage (s3). Track and cleanup what we can, add a scanner to do regular cleanups of the same dir.

Added a path to the tmp directory to isolate our files from other tmp files the system makes/needs.

Type of Change

fix (a bug fix)

test (add missing tests or correct existing tests)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have run the npm script lint on the frontend and backend
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • I have approval from the product owner for the contribution in this pull request

Further comments

Comment thread app/src/forms/file/middleware/virusScan.js Dismissed
@github-actions

This comment has been minimized.

@sonarqubecloud

sonarqubecloud Bot commented Jul 7, 2026

Copy link
Copy Markdown

@usingtechnology usingtechnology merged commit b41f3dd into bcgov:main Jul 7, 2026
7 checks passed
@usingtechnology usingtechnology deleted the chore/CCP-5189_uploadfilescleanup branch July 7, 2026 04: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.

2 participants