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

UploadPartCommand is returning a checksum while I said in CreateMultipartUploadCommand I want a FULL_OBJECT checksum check #6900

Open
3 of 4 tasks
kasir-barati opened this issue Feb 26, 2025 · 1 comment
Assignees
Labels
bug This issue is a bug. needs-triage This issue or PR still needs to be triaged.

Comments

@kasir-barati
Copy link

Checkboxes for prior research

Describe the bug

So I have a NestJS app, there I am using @aws-sdk/s3-client to upload a file in multiple part. The thing is that UploadPart is generating a checksum for each UploadPartCommand. So can you kindly confirm if this is a bug?

I mean I also suspect that I am doing something wrong, but my console.logs are not saying that

Regression Issue

  • Select this option if this issue appears to be a regression.

SDK version number

@aws-sdk/[email protected]

Which JavaScript Runtime is this issue in?

Node.js

Details of the browser/Node.js/ReactNative version

v22.12.0

Reproduction Steps

  1. Clone the repo
  2. cd microservices/grpc.
  3. pnpm i --frozen-lockfile.
  4. cp apps/file-upload/.env.example apps/file-upload/.env.
  5. Change the env variables to the ones you get from the AWS and remove the OBJECT_STORAGE_ENDPOINT. It should be used only for local testing with Minio.
  6. nx serve file-upload.
  7. nx e2e file-upload-e2e.

Observed Behavior

It says something like the calculated CRC32 does not match the one specified in the CompleteMultipartUploadCommand.

Expected Behavior

Just complete the multipart upload.

Possible Solution

Dunno.

Additional Information/Context

  • Here is where you can see the UploadPartCommand is returning a checksum for each part.
  • While you can clearly see that I have said please use a FULL_OBJECT checksum check here. BTW feel free to add more console.logs to see if I am setting those configs correctly.
  • And here is where I try to complete the file upload.

BTW if you're interested in my test, you can find it here.

@kasir-barati kasir-barati added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Feb 26, 2025
@kasir-barati
Copy link
Author

BTW here is the simplified version which works: https://github.com/kasir-barati/bugs/blob/aws-s3-multipart-completion-error/simplified.ts

But I am doing the exact same thing in my NestJS app and IDK why it is failing.

@aBurmeseDev aBurmeseDev self-assigned this Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. needs-triage This issue or PR still needs to be triaged.
Projects
None yet
Development

No branches or pull requests

2 participants