Skip to content

Allow storing files in S3, instead of just on-disk #30

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

steiza
Copy link
Owner

@steiza steiza commented Jan 18, 2021

Allow storing files in S3, instead of just on-disk

These changes are backwards-compatible, so it will first check S3 and fall back to on-disk if it cannot find the file.

There are two new config options:

  • s3_bucket_name: if this is set, we save new files here instead of on-disk. We assume you have the region and AWS credentials configured in your environment
  • doc_root: this defaults to "/file", but if you can also set this to something like "http://.s3-website-us-west-2.amazonaws.com/" to use a S3 bucket configured for static website hosting

To migrate files from on-disk to S3, you'd take the numbered directories in "storage/docs/" and put them at the top level of the S3 bucket.

Also update dependencies to more modern versions.

These changes are backwards-compatible, so it will first check S3 and
fall back to on-disk if it cannot find the file.

There's a new config option `s3_bucket_name`. We assume you have the
region and AWS credentials configured in your enviornment.

Also update dependencies to more modern versions.
These changes are backwards-compatible, so it will first check S3 and
fall back to on-disk if it cannot find the file.

There's a new config option `s3_bucket_name`. We assume you have the
region and AWS credentials configured in your enviornment.

Also update dependencies to more modern versions.
Base automatically changed from master to main February 11, 2021 22:20
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