Skip to content

Typo in property name S3ListObjectsResponse.contents.checksumAlgorithm**E** #19142

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

Open
tabarra opened this issue Apr 20, 2025 · 0 comments
Open
Labels
bug Something isn't working good first issue Something that would be good for new contributors s3

Comments

@tabarra
Copy link

tabarra commented Apr 20, 2025

What version of Bun is running?

1.2.10+db2e7d7f7

What platform is your computer?

Microsoft Windows NT 10.0.19045.0 x64

What steps can reproduce the bug?

  1. Set up the .env to contain the AWS S3 creds/info
  2. Run console.dir(await s3.list());

What is the expected behavior?

Note how in the output, the files listed contain a checksumAlgorithme instead of checksumAlgorithm, as stated in the type.

Image

What do you see instead?

The prop value should be checksumAlgorithm, as stated in the types.

Additional information

I know this is a type vs reality discrepancy, but since using the French word was probably not intended, I reckon this should be tagged as a bug instead of wrong type.
This PR seems to be the source: #16948

@tabarra tabarra added bug Something isn't working needs triage labels Apr 20, 2025
@RiskyMH RiskyMH added quick fix Something that can be fixed with low effort, but high impact s3 and removed needs triage labels Apr 20, 2025
@RiskyMH RiskyMH added good first issue Something that would be good for new contributors and removed quick fix Something that can be fixed with low effort, but high impact labels Apr 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Something that would be good for new contributors s3
Projects
None yet
Development

No branches or pull requests

2 participants