Skip to content

Allow overriding default AWS endpoint URL#373

Merged
bep merged 1 commit intobep:masterfrom
Rootz-LTD:customize-endpoint-url
Apr 25, 2023
Merged

Allow overriding default AWS endpoint URL#373
bep merged 1 commit intobep:masterfrom
Rootz-LTD:customize-endpoint-url

Conversation

@joona
Copy link
Contributor

@joona joona commented Apr 25, 2023

Adds endpoint-url option, which can be used to override endpoint URL for S3 client.

Setting custom endpoint URL allows you to use s3deploy for Cloudflare R2 deployments via S3 Compatibility API.

s3deploy -endpoint-url https://<ACCOUNT_ID>.r2.cloudflarestorage.com -region auto ...

Adds `endpoint-url` option, which can be used to provide endpoint URL
for S3 client.
@bep
Copy link
Owner

bep commented Apr 25, 2023

Cool, thanks -- I'll merge when the tests go green.

@codecov-commenter
Copy link

Codecov Report

Merging #373 (5d1ded5) into master (abd240a) will decrease coverage by 12.53%.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff             @@
##           master     #373       +/-   ##
===========================================
- Coverage   79.76%   67.24%   -12.53%     
===========================================
  Files          10       10               
  Lines        1028     1038       +10     
===========================================
- Hits          820      698      -122     
- Misses        173      307      +134     
+ Partials       35       33        -2     
Impacted Files Coverage Δ
lib/config.go 82.67% <100.00%> (-3.40%) ⬇️
lib/session.go 85.00% <100.00%> (-15.00%) ⬇️

... and 4 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bep bep merged commit 250d88f into bep:master Apr 25, 2023
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.

3 participants