Skip to content

Conversation

@GarrettBeatty
Copy link
Contributor

@GarrettBeatty GarrettBeatty commented Oct 23, 2025

Stacked PRs:


Add ContentLanguage to header collection of GetObjectResponse.

Motivation and Context

To make it easier to copy header values beteween GetObjectResponse and TransferUtilityDownloadResponse. Without this change i would have to manually copy contentLanguage from the property to TransferUtilityDownloadResponse's header collection rather than just copying the full header collection like we are already doing.

Testing

  1. ran test locally
  2. dry run d2b3cd95-f2ea-4a06-a535-a9ae116fb006 pass

Types of changes

  • [] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project
  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have read the README document
  • I have added tests to cover my changes
  • All new and existing tests passed

License

  • I confirm that this pull request can be released under the Apache 2 license

@GarrettBeatty GarrettBeatty force-pushed the GarrettBeatty/stacked/7 branch from a75c7d4 to 8fab5ec Compare October 23, 2025 17:37
@GarrettBeatty GarrettBeatty force-pushed the GarrettBeatty/stacked/7 branch from 8fab5ec to 8d8a118 Compare October 23, 2025 17:39
@GarrettBeatty GarrettBeatty changed the base branch from feature/transfermanager to development October 23, 2025 17:40
@GarrettBeatty GarrettBeatty changed the base branch from development to feature/transfermanager October 23, 2025 17:40
GarrettBeatty added a commit that referenced this pull request Oct 23, 2025
stack-info: PR: #4074, branch: GarrettBeatty/stacked/7
@GarrettBeatty GarrettBeatty force-pushed the GarrettBeatty/stacked/7 branch from 8d8a118 to 7876233 Compare October 23, 2025 17:52
@GarrettBeatty GarrettBeatty marked this pull request as ready for review October 23, 2025 18:03
stack-info: PR: #4074, branch: GarrettBeatty/stacked/7
@GarrettBeatty GarrettBeatty merged commit d1b3919 into feature/transfermanager Oct 24, 2025
3 checks passed
@GarrettBeatty GarrettBeatty deleted the GarrettBeatty/stacked/7 branch October 24, 2025 22:21
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