-
Notifications
You must be signed in to change notification settings - Fork 670
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
errors due to lack of unsupported checksums are not clear #3005
Comments
Third issue I found: if I don't specify any checksum algorithm or checksum type at all, the CompleteMultipartUpload will fail. Setup:
HTTP log for UploadPart shows:
The HTTP log is identical to the UploadPart input for
If I specified CRC64NVME, the error message is:
The former error message indicates a regression that I also called out in #2960 (comment). |
CRC64-NVME doesn't seem to be working, tracked in aws/aws-sdk-go-v2#3005. If the original CreateMultipartUploadInput doesn't specify any checksum algorith, `xy3.Upload` will choose CRC32 by default. If it didn't do this, the CompleteMultipartUpload call would fail due to issue tracked in #1.
These should all be OBE when we merge #3002. |
Acknowledgements
go get -u github.com/aws/aws-sdk-go-v2/...
)Describe the bug
My setup looks like this:
With this setup, I run into the first error:
No HTTP request was made, which is probably because the parsing fails due to https://github.com/aws/aws-sdk-go-v2/blob/061fd1b6d9940c6b37974e6aa64a5f9dc4ee1dc4/service/internal/checksum/algorithms.go not having support for CRC64NVME in the
Parse*
functions.I understand CRC64NVME is not supported at composite level (so there's really no point in specifying it during UploadPart), so when I try this:
This is what I get:
So looks like if I don't specify ChecksumAlgorithm as part of UploadInput, Go SDK will attempt to attach
x-amz-checksum-crc32
to it? Which then causes the server to fail the request with a 400.Am I flat out unable to use CRC64NVME at all during the initial CreateMultipartUpload request until #2985 (S3 object integrity: support CRC64NVME #2985 ) is addressed?
Regression Issue
Expected Behavior
The UploadPart request should complete successfully.
Current Behavior
The UploadPart request fails with or without specifying checksum.
Reproduction Steps
First problem:
Second problem:
Possible Solution
No response
Additional Information/Context
No response
AWS Go SDK V2 Module Versions Used
Compiler and Version used
go version go1.23.6 linux/amd64
Operating System and version
5.15.167.4-microsoft-standard-WSL2
The text was updated successfully, but these errors were encountered: