Skip to content

Track feature IDs for retry modes, CBOR, and request compression #9555

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

Merged
merged 1 commit into from
Jun 18, 2025

Conversation

ashovlin
Copy link
Member

Issue #, if available: CLI-5164, CLI-6502

Description of changes: Ports the following botocore PRs:

  1. Track retry mode and gzip feature ids boto/botocore#3481
  2. Track RPC v2 CBOR feature boto/botocore#3422

Deviations:

  1. CLIv2 doesn't have a legacy retry mode, so skipped that feature ID.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@kdaily kdaily self-requested a review June 18, 2025 17:26
Copy link
Member

@kdaily kdaily left a comment

Choose a reason for hiding this comment

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

LGTM! I think some other ports came with this, at least this one:

boto/botocore#3443

and the followup

boto/botocore#3444

Adding that as a note here for our porting audit purposes.

@@ -22,6 +22,25 @@
from tests import ClientHTTPStubber


def get_captured_ua_strings(stubber):
Copy link
Member

Choose a reason for hiding this comment

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

Just confirming that this comes from boto/botocore#3443?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, yeah. I think that file had intentionally diverged between CLI and botocore, so grabbed just the bits I needed.

@ashovlin ashovlin merged commit 785bba1 into aws:v2 Jun 18, 2025
43 of 45 checks passed
@ashovlin ashovlin deleted the shovlia/feature-ids-retry-cbor branch June 18, 2025 23:26
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.

2 participants