Skip to content
This repository was archived by the owner on Feb 4, 2025. It is now read-only.

fix(server): file persistence for local upload provider - #610

Open
raymon-zhang wants to merge 3 commits into
redpwn:masterfrom
raymon-zhang:master
Open

fix(server): file persistence for local upload provider#610
raymon-zhang wants to merge 3 commits into
redpwn:masterfrom
raymon-zhang:master

Conversation

@raymon-zhang

Copy link
Copy Markdown

Fix:

  • create a data/rctf-uploads folder in the install script
  • in the docker-compose file, create a docker bind mount from the data/rctf-uploads folder to /app/uploads in the rctf
  • in the local upload provider, save and load uploadMap from the uploadMap.json file

@chen-robert
chen-robert requested a review from ginkoid July 25, 2023 12:26

@ginkoid ginkoid left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if two rctf instances use the same uploads directory, we want them to be able to share uploads without a restart. can you do something similar to this? https://github.com/redpwn/rctf/blob/next/packages/server/src/providers/uploads/local/index.ts

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants