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 features to the s3 driver #18
Add features to the s3 driver #18
Conversation
The secure flag will be true by default and will change the s3 endpoint of the region to http instead of https when selected as false. The main benefits of running with secure being false is that it apparently has a roughly 33% performance boost (even on pure data transfer, not only connection setup which is what I would have expected).
9e4c197
to
416808b
Compare
v4auth will default to true and if the frankfurt (eu-central-1) region is selected with v4auth set to false explicitly, the driver will error out upon initialization.
416808b
to
a0ef0d6
Compare
e12d727
to
887e22d
Compare
Thanks a lot @AndreyKostov ! |
shouldEncrypt, err := strconv.ParseBool(encrypt) | ||
if err != nil { | ||
return nil, err | ||
encryptBool := true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
encrypt
should default to false
like the semantic defined in the README. Also, you should remove this from the list of environment variables required to run the tests.
8ed22bb
to
d63f59a
Compare
@AndreyKostov github seems to be working correctly again, so try re-triggering the circle build. |
d63f59a
to
bc9509d
Compare
LGTM |
LGTM! |
Add features to the s3 driver
Do not merge yet, as this is blocked on the v4 auth option.