Skip to content

fix: Fix double space bug in frameworks metadata in user-agent header - #3413

Merged
sichanyoo merged 4 commits into
version-3from
fix/ua-double-space
Aug 6, 2026
Merged

fix: Fix double space bug in frameworks metadata in user-agent header#3413
sichanyoo merged 4 commits into
version-3from
fix/ua-double-space

Conversation

@sichanyoo

Copy link
Copy Markdown
Contributor

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

@sichanyoo
sichanyoo requested a review from a team as a code owner August 5, 2026 18:04

@jterapin jterapin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved with the context of addressing feedbacks.

Comment thread gems/aws-sdk-core/VERSION Outdated
@@ -1 +1 @@
3.254.0
3.254.1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Our automation will handle this when the change is merged in so please remove.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Gotcha, reverted it back.

Comment thread gems/aws-sdk-core/CHANGELOG.md Outdated
@@ -1,5 +1,6 @@
Unreleased Changes
------------------
* Issue - Fix double space bug in frameworks section of user-agent header.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit:

Suggested change
* Issue - Fix double space bug in frameworks section of user-agent header.
* Issue - Fix a double space appearing in the frameworks section of the User-Agent header.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Changed as suggested.

@jterapin

jterapin commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Please fix spec failures as shown below.

@sichanyoo
sichanyoo merged commit 4394f29 into version-3 Aug 6, 2026
36 checks passed
@sichanyoo
sichanyoo deleted the fix/ua-double-space branch August 6, 2026 16:37
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