Skip to content
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

Add support for additional S3-compatible providers #399

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

caesay
Copy link

@caesay caesay commented Dec 1, 2021

This should be a backwards compatible change allowing people to use BackBlaze B2 or Linode with the aws-sdk (as they offer S3 endpoints).

The key change here is allowing an endpoint in the config instead of the region. I've also added support for the credentials to be supplied via the config. This is optional, and if credentials are not specified in the config, it will fall back to the previous logic.

@github-actions
Copy link

github-actions bot commented Mar 8, 2022

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale label Mar 8, 2022
@caesay
Copy link
Author

caesay commented Mar 8, 2022

It's only stale because it hasn't been commented on by a maintainer

@filipechagas filipechagas added triage and removed stale labels Mar 8, 2022
@lorenzo132
Copy link

This has been working stable for me for the past 1 year. i feel like this could be merged

@palvarez89
Copy link

Hey, thanks for this patch! I've been playing with it, and to get it working with Ceph S3 you also need the s3ForcePathStyle setting. Something like config.s3ForcePathStyle=true

This way, the bucket url is <endpoint>/<bucket> and not <bucket>.<endpoint>
Mentioning this here just in case is useful.

palvarez89 pushed a commit to palvarez89/haste-server that referenced this pull request Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants