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

Region is needed when using custom s3 endpoint #1304

Open
kkettinger opened this issue Oct 8, 2024 · 3 comments
Open

Region is needed when using custom s3 endpoint #1304

kkettinger opened this issue Oct 8, 2024 · 3 comments
Labels
C-upstream Category: Issue for a dependency, will be fixed upstream

Comments

@kkettinger
Copy link

Currently testing out the hetzner object storage beta with this config:

[repository]
repository = "opendal:s3"
password = "XXX"

[repository.options]
access_key_id = "XXX"
secret_access_key = "XXX"
bucket = "bucket-name"
root = "/"
endpoint = "https://fsn1.your-objectstorage.com"
# region = "us-east-1"

When not using region, opendal fails:

error: backend openDAL Backend cannot be loaded: ConfigInvalid (permanent) at Builder::build, context: { service: s3 } => region is missing. Please find it by S3::detect_region() or set them in env.

When commenting out the region key, it works.

@github-actions github-actions bot added the S-triage Status: Waiting for a maintainer to triage this issue/PR label Oct 8, 2024
@aawsome aawsome added C-upstream Category: Issue for a dependency, will be fixed upstream and removed S-triage Status: Waiting for a maintainer to triage this issue/PR labels Oct 10, 2024
@aawsome
Copy link
Member

aawsome commented Oct 10, 2024

Thanks for opening this issue @kkettinger

This is actually how opendal is working, see https://opendal.apache.org/docs/rust/opendal/services/struct.S3.html.

We can't fix this in rustic. If you want to have it fixed, can you please open an issue for opendal: https://github.com/apache/opendal/issues/new/choose

Thanks!

@simonsan
Copy link
Contributor

// Set the region. This is required for some services, if you don't care about it, for example Minio service, just set it to "auto", it will be ignored.

Probably worth to keep it, and set it to 'auto', then?

@kkettinger
Copy link
Author

kkettinger commented Oct 11, 2024

// Set the region. This is required for some services, if you don't care about it, for example Minio service, just set it to "auto", it will be ignored.

Probably worth to keep it, and set it to 'auto', then?

Setting to "auto" works!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-upstream Category: Issue for a dependency, will be fixed upstream
Projects
None yet
Development

No branches or pull requests

3 participants