Skip to content

Support S3 unsigned body#3323

Merged
jterapin merged 57 commits intoversion-3from
unsigned-body
Jan 5, 2026
Merged

Support S3 unsigned body#3323
jterapin merged 57 commits intoversion-3from
unsigned-body

Conversation

@jterapin
Copy link
Copy Markdown
Contributor

@jterapin jterapin commented Dec 6, 2025

This PR intends to improve performance for S3's PutObject and UploadPart .


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

  1. To make sure we include your contribution in the release notes, please make sure to add description entry for your changes in the "unreleased changes" section of the CHANGELOG.md file (at corresponding gem). For the description entry, please make sure it lives in one line and starts with Feature or Issue in the correct format.

  2. For generated code changes, please checkout below instructions first:
    https://github.com/aws/aws-sdk-ruby/blob/version-3/CONTRIBUTING.md

Thank you for your contribution!

@aws aws deleted a comment from github-actions Bot Dec 7, 2025
@aws aws deleted a comment from github-actions Bot Dec 7, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 7, 2025

Detected 1 possible performance regressions:

  • aws-sdk-core.gem_size_kb - z-score regression: 405.5 -> 406.5. Z-score: Infinity

Comment thread build_tools/services.rb Outdated
Comment thread gems/aws-sdk-core/lib/aws-sdk-core/plugins/checksum_algorithm.rb Outdated
Comment thread gems/aws-sdk-core/lib/seahorse/client/net_http/patches.rb
Comment thread gems/aws-sdk-s3/spec/client_spec.rb Outdated
@jterapin jterapin marked this pull request as ready for review December 8, 2025 17:36
@jterapin jterapin changed the title [WIP] Support S3 unsigned body Support S3 unsigned body Dec 8, 2025
Comment thread build_tools/customizations.rb
Copy link
Copy Markdown
Contributor

@richardwang1124 richardwang1124 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! I think it looks good overall to me. Added a few comments.

Comment thread gems/aws-sdk-core/lib/aws-sdk-core/plugins/checksum_algorithm.rb
Comment thread gems/aws-sdk-core/lib/aws-sdk-core/plugins/checksum_algorithm.rb Outdated
Comment thread gems/aws-sdk-s3/spec/client_spec.rb Outdated
Comment thread gems/aws-sdk-core/lib/aws-sdk-core/plugins/checksum_algorithm.rb
Comment thread gems/aws-sdk-core/lib/aws-sdk-core/plugins/checksum_algorithm.rb Outdated
Comment thread gems/aws-sdk-core/lib/aws-sdk-core/plugins/checksum_algorithm.rb Outdated
Comment thread gems/aws-sdk-core/lib/aws-sdk-core/plugins/checksum_algorithm.rb Outdated
Comment thread gems/aws-sdk-core/lib/aws-sdk-core/plugins/checksum_algorithm.rb Outdated
Comment thread gems/aws-sdk-core/lib/aws-sdk-core/plugins/checksum_algorithm.rb
@jterapin jterapin merged commit b35ec5e into version-3 Jan 5, 2026
28 checks passed
@jterapin jterapin deleted the unsigned-body branch January 5, 2026 15:30
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