Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@VitaliyDm
Copy link

@VitaliyDm VitaliyDm commented May 31, 2021

x4m and others added 30 commits November 22, 2019 10:14
Add WAL-E compatibility in GCS storages with multiple // in path
use wal-g/tracelog instead of tinsane/tracelog
use wal-g/tracelog instead of tinsane/tracelog
Merge pull request Tinsane#8 from reshke/master
Implement throttle retry is s3
remove redutant logging in s3
Update log level of skipped files message
* add ssh folder

* newline

Co-authored-by: Девятых Мария Алексеевна <[email protected]>
x4m and others added 26 commits October 15, 2020 21:58
Update default chunk size to be able to upload files over 1GiB
Improve GCS composition process in order to have an ability to upload large files
GCS: support customer-supplied encryption keys
set actual encryption key size for GCS Object handles
GCS: fix error handling on removing temporary chunks
Configure chunksize and retry logic for GCS uploader
Add compatibility with ListObjectsV1 S3 API
Fix the folder useListObjectsV1 reset on GetSubFolder() call
* Azure: Add SA endpoints for non AzurePublicCloud

* go mod tidy

* add defaultEnvName const, move switch in function

* Replace hardcoded values with library defined ones
Current version of Swift driver load the entire object in the buffer
that leads to high memory consumption and makes download of large
objects impossible.

Fixes wal-g/wal-g#862
Remove credentials provider settings to allow the AWS go SDK to pull credentials from Instance Profile/Metadata.
Per wal-g#45 (comment), remove the code rather than comment it out.
Remove credentials provider settings
s3/session.go Outdated
}

hostAddr := url.Parse(*config.Endpoint).Host
host, _, _ := net.SplitHostPort(hostAddr)

Choose a reason for hiding this comment

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

SplitHostPort may return an error, and it needs to be handled correctly.

@usernamedt
Copy link

usernamedt commented Jun 2, 2021

Hi! Thanks for your contribution!
First of all, please re-open your PR in this repo.
Also, it would be great to write some unit tests for this new functionality. A couple of tests for the getAWSRegion() will be enough.

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.