Skip to content

fix(codegen): add default aws regional endpoints for generated clients #7151

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 26, 2025

Conversation

kuhe
Copy link
Contributor

@kuhe kuhe commented Jun 25, 2025

This PR follows up with

smithy-lang/smithy-typescript#1589 adds default endpoint ruleset
#7077

The smithy-ts default endpoint ruleset is equivalent to the "customEndpoint" deprecated plugin where an endpoint is required during client instantiation.

This PR overrides the default endpoint ruleset when the model is for an AWS service. It uses an alternate default endpoint ruleset patterned after regional AWS services. In this ruleset, endpoint input for the client is not required, since the endpoint may be derived from the region and service endpoint prefix.

This supports Amazon/AWS model builds that occur outside of our SDK release platform, which normally applies the default endpoint rulesets. This default does not cover as many edge cases as the internal version, but works for typical regional services. Services with further endpoint customizations building outside of our release platform will need to include the endpointRuleSet trait in the model being built.

@kuhe kuhe requested a review from a team as a code owner June 25, 2025 21:50
@kuhe kuhe force-pushed the fix/default-aws-endpoints branch from f68cf32 to f2977f3 Compare June 25, 2025 22:11
@kuhe kuhe force-pushed the fix/default-aws-endpoints branch from f2977f3 to 4f78d85 Compare June 25, 2025 22:33
@kuhe kuhe merged commit cb4e033 into aws:main Jun 26, 2025
5 checks passed
@kuhe kuhe deleted the fix/default-aws-endpoints branch June 26, 2025 17:50
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