-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[v2] User agent 2.1: Track client features #9333
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good. For the S3 high-level commands, can we track the HeadObject
calls too (unless the internal design says not to)?
Addressed in 35e6716 I considered only registering at the point of |
Ports boto/botocore#3389 and boto/s3transfer#335
I also added a CLI-specific change to wrap the CLI entrypoint under the
start_as_current_context
context manager so future work can add CLI-specific features.