-
Notifications
You must be signed in to change notification settings - Fork 676
WARN Response has no supported checksum. Not validating response payload. for simple S3 get #3032
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
Comments
This warning appeared to start just firing randomly after we upgraded to the latest version of aws-sdk-go-v2 and called GetObject as well -- this strikes me as a regression since we didn't see this before -- agree that it's not entirely clear what this error means from the docs. Seems like this might have something to do with the aws-sdk-go-v2 not supporting newer checksum algorithms that S3 supports? |
This warning is triggered for error responses. Make sure that the request actually correctly responds with the S3 object. Supposedly the warning for error responses is intentional and my ticket about this problem was closed #3020 |
Sounds like from the issue that it's also triggered for 404s, which I think might be what our particular use-case is hitting. We are handling the 404 in the error responses but seems like the warning still gets logged. Is the recommendation just to disable that warning log in the case where 404s are expected? I'm a little skeptical of how "safe" that is if that log is there for a reason and is intended to actually tell us something |
#3034 should turn off the validation skip warning if no object is fetched in response intentionally or non-exist error is returned |
Just to clarify, I am getting this warning on successful [200] fetch of a json file from S3. |
We support every algorithm in Go now, that's not the issue. You're still going to get this on successful object reads if the object did not have a checksum to begin with. Post-integrity changes, S3 is tagging all objects w/ crc64s by default. Objects that existed before the integrity rollout did not get retconned checksums. |
Same issue, these are my dependencies
|
Acknowledgements
go get -u github.com/aws/aws-sdk-go-v2/...
)Describe the bug
This looks like a regression of #1606
Using
github.com/aws/aws-sdk-go-v2/service/s3 v1.78.1
a simple getObject request causes a warning:I understand that there is now a setting to turn of these warnings generally, but that's rather like cutting the wire to the warning light. Either this warning is a false positive and needs fixing, or else better instructions need to be given here whatever developer error causes the error.
Regression Issue
Expected Behavior
Be able to perform a simple get request without warnings by following the documentation
Current Behavior
Generates a warning
Reproduction Steps
Follow instructions for a GetObject request in documentation.
Possible Solution
No response
Additional Information/Context
No response
AWS Go SDK V2 Module Versions Used
Compiler and Version used
go version go1.24.0 darwin/amd64
Operating System and version
MacOS 15
The text was updated successfully, but these errors were encountered: