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

modernize s3/aws interface #137

Merged
merged 1 commit into from
Dec 3, 2019
Merged

modernize s3/aws interface #137

merged 1 commit into from
Dec 3, 2019

Conversation

chassell
Copy link

@chassell chassell commented Dec 2, 2019

This is to handle the "filter=" URL attribute (in place of "id=" and others) as well as to pass on a 'content-md5' header with a prepared MD5 hash for a lifecycle query. (? limited knowledge here ?).

This was proven to solve our issues with AWS in several ways.

device-src/s3.c Show resolved Hide resolved
@chassell
Copy link
Author

chassell commented Dec 3, 2019 via email

@djmitche
Copy link
Contributor

djmitche commented Dec 3, 2019

Ah, got it.

@chassell chassell merged commit 6d40f37 into zmanda:3_5 Dec 3, 2019
@chassell chassell deleted the 3_5.s3-fixes branch December 3, 2019 18:11
@jamshid
Copy link

jamshid commented May 25, 2023

I'm seeing newer amanda backups to S3-compatible object storage send a Content-MD5 header and Content-length: 44 on GETs. Why would it send a body on a GET to S3? Is it possible that is from this change?

The requests fail with 403 SignatureDoesNotMatch for me but maybe it works against AWS S3.
Amanda does not seem to include the Content-MD5 header in the V2 signature of a GET nor does it validate it matches the body of the GET or the requested object. So I don't see why it's being included on GETs.

BEGIN REQUEST: GET '/amanda-backups/nightly1/slot-foo-tapestart' ...
Authorization: AWS 8b65b6X:ionZiFuzyW4T8j2nVJ56R78ASGI=
Content-Length: 44
Date: Sun, 21 May 2023 14:00:09 GMT
Content-MD5: 3tjrjT2tg/51Vteuk7ohIQ==

Or maybe I'm seeing the libcurl-related bug #213.

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