Skip to content

Add exec-env to User-Agent and warn on long app IDs - #876

Merged
ubaskota merged 1 commit into
smithy-lang:developfrom
ubaskota:user_agent_fix
Oct 30, 2026
Merged

ubaskota merged 1 commit into
smithy-lang:developfrom
ubaskota:user_agent_fix

Conversation

@ubaskota

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:
Read AWS_EXECUTION_ENV in the AWS user agent interceptor and add it as exec-env/<value> after lang/, as required by the User Agent 2.0 SEP. Log a warning when sdk_ua_app_id is longer than 50 characters.

Tests:
Added unit tests for:

  • exec-env being added or omitted
  • The app ID warning at 51 characters but not at 50 or when unset
  • Validated that all existing and new tests pass

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

@ubaskota
ubaskota requested a review from a team as a code owner October 29, 2026 22:42
@ubaskota
ubaskota merged commit cb66fd6 into smithy-lang:develop Oct 30, 2026
9 checks passed
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